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

Sibling inserter doesn't show for focused blocks #7519

Closed
nbrummerstedt opened this issue Jun 24, 2018 · 5 comments
Closed

Sibling inserter doesn't show for focused blocks #7519

nbrummerstedt opened this issue Jun 24, 2018 · 5 comments
Labels
[Feature] Inserter The main way to insert blocks using the + button in the editing interface [Feature] Writing Flow Block selection, navigation, splitting, merging, deletion... [Type] Enhancement A suggestion for improvement.

Comments

@nbrummerstedt
Copy link

Describe the bug
When a block is focused, I don't see the sibling inserter when i hover the top of the block.

To Reproduce
Using
Using
Create a page, and create 2 blocks. Focus on the second block.
Sibling inserter will show upon hover of the upper border of the first block, but not the second.
Switch focus to the first block. Now the first block doesn't show inserter

Expected behavior
Inserter should be visible for focused blocks.

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser Chrome 67
  • Gutenberg 3.1.0
@chrisvanpatten
Copy link
Contributor

This was changed intentionally in #7220 to better handle cases where the sibling inserter would be covered by the toolbar.

@nbrummerstedt
Copy link
Author

nbrummerstedt commented Jun 24, 2018

Why not move the sibling inserter, then?
I would argue that the case where the sibling inserter is most desired is exactly when a block is focused.

@gziolo
Copy link
Member

gziolo commented Jun 25, 2018

@chrisvanpatten - I know that this change was intentional. However, it is now very counter-intuitive when you want to prepend a block. I'm not quite sure how much it is an issue in a regular flow, but definitely, it's inconvenient when doing some developer oriented testing. I wanted to share my observation, but please treat it as less important because I don't write at the moment as a regular user :)

@gziolo gziolo added the [Feature] Writing Flow Block selection, navigation, splitting, merging, deletion... label Jun 25, 2018
@gziolo
Copy link
Member

gziolo commented Jun 25, 2018

I just saw this screencast recorded by @bph in #7508:
http://recordit.co/x2XzEuyoLc

In my opinion, it only confirms, that it isn't the best UX.

@gziolo gziolo added the [Type] Enhancement A suggestion for improvement. label Jun 25, 2018
@gziolo gziolo added this to the 3.2 milestone Jun 25, 2018
@gziolo gziolo added the [Feature] Inserter The main way to insert blocks using the + button in the editing interface label Jun 25, 2018
@jasmussen
Copy link
Contributor

jasmussen commented Jun 25, 2018

The sibling inserter works below the focused block, or above and below any block. You can also choose the option to fix the toolbar to the top, in which case you can use the sibling inserter before and after a block. #7297 will also bring a menu option and keyboard shortcut inserting blocks before and after any selected block.

screen shot 2018-06-25 at 11 37 46

As such, this is intentional behavior, and it is for the time being the best solution we have for #7168. Alternate solutions are very much welcome, but they should be take into account the challenges outlined in #7168, such as making sure there are no conflicts with the toolbar on the selected block. As a bit of repo management, I'm closing this in favor of #7168 and would encourage thoughts on that ticket instead.

Noting that #7508 is a bug that we need to fix, which is definitely related to this, but separate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Inserter The main way to insert blocks using the + button in the editing interface [Feature] Writing Flow Block selection, navigation, splitting, merging, deletion... [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

4 participants