Skip to content
This repository was archived by the owner on Oct 25, 2023. It is now read-only.

Unread badge favicon#2045

Open
HaydenBL wants to merge 3 commits intozhukov:masterfrom
HaydenBL:unread-favicon
Open

Unread badge favicon#2045
HaydenBL wants to merge 3 commits intozhukov:masterfrom
HaydenBL:unread-favicon

Conversation

@HaydenBL
Copy link

I recently started using Telegram, and it's pretty good but I was getting a bit frustrated with the unread favicon indicator in the webapp. Most other chat webapps I use make it a little more obvious when you have a notification, whereas Telegram just has a mail icon with the same color scheme.

I did some Googling and found I was not the only one with this problem:

#1585
#1718

So this PR hopefully solves this issue! I was originally just gonna make it a red dot, but I saw in the one issue someone mentioned adding a count to the badge, so I figured I might as well do that while I'm messing around here.

Here's a picture of what it looks like in a tab:
Screen Shot 2021-01-11 at 8 19 22 PM

Let me know if something needs fixing!

@HaydenBL
Copy link
Author

Actually now that I'm actually taking time to read the gulpfile I'm thinking I might not need to add that since it just puts those files into the top level when you build. I'll await confirmation on whether or not I need to do anything there.

Also, since I forgot to mention this in the initial PR description, I also stopped the badge/title from flashing since it's unnecessary when the badge is there, and if anything would be kinda distracting with a red dot.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

1 participant