fairvotereform / RankIt

https://rankit.vote
1 stars 2 forks source link

Results: add threshold value as label to threshold line #149

Closed ggordn3r closed 4 years ago

ggordn3r commented 4 years ago

See screenshot. Users are not clear about the purpose of the threshold line. To make this more clear, add a label above it with the threshold value.

Screenshot (385)

proggeramlug commented 4 years ago

Implemented. The threshold is the total number for us (like 4). I'm calculating the % by dividing that number by the total vote count - is that correct?

ggordn3r commented 4 years ago

No, that's not quite right. The threshold as a % value is already stored exactly as we want it in the explanation text as: ${this.percentToString(this.winning_percentage)}

Can you just invoke that again for the label on the graph side?

proggeramlug commented 4 years ago

Fixed!