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

Sortable lists should be more obviously sortable #826

Open
jagthedrummer opened this issue May 7, 2024 · 2 comments
Open

Sortable lists should be more obviously sortable #826

jagthedrummer opened this issue May 7, 2024 · 2 comments

Comments

@jagthedrummer
Copy link
Contributor

Currently if you super scaffold a sortable model you'll end up with an index page that is technically sortable, but there's no indication given to the user that the items can be reordered.

rails g super_scaffold Blog Team name:text_field --sortable

CleanShot 2024-05-07 at 11 10 14

I think we should probably add a defined "drag handle" (something like ↕️) to help indicate that the items are draggable.

It would also be nice to change the cursor when someone is hovered over the draggable item. Probably cursor: grab when hovered over the item, and then cursor: grabbing when the drag has been started.

The lack of dragging indicators might be at play in this comment: #825 (comment)

@swombat
Copy link
Contributor

swombat commented May 9, 2024

Agreed. I ended up disabling on most lists this since it made clicking less reliable, and gave rise to users saying "the list isn't working" because they moved the mouse a tiny bit while clicking.

I'd advise:

  • drag handle
  • light dashed outline

Between those two things it should be more obvious that this is draggable.

@swombat
Copy link
Contributor

swombat commented May 20, 2024

This from the Refactoring UI book suggests an even better solution, visually:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants