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

feat(app-sidebar): allow to define inline for secondary buttons #5458

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

szaimen
Copy link
Contributor

@szaimen szaimen commented Apr 8, 2024

Signed-off-by: Simon L <szaimen@e.mail.de>
@szaimen szaimen added the 2. developing Work in progress label Apr 8, 2024
@szaimen szaimen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Apr 8, 2024
@szaimen szaimen marked this pull request as ready for review April 8, 2024 13:08
@szaimen szaimen added enhancement New feature or request design Design, UX, interface and interaction design labels Apr 8, 2024
Copy link
Contributor

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also here, relevant comment at nextcloud/calendar#5906 (review)
→ Showing so many icons without labels makes things quite difficult to understand for normal people, and not all of these actions are very important. Thus there is a reason some are hidden. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews design Design, UX, interface and interaction design enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants