-
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
Fix menu font for apps #1543
Fix menu font for apps #1543
Conversation
Signed-off-by: GretaD <gretadoci@gmail.com>
470287e
to
a846e65
Compare
this to be merged after this #1542 |
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.
It's been bothering me for months! ^^'
Yes pleeaaaase ! 🚀 😍
/backport to stable2 |
doesn't seem to fix nextcloud/spreed#4330, so the problem for the latter might be elsewhere |
not sure why the action buttons in the talk menu would behave differently, it seems that the font size of the "button" is stronger than the one of the "span" that's inside. I had to move the font-size from the
|
Maybe this and inherit in the title/text elements? |
with the above diff on top of the PR it works for me. if it also works for mail then we could go with that ? |
Signed-off-by: GretaD <gretadoci@gmail.com>
a846e65
to
ba2992e
Compare
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.
👍 looks good now also for the Talk menu, thanks
This fixes it for mail
before
after
before
after
before
After