Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blocks: Floated button block takes more space when it gets focus #3869

Closed
gziolo opened this issue Dec 8, 2017 · 3 comments
Closed

Blocks: Floated button block takes more space when it gets focus #3869

gziolo opened this issue Dec 8, 2017 · 3 comments
Labels
[Feature] Blocks Overall functionality of blocks [Type] Bug An existing feature does not function as intended
Milestone

Comments

@gziolo
Copy link
Member

gziolo commented Dec 8, 2017

Issue Overview

Floated button block takes more space when it gets focus. This causes the following block to display differently depending on the focus of the button.

Steps to Reproduce (for bugs)

  1. Add a new button.
  2. Make it floated to left.
  3. Add another block.
  4. Click on the button block and observe how 2nd block changes.

Browser: Chrome
System: macOS

Expected Behavior

Floated button block has always the same width.

Current Behavior

Issues around the floated button, because it’s born with a min-width so it looks weird.

Screenshots / Video

screen shot 2017-11-30 at 10 11 37
screen shot 2017-11-30 at 10 12 51

/cc @jasmussen

@gziolo gziolo added [Feature] Blocks Overall functionality of blocks [Type] Bug An existing feature does not function as intended labels Dec 8, 2017
@jasmussen
Copy link
Contributor

After thinking about this, I think maybe this is okay.

For the time being it is intentional that an unselected block is a preview, and a selected block can show extended configuration controls, in this case an input field for the URL. Many other blocks are likely to do similar things, and so long as the preview looks accurate when the block is unselected perhaps that's okay.

However I also can't quite perceive all the ways this can go wrong. We might indeed need some tweaks to how floats are handled, in case they break more deeply than the above.

As such, I would suggest we keep this issue open, but on the backburner.

@karmatosed karmatosed added this to the Merge Proposal milestone Jan 25, 2018
@jasmussen
Copy link
Contributor

While the recent improvement to floats hasn't fixed this issue, it has nonetheless made the experience much less jarring and more predictable:

float-block-preview-changes

Clicks on buttons do what you expect them to, and although it's perhaps not ideal that things shift around a lot when you select them, it is a design principle that a selected block can show additional controls, in this case a URL bar.

As such, I'm going to close this for now, we can then revisit if need be. One solution would be to redesign the URL bar of the button.

@gziolo
Copy link
Member Author

gziolo commented Feb 23, 2018

Much better, thanks 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Blocks Overall functionality of blocks [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

3 participants