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

Improve drop indicator within the List view #39068

Open
richtabor opened this issue Feb 24, 2022 · 3 comments
Open

Improve drop indicator within the List view #39068

richtabor opened this issue Feb 24, 2022 · 3 comments
Labels
[Feature] List View Menu item in the top toolbar to select blocks from a list of links. Needs Design Needs design efforts. [Type] Enhancement A suggestion for improvement.

Comments

@richtabor
Copy link
Member

What problem does this address?

We currently have a bold indicator when dragging/dropping blocks within the editor, but in the List view, it is quite thin/not as easily distinguishable.

What is your proposed solution?

  1. Let's try a more consistent drop indicator here that also improves visibility of the indicator — perhaps 1.5 or even 2px wide. Perhaps even matching the focus state width (in the last screenshot).

  2. As an extra, perhaps lean into matching a corresponding animation (perhaps not as bold as the editor itself, but relative to the List view size) for the line indicator as it appears.

Current drop indicator within the editor:

CleanShot 2022-02-24 at 13 01 28@2x

Current drop indicator within List view:

CleanShot 2022-02-24 at 13 02 1

CleanShot 2022-02-24 at 13 05 05@2x

@richtabor richtabor added Needs Design Needs design efforts. [Feature] List View Menu item in the top toolbar to select blocks from a list of links. labels Feb 24, 2022
@talldan
Copy link
Contributor

talldan commented Feb 25, 2022

I noticed this disparity too recently. It'd be great to improve it. I don't think it'll be too difficult, once we have a design I think it can be considered a slightly more challenging 'Good First Issue'. 😄

There has been some work on changing the way drag and drop works here - #34022. But I think it'd also be fine to ship an interim improvement to the indicator.

For anyone interested in contributing, the code is here:

@richtabor
Copy link
Member Author

Probably worth a separate issue, but I've noticed that in Safari the drop indicator is not indicative of where the drop actually happens.

CleanShot.2022-07-06.at.16.10.33.mp4

@talldan
Copy link
Contributor

talldan commented Jul 8, 2022

@richtabor Yep, there was an issue reported here for that - #42058. It seems to be related to recent changes to the popover component.

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. Needs Design Needs design efforts. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

3 participants