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

Governance UI Improvements #168

Merged
merged 9 commits into from
Aug 6, 2023
Merged

Governance UI Improvements #168

merged 9 commits into from
Aug 6, 2023

Conversation

JSKitty
Copy link
Member

@JSKitty JSKitty commented Aug 4, 2023

Abstract

This PR introduces a new 'Congrats' popup upon finalising a proposal, with the Vote Hash included (which was previously missing, meaning this PR resolves #167).

image

It also renames "Title" to "Name" in the proposal creation process, to reduce any confusion from escaping PIVX's older conventions and terminology, resolving #166 too.

image

And lastly, as suggested by designer YuurinBee, this PR adds a bunch of UI enhancements to the Proposals list, such as: status colour-coding, vertical centre cell alignment, proposal receiver address along with larger White title, and light colour changes.

image

What does this PR address?

It fixes a UX issue where the Vote Hash was unobtainable, it makes the proposal submitter feel awesome with a friendly 'Congratulations' dialog and a mini DAO help tip, while also correcting some confusing UI elements and naming schemes.

What features or improvements were added?

A new 'Congrats' dialog after finalising a PR, as well as fixing and improving multiple other Governance UI elements.

And other minor tweaks like changing "Title" to "Name" and fixing missing translation templating.
@JSKitty JSKitty added Bug This is either a bugfix (PR) or a bug (issue). Enhancement New feature or request labels Aug 4, 2023
@JSKitty JSKitty requested review from Liquid369, Duddino and a team August 4, 2023 14:51
@JSKitty JSKitty self-assigned this Aug 4, 2023
@JSKitty JSKitty linked an issue Aug 4, 2023 that may be closed by this pull request
@JSKitty
Copy link
Member Author

JSKitty commented Aug 6, 2023

This PR (and consecutive August PRs) will be merged without reviews.

@JSKitty JSKitty merged commit e751bc1 into master Aug 6, 2023
4 checks passed
@JSKitty JSKitty mentioned this pull request Aug 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is either a bugfix (PR) or a bug (issue). Enhancement New feature or request
Projects
1 participant