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

Missleading dragable area of the nodes in the clipboard #3183

Closed
radinamatic opened this issue Jun 1, 2021 · 2 comments · Fixed by #3330
Closed

Missleading dragable area of the nodes in the clipboard #3183

radinamatic opened this issue Jun 1, 2021 · 2 comments · Fixed by #3330
Assignees
Labels
design: complete Ready for implementation DEV: frontend P1 - important Priority: High impact on UX

Comments

@radinamatic
Copy link
Member

Observed behavior

While hovering over nodes in the clipboard, the entire item line/slot changes the background color. However, when the user tries to grab the topic/resource anywhere before (on the left) of the checkbox, with the intention to drag and drop it in the channel, they are unable to.

drag-drat

As it seems, only the area starting from the checkbox and moving towards right (in LTR UI), is draggable, and depending on how deep is the nesting of the node in the clipboard, the non-draggable area can be half or more of what is highlighted.

QA v4 - QA v4 - Kolibri Studio — Mozilla Firefox_027

In RTL UI, the problem is less pressing as the non-draggable area remains on the left edge of the screen, and it's more probable that the user will try to grab the area that is actually draggable.

QA v4 - QA v4 - استوديو كوليبري — Mozilla Firefox_028

Not sure if the RTL here is the proper flip of LTR - all checkboxes are aligned right and not indented - but it does not seem to be a big blocker, and is not related to this issue.

Expected behavior

Since the hover-highlighted space extends to the full clipboard width, I suggest one of these solutions:

  1. (optimal) Make the entire hover-highlighted area draggable.
  2. (passable) Reduce area that changes the background color on hover to match just the draggable area.

User-facing consequences

User frustration and impression that the dragging items from the clipboard is not working.

Usage Details

  • OS: Ubuntu 20.04
  • Browser: both Firefox and Chrome
  • URL: any channel

cc @khangmach & @jtamiace for thoughts 🙏🏽

@radinamatic radinamatic added P1 - important Priority: High impact on UX DEV: frontend labels Jun 1, 2021
@radinamatic radinamatic changed the title Missleading item dragable area in the clipboard Missleading dragable area of the nodes in the clipboard Jun 1, 2021
@jtamiace jtamiace added the design: todo Scoped and ready for design work to begin label Jul 2, 2021
@jtamiace jtamiace added design: complete Ready for implementation and removed design: todo Scoped and ready for design work to begin labels Jul 20, 2021
@rtibbles
Copy link
Member

Hey @d0sadata I’m using ZenHub in GitHub, click this link to join my workspace and see other features available in GitHub or download the ZenHub extension and sign up with your GitHub account.
Posted using ZenHub

@d0sadata d0sadata closed this as completed Feb 1, 2022
@d0sadata d0sadata reopened this Feb 1, 2022
@bjester
Copy link
Member

bjester commented Feb 1, 2022

@d0sadata This doesn't appear to be related to its draggable nature at all. The clipboard has a nested tree structure so clicking topics/folders expand them, and clicking resources previews them. It appears that the pointer cursor style needs refined to align with this behavior and the clipboard specific layout.

I suggest modifying <LazyListGroup> to remove or conditionalize the cursor pointer on the header, since the clipboard needs to visually indent the header content for each item.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design: complete Ready for implementation DEV: frontend P1 - important Priority: High impact on UX
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants