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.

Sign up for GitHub

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

Jump to bottom

Move nodes without prompting for position #830

Closed
dimaip opened this issue Aug 21, 2017 · 5 comments
Closed

Move nodes without prompting for position #830

dimaip opened this issue Aug 21, 2017 · 5 comments
Assignees
@dimaip
Labels
Discussion Enhancement
Milestone
1.0.0 must haves

Comments

@dimaip
Copy link
Contributor

dimaip commented Aug 21, 2017 •
edited
Loading

<- must-have for 1.0.0

When moving nodes now you are presented with a position selection dialog. I prefer how the move used to work in the old ui. Now every move requires a bunch of extra clicks and it's not easy to grasp the concept at first. Wdyt?
@dimaip
Copy link
Contributor Author

dimaip commented Aug 21, 2017 •
edited by daniellienert
Loading

Feedback from Daniel: "I noticed the same, and share your opinion. It’s more clear what is happening one the one hand but a lot more clicks on the other. Makes a quick resorting of the tree impossible."

@dimaip
Copy link
Contributor Author

dimaip commented Sep 1, 2017

I would suggest to target this for 1.0, but only if we have time left. Still this is a big UX issue IMO.
@skurfuerst your thoughts?

@daniellienert
Copy link
Member

@dimaip / @skurfuerst - after working with the UI as an author for some days now, I would like to push this issue again. Reordering nodes or moving them into container nodes is really painful now.

@dimaip
Copy link
Contributor Author

dimaip commented Sep 12, 2017

Yep, we agreed to re-implement tree DND properly. Will take care.

@dimaip dimaip self-assigned this Sep 13, 2017
@dimaip dimaip added this to the 1.0.0 must haves milestone Sep 13, 2017
@dimaip dimaip mentioned this issue Sep 14, 2017
Merged
1 task
@dimaip
Copy link
Contributor Author

dimaip commented Sep 14, 2017

@daniellienert reviews are welcome

@dimaip dimaip closed this as completed Sep 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dimaip dimaip

Labels
Discussion Enhancement
Projects
None yet
Milestone
1.0.0 must haves
Development

No branches or pull requests
2 participants
@daniellienert @dimaip