-
Notifications
You must be signed in to change notification settings - Fork 85
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
Use material design icon for settings icon #2211
Conversation
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.
can you add opacity: 1 !important;
so that the stile fits the trashbin?
This is a server style which I just copied. I guess the lower opacity is intendend. |
jep, but only for the files app (because the other labels have the same opacity there). |
So it was wrong all the time in every other app, because the Files app messed with the opacity? 😄 |
3cc262e
to
8f905ab
Compare
Fixed. I updated the screenshot in the first post. |
Signed-off-by: Raimund Schlüßler <[email protected]>
8f905ab
to
d72b67f
Compare
@raimund-schluessler How did you fix it? I don't see |
|
/backport to stable4 |
This PR replaces the current settings icon with the material design equivalent and also fixes the settings label alignment to the
AppNavigationItem
component. Supersedes #2210, closes #2138 and fixes the second part of #2199.After:
Before: