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

UI: suggestions for 'DAO -> Governance -> Browse open proposals' page #2071

Closed
BravoIndia opened this issue Dec 6, 2018 · 10 comments
Closed

Comments

@BravoIndia
Copy link
Contributor

In version 0.9.0 under DAO -> Governance -> Browse open proposals

#1: The following red rectangle area is quite confusing and unintuitive upon first use:

image

I suggest putting the title "My vote" in the area highlighted below:

image

#2: There should be a confirmation button upon choosing a vote, and perhaps a pop up message box (e.g. "You're about to change your vote to: Accepted. Are you sure?")

#3: If in the 'Link' column, the entered text is not a URL, the following pop-up should not appear:

image

#4: It also may be good to change "Open the web page and don't ask again" to "Open the web page" and include a "Don't ask again" checkbox (which will apply to either selection if checked, so those wanting to simply copy don't need to click repeatedly either).
For example:

image

@devinbileck
Copy link
Member

Good suggestions!
2. I wonder if another popup is really necessary? It is simple enough to undo or change your vote afterwards.
3. May actually want to add input validation and only accept URL links, rather than random text. I recall seeing some errors in the debug log when a proposal was not a URL.

@BravoIndia
Copy link
Contributor Author

Thank you,
2. I agree, it doesn't have to be a popup. But if not, there should at least be a confirmation button, no?
I was thinking that changing a vote on a proposal should take more than one click, to prevent accidentally changing a vote on a certain proposal without noticing (while looking at others, for example).
3. Makes sense. @ManfredKarrer mentioned in his post that "we don’t want to spam Github with entrties for testing purposes". If non-URL are done away with, maybe we can use other sites during testing?

@devinbileck
Copy link
Member

  1. Thats true, confirmation may be useful.
  2. Yeah I think just using random links should be good enough for testing, whether that be to existing GitHub proposals or whatever link you want to put in there.

@ripcurlx ripcurlx added the in:gui label Dec 6, 2018
@ManfredKarrer
Copy link
Contributor

ManfredKarrer commented Dec 6, 2018

@BravoIndia Thanks for the suggestions! Basically agree to all beside the confirmation popup for vote change, but no strong opinion on that (if at least with a dont show again) - but generally we want to avoid popups if not absolutely needed as there are already way too many.

I think a final confirmation with overview screen would be good like at create offer the final confirm window with all data. Then the user should see if his votes have been as intended.

Do you have a suggestion for Re 1?

Yes links are not validated yet but we could handle the exception to not log. I think its the only place in Bisq where user can edit the URL so on other places its exceptional that a URL does not work.

Re 3: Yes agree would be good to use more realistic situation. Maybe we also can create a dao-testing repo which gets deleted later? Or any other idea for a place where its easy without extra registration to post links? Maybe pastebin?

@sqrrm
Copy link
Member

sqrrm commented Dec 6, 2018

  1. Agree
  2. I don't like popup, but perhaps some kind of indication that a vote was changed could be good
  3. We could create one issue for proposal testing and just add a comment in the issue and link to that one

@sqrrm
Copy link
Member

sqrrm commented Dec 7, 2018

I have fixes for the header and jumping vote icons on scrolling.

Still not sure what's the better way to show a changed vote, I'm thinking something like changing the background color of the vote for 10s after clicking to change the vote. Then it's obvious that the vote was recently changed but not too annoying.

@ManfredKarrer
Copy link
Contributor

ManfredKarrer commented Dec 7, 2018

@sqrrm Great that u fixed the scroll issue! The animations use in checkboxes of the redisigned version might be useful. Bit not sure if its easy to implement. Would focus on higher prio stuff atm...

@BravoIndia
Copy link
Contributor Author

All good ideas, thank you everyone. @sqrrm Great thought to change the background color for 10s after a vote change, and thank you for implementing the header and jumping vote icon fixes!

@stale
Copy link

stale bot commented Mar 7, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the was:dropped label Mar 7, 2019
@stale
Copy link

stale bot commented Mar 14, 2019

This issue has been automatically closed because of inactivity. Feel free to reopen it if you think it is still relevant.

@stale stale bot closed this as completed Mar 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants