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

Column block: Add block icon (+) is hidden under column toolbar #16742

Closed
mapk opened this issue Jul 24, 2019 · 10 comments
Closed

Column block: Add block icon (+) is hidden under column toolbar #16742

mapk opened this issue Jul 24, 2019 · 10 comments
Labels
[Block] Columns Affects the Columns Block Needs Design Needs design efforts.

Comments

@mapk
Copy link
Contributor

mapk commented Jul 24, 2019

Describe the bug
There's a situation that when the column in a Column block is selected, a user can hover just in the right spot to reveal a hidden "Add block" inserter icon.

To reproduce
Steps to reproduce the behavior:

  1. Add a Column block.
  2. Add some content to a column.
  3. Click up to the Column level and hover right above the content block within the column.
  4. Observe the Inserter + icon appear but hidden under a toolbar.

Expected behavior
I expect the icon to be completely visible or not show at all. It's a weird situation.

Screenshots

column-addblock

Desktop (please complete the following information):

  • OSX
  • Firefox 68.0
  • Gutenberg 6.1.1
@mapk mapk added Needs Design Needs design efforts. [Block] Columns Affects the Columns Block labels Jul 24, 2019
@karmatosed karmatosed added Needs Design Feedback Needs general design feedback. and removed Needs Design Needs design efforts. labels Jul 29, 2019
@karmatosed
Copy link
Member

I 'think' this should just be a case of a z-index bug and maybe doesn't need design? For me, it should be visible no matter what. I'll swap out labels but happy to change that.

@ZebulanStanphill
Copy link
Member

Related: #13571

@enriquesanchez
Copy link
Contributor

Discussed during today's design triage meeting in Slack (login required).

The team agreed that applying z-index would not take care of the underlying issue. It was suggested to explore only showing block mover icons when the block is selected and to perhaps rethink sibling inserters to find different ways they can exist.

@enriquesanchez enriquesanchez added Needs Design Needs design efforts. and removed Needs Design Feedback Needs general design feedback. labels Jul 30, 2019
@mapk
Copy link
Contributor Author

mapk commented Sep 23, 2019

Only showing block movers when the block is selected is being worked on here: #16745

@mapk
Copy link
Contributor Author

mapk commented Sep 23, 2019

Actually, nevermind... that other one is a duplicate of this one which has already been merged. #17315

@mapk
Copy link
Contributor Author

mapk commented Sep 23, 2019

Just tested and this isn't resolved because the mover icons aren't the issue. Darnit! So still need to rethink the Sibling Inserter here and the interaction.

@karmatosed
Copy link
Member

@mapk would you mind testing this on master please? I am not getting it right now but this could be me missing a recreation step.

@karmatosed
Copy link
Member

It would be great to have this worked on as part of #18667

@mapk
Copy link
Contributor Author

mapk commented Dec 16, 2019

Just tested again today in master. I still see the issue. Albeit it was a bit more difficult to get to this problem though due to traversing up from nested columns seems harder nowadays without the additional parent block borders.

Single Column selected:
Screen Shot 2019-12-16 at 2 33 17 PM

and Columns block selected:
Screen Shot 2019-12-16 at 2 35 53 PM

@youknowriad
Copy link
Contributor

This seems fixed in my testing. We still have the issue that it's impossible to insert a block at the beginning of the column but this is tracked separately #16094

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Columns Affects the Columns Block Needs Design Needs design efforts.
Projects
None yet
Development

No branches or pull requests

5 participants