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

Allow config value re-ordering for lists via drag and drop #1458

Open
Abrynos opened this issue Jun 30, 2021 · 1 comment
Open

Allow config value re-ordering for lists via drag and drop #1458

Abrynos opened this issue Jun 30, 2021 · 1 comment
Labels
✨ Enhancement Issues marked with this label indicate further enhancements to the program, such as new features. 👍 PR-ok Issues marked with this label are good candidates for being accepted in a pull request. 🙏 Wishlist Issues marked with this label are wishlisted. We'd like to make them happen but they're not crucial.

Comments

@Abrynos
Copy link
Contributor

Abrynos commented Jun 30, 2021

Purpose

For some configuration properties such as FarmingOrders the order of selection matters. If I want to change the order of the currently selected values, currently I have to remove a value from the middle and add it again (placing it at the end).

Solution

Implement drag-and-drop for the selected values, which are shown below the dropdown menu.

Alternatives

Don't do anything and leave it as is.

Thank you in advance for considering this enhancement.
@Abrynos Abrynos added ✨ Enhancement Issues marked with this label indicate further enhancements to the program, such as new features. 🧐 Evaluation Issues marked with this label are currently being evaluated if they're going to be considered. labels Jun 30, 2021
@Botan626 Botan626 mentioned this issue Jun 30, 2021
Closed
@MrBurrBurr MrBurrBurr added 👍 PR-ok Issues marked with this label are good candidates for being accepted in a pull request. 🙏 Wishlist Issues marked with this label are wishlisted. We'd like to make them happen but they're not crucial. and removed 🧐 Evaluation Issues marked with this label are currently being evaluated if they're going to be considered. labels Jul 13, 2021
@MrBurrBurr
Copy link
Member

Good catch, this will be a nice enhancement for the user. The current flow for changing farming order is very user-unfriendly.

I just wonder if there is a sane way to implement this without using a library. Lets see when the time comes to implement this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
✨ Enhancement Issues marked with this label indicate further enhancements to the program, such as new features. 👍 PR-ok Issues marked with this label are good candidates for being accepted in a pull request. 🙏 Wishlist Issues marked with this label are wishlisted. We'd like to make them happen but they're not crucial.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Abrynos @MrBurrBurr