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

Voting: add support / approval copy suggestion #1110

Merged
merged 1 commit into from
Apr 7, 2020

Conversation

Evalir
Copy link
Contributor

@Evalir Evalir commented Apr 3, 2020

Spring cleaning 🌸, closing #1346 over at aragon/aragon.

Your affinity for fixes is apparent. I'm confident you'll find a typo there as well.

Adds suggestion for the voting copy regarding approval & support by @john-light . Maybe we want a bit more discussion around it?

Screenshots:
Screen Shot 2020-04-07 at 2 24 09 PM

Screen Shot 2020-04-07 at 2 24 22 PM

@bpierre bpierre requested review from dizzypaty and removed request for bpierre April 7, 2020 11:03
Copy link

@dizzypaty dizzypaty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ✅

@Evalir Evalir merged commit f53a70a into master Apr 7, 2020
@Evalir Evalir deleted the voting-support-copy-fix branch April 7, 2020 12:23
Copy link
Contributor

@sohkai sohkai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Evalir Could we have a screenshot accompanying these types of changes?

Wanted to check how this looks, and whether we would prefer having a space after the > :).

@Evalir
Copy link
Contributor Author

Evalir commented Apr 7, 2020

Ah yes, I'm actually putting up screenshots for these changes; may have missed posting these. I'll edit the original PR message to have them.

@sohkai
Copy link
Contributor

sohkai commented Apr 7, 2020

Right... so in this case, we can see that the title is now overflowing into two lines and is most likely not what we'd like:

Screen Shot 2020-04-07 at 9 01 22 PM

To get that character back... maybe we could rephrase to (needs >50% support) cc @dizzypaty?

@Evalir
Copy link
Contributor Author

Evalir commented Apr 7, 2020

Seems that this would work!
Screen Shot 2020-04-07 at 3 08 13 PM

But in fact, if the numbers are too big, it doesn't work at all:
Screen Shot 2020-04-07 at 6 50 43 PM

To get around this, I believe we have two options:

  • Remove the >
  • Change the font size to be able to accommodate the characters.

@sohkai
Copy link
Contributor

sohkai commented Apr 7, 2020

Another option is to drop the "support" or "quorum" from the copy, given that it's already in context due to the Box's title.

We could also add Help tooltips to explain both support and quorum, given that they are typically overloaded terms and confusing for users who have not read our definitions for them.

cc @dizzypaty

@Evalir
Copy link
Contributor Author

Evalir commented Apr 7, 2020

This makes sense. Something like 99.62% (needs >50%)

I'd say we only need to wait for @dizzypaty 's thoughts and we can do a quick PR.

@john-light
Copy link
Contributor

@Evalir while we're modifying this section's copy, could we change Relative Support % in the box's title to Support %? I'm not sure what "Relative" is supposed to mean here, and anyways AFAIK we don't call it "Relative Support" anywhere else (neither in the Permissions for this parameter nor the setup flow when first setting this parameter).

@Evalir
Copy link
Contributor Author

Evalir commented Apr 8, 2020

I believe we can @john-light ; we indeed are not using the "relative term" anywhere else, even in the internal contract documentation.

@sohkai
Copy link
Contributor

sohkai commented Apr 8, 2020

We use "relative percentage" when describing support during onboarding:

Support is the relative percentage of tokens that are required to vote “Yes” for a proposal to be approved

This was meant to contrast the absolute nature of calculating quorum (support is relative to current voters, quorum is absolute against all possible voters).

But I agree, we should be consistent with "support". Perhaps we can lift the Help tooltips directly from onboarding?

ramilexe pushed a commit to ConsiderItDone/aragon-apps that referenced this pull request Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants