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

make variants movable #1398

Open
zonky2 opened this issue Apr 3, 2020 · 2 comments
Open

make variants movable #1398

zonky2 opened this issue Apr 3, 2020 · 2 comments
Assignees
@discordier @stefanheimes @zonky2
Labels
enhancement This issue is about an enhancement (aka new feature)
Milestone
Future

Comments

@zonky2
Copy link
Contributor

zonky2 commented Apr 3, 2020

make variants movable: currently, variants (children) cannot be moved to another variant or a child cannot be made a parent
@zonky2 zonky2 added the enhancement This issue is about an enhancement (aka new feature) label Apr 3, 2020
@zonky2 zonky2 added this to the 2.2.0 milestone Apr 3, 2020
@discordier
Copy link
Member

Making variants a base should not be a big deal, moving variants between base items will be harder I guess, as we will then have to manipulate not only the varbase field but also all varianting attribute values in the paste handling.

@zonky2
Copy link
Contributor Author

zonky2 commented Apr 3, 2020

we also have the problem that we only have "manual sorting" to move - you don't really want that especially since you also have to prevent child elements from becoming child elements...
you would need a special "moving icon"

@zonky2 zonky2 modified the milestones: 2.2.0, Future May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@discordier discordier

@stefanheimes stefanheimes

@zonky2 zonky2

Labels
enhancement This issue is about an enhancement (aka new feature)
Projects
None yet
Milestone
Future
Development

No branches or pull requests
3 participants
@discordier @stefanheimes @zonky2