-
-
Notifications
You must be signed in to change notification settings - Fork 408
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
Improve experience, usability giving proposals minimum votes number #13349
Conversation
It seems like you did not give us much information about what you are trying to do here. We would appreciate it if you could provide us with more info about this issue/PR! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request does not contain a valid label. Please add one of the following labels: ['type: feature', 'type: change', 'type: fix', 'type: removal', 'target: developer-experience', 'type: internal']
decidim-proposals/app/views/decidim/proposals/proposal_votes/update_buttons_and_counters.js.erb
Fixed
Show resolved
Hide resolved
From Code perspective this is ok now, yet i have some issues regarding the usability, and i would really like to discuss with @decidim/product about the way this implemented. My issue is that when voting is enabled you get a confirmation whether to navigate away from proposal voting, even though you may just browse the proposals. This is different than the Budget functionality as on the budget this posp-up get's activated just when you actually get to participate to voting. cc @ElviaBth , @andreslucena , @microstudi , @carolromero , @NilHomedes |
What you say have lots of sense. I think you are right, maybe we should show this modal only if the user has started to vote? Is this behaviour also happening even if the user is not logged in? |
TBH, i have not checked the visitor behavior, but i have discussed a lot with @ElviaBth yesterday, so she knows my pain and doubts. @ElviaBth could you check and reply here, so that Product team can also see ? |
Yes, is the same behavior for visitor. |
@alecslupu we've discussed this and we will only do this check if there's at least one vote. This will cover the case when non logged users too. |
Ok, looking forward for a new review. |
Hi @alecslupu, I have applied the suggested changes to the exit modal behavior and look forward to your review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Merging with failing spec on Codecov
…13349) Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>
🎩 What? Why?
This PR is implementing a solution to improve participants' experience and usability by giving a minimum number of votes to proposals. The notifications with the voting rules will always be visible enough and my support may end up being counted with me being aware.
📌 Related Issues
Testing
📷 Screenshots
voting rules banner
voting rules dropdown
exit_modal