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

Move buttons are hard to use within a nested block #16617

Closed
shaunandrews opened this issue Jul 16, 2019 · 5 comments · Fixed by #19344
Closed

Move buttons are hard to use within a nested block #16617

shaunandrews opened this issue Jul 16, 2019 · 5 comments · Fixed by #19344
Labels
Needs Design Feedback Needs general design feedback. [Type] Bug An existing feature does not function as intended

Comments

@shaunandrews
Copy link
Contributor

Describe the bug
Using a meda+text block, its hard to move the paragraphs around without actually focusing on them — the move buttons appear when hovered, but you can't actually interact with them. Here's a GIF that hopefully helps you understand the problem:

confusing buttons

Alternatively, sometimes the opposite is true — the move buttons get in the way of resizing the image. Here's another GIF:

confusing buttons 2

The behavior seems to change depending on what block you currently have focused, but I can't seem to understand the logic behind the change.

@shaunandrews shaunandrews added the [Type] Bug An existing feature does not function as intended label Jul 16, 2019
@karmatosed karmatosed added Needs Design Feedback Needs general design feedback. Needs Decision Needs a decision to be actionable or relevant labels Jul 16, 2019
@mapk
Copy link
Contributor

mapk commented Jul 23, 2019

Talked about this in today's design triage.

Let's explore an option to only show the move icons when the block is selected. Although I feel like this is a workaround, it may be the workaround we need. A try/ PR could help communicate this.

Related: #16718 and kind of #16646

@mapk mapk removed the Needs Decision Needs a decision to be actionable or relevant label Jul 24, 2019
@mapk
Copy link
Contributor

mapk commented Aug 6, 2019

Another related issue with weirdness around these controls:
#16742

@karmatosed
Copy link
Member

Another issue with spacer:

68587735-ad1df200-047f-11ea-860a-0afd7cf9ac46

@karmatosed
Copy link
Member

I think this can be solved as part of the iterations here: #18667

@jasmussen
Copy link
Contributor

Yes, this would definitely be fixed by #18685!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Design Feedback Needs general design feedback. [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants