-
-
Notifications
You must be signed in to change notification settings - Fork 476
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
Tray: unread messages icon fix for linux #1365
base: main
Are you sure you want to change the base?
Conversation
9058454
to
9ddde50
Compare
So the issue was in the |
Thanks for the fix! Could you please update your commit message to match the commit style guidelines? Also, please add before/after screenshots to the PR description. |
9ddde50
to
45a4c02
Compare
Done! Let me know if there are any other changes... |
I've had this problem for a while, under both X11 and Wayland, using a 4K screen with my display scaled to 200%. I wanted to confirm that the fix here works for me--thanks @Aitchessbee Would love to see this PR included in the next release. Thanks for all the great work on Zulip! |
@andersk are you up for reviewing this proposed one-line change? |
I just started using the v5.12.0 release that came out yesterday. It doesn't look like this fix made it in--the unread icon is broken. I had rebuilt the previous release with the patch here and have been running it since August. It's working very consistently. I can do that again, but I wanted to ask if there's any chance to sneak this fix in a new patch release? Thanks! |
Fixes #1364
You have tested this PR on: