Add favicon set using the official OpenVox hexagon sticker - #18
Open
miharp wants to merge 1 commit into
Open
Conversation
Every page was 404ing on /favicon.ico with a blank tab icon. Uses the OpenVox hexagon sticker (voxpupuli/logos) so the tab icon matches docs.openvoxproject.org, but vendored locally instead of hotlinking GitHub raw like docs does. The upstream sticker SVG is 244KB with an embedded raster, so it's rebuilt as a 1.9KB vector: hexagon polygon in the official orange (#eb8521) plus the official openvox_bildmarke fox path (identical to the site header logo path). - favicon.svg: the vector sticker - favicon-32.png: fallback for Safari, which lacks SVG favicon support - apple-touch-icon.png: 180x180 on a solid light background with rounded corners, since iOS touch icons should not be transparent PNGs rasterized from the same SVG with macOS qlmanage. Addresses the favicon finding on OpenVoxProject#14. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Signed-off-by: Michael Harp <mike@mikeharp.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Every page 404s on
/favicon.icoand shows a blank tab icon (flagged on #14).The icons
Approach
This uses the official OpenVox hexagon sticker (voxpupuli/logos), so the tab icon matches docs.openvoxproject.org — but vendored locally rather than hotlinking GitHub raw the way docs currently does. The upstream sticker SVG is 244KB with an embedded raster image, so it's rebuilt here as a 1.9KB vector: a hexagon polygon in the official orange (
#eb8521, sampled from the sticker) plus the officialopenvox_bildmarkefox path — which is byte-identical to the path already used for the site header logo.favicon.svg— primary icon for modern browsersfavicon-32.png— fallback for Safari, which doesn't support SVG faviconsapple-touch-icon.png— 180×180 on a solid light background (#f5faf9) with rounded corners, since iOS touch icons shouldn't be transparentVerified locally: all three serve with 200, the homepage console is clean (the
favicon.ico404 is gone), and the full build passes html-proofer.Aside for a possible docs follow-up: the same 1.9KB vendored SVG would be a drop-in replacement for the 244KB hotlinked favicon on docs.openvoxproject.org.
Addresses the favicon finding on #14.
Written with AI assistance (Claude Code); reviewed and verified by me.