Skip to content
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

Favicon might be invisible #1560

Closed
SharakPL opened this issue Nov 23, 2020 · 8 comments
Closed

Favicon might be invisible #1560

SharakPL opened this issue Nov 23, 2020 · 8 comments
Assignees
Labels
triage Do not begin working on this issue until triaged by the team

Comments

@SharakPL
Copy link

SharakPL commented Nov 23, 2020

What is the current behavior?

obraz

What changes are you suggesting?

Make favicon black and white across all GitHub services available from the browser. Otherwise it can be invisible when browser uses dark mode or a theme with dark colors.

Edit: Even though dark mode does update favicon in tabs it still doesn't change favicon in bookmarks so it really needs 2 colors:

obraz

@SharakPL SharakPL added the engineering Will involve Docs Engineering label Nov 23, 2020
@welcome
Copy link

welcome bot commented Nov 23, 2020

Thanks for opening this issue. A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

@b4mboo
Copy link
Member

b4mboo commented Feb 23, 2021

I checked the favicon's behavior in dark mode in Safari and Chrome on macOS Catalina:

favicon-in-dark-mode

Can we close this issue?

@b4mboo
Copy link
Member

b4mboo commented Feb 23, 2021

@SharakPL Thanks again for opening this issue. ✨
While we won't be able to address all possible color schemes, I wonder whether you're able to produce a situation where enabling dark mode would render the favicon invisible? If so, please feel free to reopen this issue with more details.

It's worth mentioning that this repository only serves https://docs.github.com, so other GitHub-owned services or pages are out of scope.

@b4mboo b4mboo closed this as completed Feb 23, 2021
@SharakPL
Copy link
Author

SharakPL commented Feb 23, 2021

@b4mboo maybe it works for Chrome/Edge and Safari because it relies on browser's dark mode feature, but Firefox doesn't seem to have that. Dark mode is enabled simply by activating dark theme and it looks even worse there:

obraz

Adding a background-color to svg icon or <circle ... fill="#fff"> element would fix the problem.

Just look how Google did it ;)


Actually Firefox does have dark mode feature in default theme, but it relies on OS' dark mode (Windows in my case) so only if I enable dark mode for apps in Windows settings Github favicon changes to white. I am using this theme, but I don't want dark mode for apps. So maybe it's dark mode checking script that needs fixing to consider windows specific settings...

@b4mboo
Copy link
Member

b4mboo commented Feb 24, 2021

Your observation is correct. GitHub relies on the browser to communicate the dark mode flag. Changing this behavior, however, is beyond the scope of this open source repository, which only holds the content for GitHub's documentation.

@SharakPL
Copy link
Author

@b4mboo I was aware it's not specific to @github Docs but couldn't find any better place as it does impact all sites the same way. Could you direct me to repository that can handle this issue?

@b4mboo
Copy link
Member

b4mboo commented Feb 24, 2021

@SharakPL The best way to request new features or enhancements like this one would be through GitHub's Support: https://support.github.com/contact

@SharakPL
Copy link
Author

Thanks 😊

jnidzwetzki pushed a commit to jnidzwetzki/docs that referenced this issue Oct 6, 2022
* Reworking some of these broken anchors

* missed one

* Add some fixes to try to get this to work

Co-authored-by: Charis Lam <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage Do not begin working on this issue until triaged by the team
Projects
None yet
Development

No branches or pull requests

4 participants