-
Notifications
You must be signed in to change notification settings - Fork 94
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
Sidebar controls: bad UX #523
Comments
cc @nextcloud/designers as right now we still use the viewer app. However I was already thinking if we should just take care of the modal display manually as the sidebar can now be triggered individually. This would also help a lot with #82 |
@juliushaertl What is wrong with the sidebar being opened by the viewer? |
Very related to nextcloud/viewer#328 in general.
👍 I’d say it should be the other way around, that icon should be adjusted.
Yep
👍 Additionally:
|
Fully agree with @jancborchardt 's suggestions above |
But the sidebar already have the close button, is hiding it good enough? :) |
no, see 3. |
This is fixed these days in the viewer |
At start there are 2 buttons:
After opening the sidebar, the Open sidebar button still has the tooltip Open sidebar, even though the sidebar is already open. Clicking on the button does nothing.
In order to close the sidebar however, you don't have to click on the nearby Close button, but on the close button within the sidebar. (side note: Close ≠ close --> case sensitivity)
Here's my suggestion:
The text was updated successfully, but these errors were encountered: