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

Enable the block movers in List View and add e2e tests #22287

Closed
Tracked by #29733
talldan opened this issue May 12, 2020 · 4 comments
Closed
Tracked by #29733

Enable the block movers in List View and add e2e tests #22287

talldan opened this issue May 12, 2020 · 4 comments
Labels
[Feature] List View Menu item in the top toolbar to select blocks from a list of links. [Status] Blocked Used to indicate that a current effort isn't able to move forward [Type] Task Issues or PRs that have been broken down into an individual action to take

Comments

@talldan
Copy link
Contributor

talldan commented May 12, 2020

Description

#18014 introduced block movers for the block navigation. Currently they're enabled in the Navigation Block, so that they can be tested and any issues ironed out, but at some point could be introduced in the general purpose block navigation in the editor.

This should be a simple task, but as part of that, it'd be prudent to also include e2e tests for the feature.

I've labelled this as Blocked, which can be removed when we're comfortable enabling the feature.

@talldan talldan added [Type] Task Issues or PRs that have been broken down into an individual action to take [Status] Blocked Used to indicate that a current effort isn't able to move forward [Feature] List View Menu item in the top toolbar to select blocks from a list of links. labels May 12, 2020
@talldan talldan changed the title Enable the block movers in the main block navigation Enable the block movers in the main block navigation and add e2e tests May 12, 2020
@SchneiderSam
Copy link

Oh yes! So useful!!!

@vcanales vcanales self-assigned this Jul 13, 2021
@talldan talldan changed the title Enable the block movers in the main block navigation and add e2e tests Enable the block movers in List View and add e2e tests Mar 9, 2023
@richtabor
Copy link
Member

I noticed we have a showBlockMovers prop in the ListViewComponent.

Has this been considered for the List view lately? cc. @WordPress/gutenberg-design

CleanShot 2023-04-12 at 12 33 37

@jasmussen
Copy link
Contributor

They always felt a bit fiddly to me due to their small hit area, which is tricky to fix. If I'm using a mouse to move things around, drag and drop in the list view feels superior. Which, to be clear isn't a knock on the movers: in many times the explicit up/down action can be more precise, whereas drag and drop can be imprecise. But combined with the separate discussions on the list view becoming hard to navigate in deep nesting contexts, I'd love to get a bit further with improving those pieces first.

@richtabor
Copy link
Member

in many times the explicit up/down action can be more precise, whereas drag and drop can be imprecise.

Yes, exactly. I've found my flow is to select a block in List view, see which block/group is selected within the content area, then drag it around there in the canvas.

Perhaps if drag and drop were improved, and group naming in List View #42605 happens, that would add enough confidence to not look for better precision placement controls.

@talldan talldan closed this as not planned Won't fix, can't repro, duplicate, stale Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] List View Menu item in the top toolbar to select blocks from a list of links. [Status] Blocked Used to indicate that a current effort isn't able to move forward [Type] Task Issues or PRs that have been broken down into an individual action to take
Projects
None yet
Development

No branches or pull requests

5 participants