-
Notifications
You must be signed in to change notification settings - Fork 460
Confusing prominent buttons on poll dialogue #10893
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
Comments
I guess that is a general problem with theming atm. But yeah gets more complicated when the buttons are next too each other. |
Not only with theming, as I tried to point it out. Such uncommon but decisive action could be hidden behind a ⚙️ icon or something, but not be shown by default in such dominant manner. |
This sounds like the best option from the things still doable atm |
The menu button is also doable ( NcActions ) |
yeah but I don't want the button to submit the poll to be that hidden. |
I think that this is us being too permissive with themes. All "error" buttons are going to look like primary with such theming colors. I don't think that hiding all of them is a workable solution. cc @nextcloud/designers |
I think the |
How to use GitHub
Steps to reproduce
Expected behaviour
I shall not be confused with an destructive action i cannot undo.
Actual behaviour
When I am about to sumbit my poll the primary color and the big red end button are very similar. But because the submit button is disabled before I select the option, I see the change and survive this step.
Then, I read the results and want to close the dialogue, so i click on the lower right button, expecting close. While I do not expect to be able to close the poll (I did not start, but am moderator), I also would expect a close button there, rather an end one.
Eventually, I cannot undo ending the poll.
While you can argue it is my fault to have an orange primary color, the styling, positioning and incorrectability of my action may be considered nevertheless.
Talk app
Talk app version: 18.0.0-beta2
Browser
Browser name: Firefox
Browser version: 119.0.1
Server configuration
Nextcloud Version: 28.0.0 beta 2
The text was updated successfully, but these errors were encountered: