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

Quorum text improvement on mobile #2687

Closed
lukebp opened this issue Dec 6, 2021 · 1 comment · Fixed by #2695
Closed

Quorum text improvement on mobile #2687

lukebp opened this issue Dec 6, 2021 · 1 comment · Fixed by #2695
Labels
pi-not-deployed This item has been merged into master, but has not been deployed to production yet. UX

Comments

@lukebp
Copy link
Member

lukebp commented Dec 6, 2021

A user provided feedback that quorum text on mobile is confusing. 1997 votes left does not mention that this is how many votes are left before a quorum is reached, which implies that there are only a total of 1997 votes left.

image
@lukebp lukebp added bug A bug that made it into a production environment. 91cfcc8 UX and removed bug A bug that made it into a production environment. labels Dec 6, 2021
@lukebp
Copy link
Member Author

lukebp commented Dec 6, 2021 •
edited
Loading

After some internal discussion, we decided to go with two lines on top of each other. The first line is the quorum requirement. The second line displays the number of cast votes and the number of eligible votes.

8000 votes until quorum
1015/40960 votes

quorum reached!
9015/40960 votes

This should be applied to both mobile and desktop.

@tiagoalvesdulce tiagoalvesdulce mentioned this issue Dec 20, 2021
Merged
@tiagoalvesdulce tiagoalvesdulce added the pi-not-deployed This item has been merged into master, but has not been deployed to production yet. label Jan 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
pi-not-deployed This item has been merged into master, but has not been deployed to production yet. UX
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: quorum text tiagoalvesdulce/politeiagui
2 participants
@lukebp @tiagoalvesdulce