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

Show number of votes on claims #559

Closed
henriquecbuss opened this issue Jun 8, 2021 · 22 comments · Fixed by #594
Closed

Show number of votes on claims #559

henriquecbuss opened this issue Jun 8, 2021 · 22 comments · Fixed by #594
Assignees

Comments

@henriquecbuss
Copy link
Member

Is your feature request related to a problem? Please describe.
In order to know how many people have voted on a claim, people have to click on the claim, open the claim page, and see who has already voted

Describe the solution you'd like
Show the number of votes on the claim card and on the claim page

Describe alternatives you've considered
N/A

Additional context
N/A

@henriquecbuss
Copy link
Member Author

@rafachadud @juramos-2020 do we have a design for this already?

@juramos-2020
Copy link

Hi @NeoVier @lucca65 @rafachadud

Now that we have the claim opening a modal, is it still necessary to have this information on the number of votes on the card?

https://www.figma.com/file/LTaoCg6DKv6EPQAlt6Fngs/New-analysis-page?node-id=64%3A1413

As you can see on this screen, when you open the modal, the number of people who voted for the claim appears

@lucca65
Copy link
Member

lucca65 commented Jun 8, 2021

Now that we have the claim opening a modal, is it still necessary to have this information on the number of votes on the card?

I still think this is needed, because a validator has no way to know if this action requires 3,5, 7 votes.

also since "no one" looks like an user, it can be misleading

As you can see on this screen, when you open the modal, the number of people who voted for the claim appears

It shows who voted on what but not the number done and the number required

@juramos-2020
Copy link

juramos-2020 commented Jun 8, 2021

In this case, the number done and the number required should they appear on the card or when opening the modal? @lucca65

@lucca65
Copy link
Member

lucca65 commented Jun 8, 2021

I think the numbers could appear on the modal, but on the cards we could have a progress bar, this way validators can quickly inspect the ones needing more attention, without having to open them up or read anything.

here is a quick video with several cards with progress bars on it: https://www.youtube.com/watch?v=juLk17Awr_Q

@henriquecbuss
Copy link
Member Author

I think it would be nice to have a progress bar like the one from GitHub:
2021-06-08-154357_129x31_scrot
Where it shows the number of people that voted yes, the number of people that voted no and the number of people that still have to vote

@lucca65
Copy link
Member

lucca65 commented Jun 8, 2021

+1! I like the idea of having a notion on how its going just by glancing at it!

@juramos-2020
Copy link

that's right. I also think it makes more sense if this information appears on the card.

we already have something similar on the governance screen. I can do something like this:

https://www.figma.com/file/7Ci3rdDNQQ7KabYyDLE3LY/Governance?node-id=4%3A445

@lucca65
Copy link
Member

lucca65 commented Jun 8, 2021

I like it @juramos-2020, it could be a little different as we have approvals/reprovals and then missing votes, which could be gray or something like it!

Also we can have a different way to showing it when voting is completed

because lets imagine it requires 5 votes.

3 positives and 2 negatives are complete
3 positives and 2 pending is also complete
3 negatives and 2 positive too.. and so on

completion means you don't have to do anything else, but might not "fill the bar" you know?

@juramos-2020
Copy link

Yes, I know.

I can create a proposal until friday. Can be?
@lucca65 @NeoVier

@henriquecbuss
Copy link
Member Author

Sure @juramos-2020, thanks 😄

@juramos-2020
Copy link

Hi @lucca65 @NeoVier

What do you think?

https://www.figma.com/file/LTaoCg6DKv6EPQAlt6Fngs/New-analysis-page?node-id=230%3A0

Maybe on this screen we can leave “No One” just as a tag. What do you think?

https://www.figma.com/file/LTaoCg6DKv6EPQAlt6Fngs/New-analysis-page?node-id=240%3A3130

@lucca65
Copy link
Member

lucca65 commented Jun 9, 2021

I love the bars! the colors are bright and visible! Do you think we could bring the status tag (approved, disapproved, pending) more close to the progress bar?

@henriquecbuss
Copy link
Member Author

I love it @juramos-2020, good job! 🔥

@juramos-2020
Copy link

juramos-2020 commented Jun 10, 2021

Hi @lucca65

I made an example to show you what it would like with the tag next to the progress bar. The impression it gives me is that have two elements talking the same thing.

https://www.figma.com/file/LTaoCg6DKv6EPQAlt6Fngs/New-analysis-page?node-id=265%3A312

I was think about this... What if on this screen (which shows the already analyzed claim) we used only one of these elements?
We could just use the tag.

Even because the progress bar arose from the need for administrators to quickly analyze those claims that still needed votes.
So I think it makes sense for it to be only in the "claims that need to vote" screen.
In the "claim analyzed" screen have only the tag and when clicking "more details" the admin will have the details of the votes.

What do you thing?

https://www.figma.com/file/LTaoCg6DKv6EPQAlt6Fngs/New-analysis-page?node-id=269%3A818

@henriquecbuss
Copy link
Member Author

@juramos-2020 Not sure how I feel about not showing the bars on the "Analyzed" page... some people might still want to see how the voting is going, or just know the result from a glance. Maybe we could incorporate the function of the tag on the bar itself and get rid of the tag? What do you think @juramos-2020 @lucca65?

@lucca65
Copy link
Member

lucca65 commented Jun 12, 2021

i love this solution, because the way the tag is, it's not as visible as it could be...

@juramos-2020
Copy link

juramos-2020 commented Jun 12, 2021

It's ok. I'll send the screen change tomorrow okay? I'm not home right now 😅

@juramos-2020
Copy link

@victorgdev
Copy link
Contributor

@lucca65 and @NeoVier can I tackle this issue?

@henriquecbuss
Copy link
Member Author

@victorgdev go ahead bro! This one should actually be a good first issue 😅

You've already seen the part of the code that deals with claims, so you kind of know where to find information on approved/disapproved/pending votes each claim has (and we already have all that information on the frontend), so this one should be 🍰

@lucca65
Copy link
Member

lucca65 commented Jun 23, 2021

+1000

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants