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

Allow a selection of tracks #125

Open
Tbaut opened this issue Sep 11, 2024 · 1 comment · May be fixed by #188
Open

Allow a selection of tracks #125

Tbaut opened this issue Sep 11, 2024 · 1 comment · May be fixed by #188
Labels
Priority: 🟠 P1 Added to issues and PRs relating to a high severity bugs. Type: 🏗️ Feature Added to issues and PRs to identify that the change is a new feature.

Comments

@Tbaut
Copy link
Contributor

Tbaut commented Sep 11, 2024

Creating this as I believe it'll be one of the most demanded feature: track selection
Since our delegating system already has the code to handle everything needed to only delegate on the right tracks (per default it's all of them, see getDelegateTx

const tx = getDelegateTx({
) all that's missing is a nice UI.

Note: Every track should be selectable, the transaction logic is clever enough already to undelegate/unvote if needed.

I guess we can do a very simple "Advanced" menu in a ContentReveal and show a list of checkboxes with tracks (on 3 or 4 columns?) with their name and number. By default they're all selected.

@Tbaut Tbaut added Priority: 🟠 P1 Added to issues and PRs relating to a high severity bugs. Type: 🏗️ Feature Added to issues and PRs to identify that the change is a new feature. labels Sep 11, 2024
@Tbaut
Copy link
Contributor Author

Tbaut commented Sep 11, 2024

e.g something like this (no need for info tooltip for now)
image

@uDaiko uDaiko linked a pull request Oct 15, 2024 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 🟠 P1 Added to issues and PRs relating to a high severity bugs. Type: 🏗️ Feature Added to issues and PRs to identify that the change is a new feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant