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 is not working in our repository #241

Open
bartekpacia opened this issue Oct 25, 2022 · 8 comments
Open

Favicon is not working in our repository #241

bartekpacia opened this issue Oct 25, 2022 · 8 comments
Assignees
Labels
bug Something isn't working

Comments

@bartekpacia
Copy link
Contributor

bartekpacia commented Oct 25, 2022

Repository concerned: https://github.com/leancodepl/patrol

I think I have set the correct favicon in our docs.json and reference the correct asset, but the favicon is still the default one. I'd be very grateful if somebody of you, dear creators, took a look at our repository and told me what I'm doing wrong 😅

@bartekpacia
Copy link
Contributor Author

Huh, social preview is not working as well :(

@cabljac
Copy link
Contributor

cabljac commented Oct 25, 2022

Hey, thanks for raising this!

I took a look at your config file, and favicon seems to be set correctly, which means this is probably a bug. For the social preview image, try changing it to a URL of the image and see if that fixes things, i.e

https://github.com/leancodepl/patrol/blob/master/docs/assets/preview.jpg?raw=true

As a workaround until a fix comes in, changing favicon to logo might work for you.

I'll look into this ASAP though

@cabljac cabljac self-assigned this Oct 25, 2022
@cabljac cabljac added the bug Something isn't working label Oct 25, 2022
@cabljac
Copy link
Contributor

cabljac commented Oct 25, 2022

Also note that twitter and other sites will cache social previews, so any changes may take a while to appear.

@bartekpacia
Copy link
Contributor Author

Thanks a lot for your help, will try out the workarounds tomorrow.

bartekpacia added a commit to leancodepl/patrol that referenced this issue Oct 29, 2022
Ehesp added a commit that referenced this issue Dec 10, 2022
@Ehesp
Copy link
Member

Ehesp commented Jan 10, 2023

@bartekpacia Might need to re-added, but favicon should work now in your config.

@bartekpacia
Copy link
Contributor Author

Yeah, it works. Done in leancodepl/patrol#744.

It'd be also great if we were able to specify faviconDark :)

@Ehesp
Copy link
Member

Ehesp commented Jan 10, 2023

Ah that needs JavaScript to do that from what I remember - I'll take a look.

@Ehesp
Copy link
Member

Ehesp commented Jan 10, 2023

Oh, or something like

<link rel="icon" href="/favicon.ico?dark"  media="(prefers-color-scheme:dark)">
<link rel="icon" href="/favicon.ico?light" media="(prefers-color-scheme:light)">

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants