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

UX Improvement - better management of placeholder location #394

Closed
2 tasks
mfeherpataky opened this issue Mar 24, 2018 · 4 comments
Closed
2 tasks

UX Improvement - better management of placeholder location #394

mfeherpataky opened this issue Mar 24, 2018 · 4 comments
Labels
feature New feature or enhancement help wanted

Comments

@mfeherpataky
Copy link
Contributor

mfeherpataky commented Mar 24, 2018

My users and I, really appreciate a placeholder in the place of a valid dropTarget. Styling is super-easy, and assigning the dimensions of the dragItem to the placeholder is super useful feature.

Today placeholders are placed when a dragItem enters a valid dropTarget, but not removed when the dragItem leaves the location. This leads to a user-confusion as where the dragItem would be dropped when released once it is somewhere outside of a valid dropTarget - would it be the last valid location (where the placeholder still is), would it be the initial location? Finally it is the original location, but it is nowhere clear to first time users.

I propose a following Usability Improvement:

  • First step would be to remove placeholders when a dragItem is no longer over a valid dropTarget.
  • Finally the placeholder should go back to the originalContainer and its initial index (unless a dragItem was cloned from a copy-only list).
@lukasoppermann
Copy link
Owner

Hey @mfeherpataky, I totally agree. I would like to at some point also allow for the possibility to remove elements when dragged outside a list and dropped without a valid target. But this is probably to much to consider now.

However, I would really appreciate those improvements.

@lukasoppermann lukasoppermann added feature New feature or enhancement help wanted labels Mar 24, 2018
@kaffarell
Copy link
Collaborator

I think this issue is resolved...
Can I close it @lukasoppermann ?

@lukasoppermann
Copy link
Owner

Sadly it is not resolved. Maybe we could split this out into 2 different issues and add a better explanation to them.

@lukasoppermann
Copy link
Owner

Closing this in favour of #553 and #554

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or enhancement help wanted
Projects
None yet
Development

No branches or pull requests

3 participants