-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Show quick permission subline on newline #43333
Conversation
nfebe
commented
Feb 4, 2024
Before | After |
---|---|
![]() |
![]() |
- Show quick permission details/explanations on subline
- Fix icon colors, should be white (or component set color) when highlighted.
- Resolves: Custom permission subline displays as part of headline on master #43225
/compile |
501b3fd
to
62d08cd
Compare
- Show quick permission details/explanations on subline - Fix icon colors, should be white (or component set color) when highlighted. Signed-off-by: fenn-cs <[email protected]>
62d08cd
to
60959db
Compare
/compile |
Signed-off-by: nextcloud-command <[email protected]>
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.
IMO, to me, this change looks way cleaner than having the smaller text right next to the big text. I will cc desginers to get them on the same page, though. cc @nextcloud/designers
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.
Yes, looks much nicer from design perspective (didnt test and didnt review the code)
backport? |
think at least for 28! 27 seems fine. |
/backport to stable28 |