-
Notifications
You must be signed in to change notification settings - Fork 3.2k
The "unread" dot in the Chrome tab icon is so small as to be almost invisible #9829
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
Comments
Triggered auto assignment to @robertjchen ( |
Yep, definitely not ideal! Wanted to loop in @Expensify/design on this: could we possibly get a mockup of what a new "unread" indicator would look like? It sounds like we might be changing it to a green color in addition to making it larger? |
Triggered auto assignment to @shawnborton ( |
Yup and perhaps we want to add some logic here:
Though I'm not sure if we have any red brick road logic just yet... |
Posted an idea here |
Looks great! We just need to swap out https://github.com/Expensify/App/blob/main/web/favicon-unread.png and we should be good to go here. |
Won't be that simple if we are looking to add two different ones based on error or non-error state. |
Agreed- maybe we can just narrowly-scope this for just the unreads first. The error state would require some planning and discussion, especially since how the way we surface errors may be changing. |
Let me know if these images work: Notification Dot.zip |
It's beautiful, thanks! |
Need to test things out before opening it up for review. |
Update before going OOO: working through the test/screenshots checklist in the PR. |
Not overdue |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
Expected Result:
User should be able to see that there are unread messages
Actual Result:
User can see unread message indicator, but the The "unread" dot in the Chrome tab icon is so small as to be almost invisible
Workaround:
unknown
Platform:
Where is this issue occurring?
Version Number: 1.1.82-1
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @quinthar
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1657511930574589
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: