-
Notifications
You must be signed in to change notification settings - Fork 4
Better Favicon #12
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
Just a little bit down the forum thread, I found this line: <link rel="shortcut icon" href="', WT_THEME_DIR, 'images/favicon.ico" type="image/x-icon" /> |
Looks quite nice! But I was productive in the meantime and went through your code to find the color gradient and font used in the header. So I created this logo here. It's not perfect, but I like it because it also uses the big W from the original logo. What do you think? I wasn't sure about the corner radius, so I didn't set one. |
Yes I can sure do. I did this in MS Publisher, but I think a proper .svg will be nicer to work with. Unfortunately I don't have much spare time at the moment. Maybe I'll get to it next week. Tasks:
I could use your help with that. Basically, I need to know the font size related to |
Sorry, I'm not sure what you mean. |
ok I figured it out. I actually created a fork and would be happy to make a PR. Unfortunately, I don't have the knowledge to also take care of making the right changes in the theme files. My suggestion is that you create a different develop branch to which I then will make a PR so that we can work on this together. Sounds good? This is what I have done:
This is what we/you need to figure out:
So, what do you think? Did your plans go another direction or are you happy with this? |
A note on the earlier version of the logo: I had the orientation of the background gradient wrong. I hope that it is correct now... |
Hi, sorry you did all that work, but I'm thinking it's overkill to add a GitHub workflow for this, especially since there's no reason for the icons to be regenerated every single time. Replacing the icons is more challenging than I thought, since the
And add them to the |
Yeah I understand, it is a bit overkill. My intention was to make the generation of the About replacing the icons: I have no idea how all of this works. All content inside |
Hi @jchue I am about to make a PR that contains the png-files you specified and some short instructions on how to set them up. It also contains a workflow that generates the png files from the provided svg for you. But this only runs, if the svg file has been changed. Hope thats ok. Have you tried this method from the forum thread I mentioned? <link rel="shortcut icon" href="', WT_THEME_DIR, 'images/favicon.ico" type="image/x-icon" /> |
First, I want to thank you for the updates you released. The Theme looks so much better now!
I wondered, if the favicon could be replaced and did some google research. Appearently, it is really easy to change the favicon if you do the change inside a theme. It's literally just a line of code in the theme header (see this forum thread):
I imagine one could set it to a path inside the theme folder.
I also know of a tool that could come in handy: https://realfavicongenerator.net/.
What I unfortunately don't have, is a ready to use favicon. I just think that the original webtrees favicon doesn't quite fit the clean and esthetic looks of your theme, especially if webtrees is used as a PWA on mobile. Maybe you or someone else (or future me) has the inspiration to make a better favicon.
The text was updated successfully, but these errors were encountered: