🌱 brand: adopt the hivecommons.dev logo - #12
Merged
Conversation
Hive Commons had no mark of its own -- every "Hive Commons logo" slot on
docs.hivecommons.dev was rendering the Hive bee. The bee is Hive's, and it stays
with the hive repo; the umbrella org now has its own identity, so the shared
slots use it.
Replaces public/hive-commons-logo.png (navbar, footers, 404) and src/app/icon.png
(the favicon).
Two things fixed while swapping the asset, both of which predate it:
- The Open Graph and Twitter cards pointed at the square logo while DECLARING it
1200x630. The file was 154x154, so every social unfurl was handed a size that
did not describe the image. Adds a real 1200x630 card and points the metadata
at it. The card needs no lettering because the mark carries the wordmark.
- Five <Image> usages declared width={160} height={40} for a file that has always
been square, so the reserved box was 4:1 against a 1:1 image and shifted on
load. Now 40x40.
The favicon uses a text-free crop of the mark. The full sticker carries a curved
"hivecommons.dev" wordmark that is illegible below about 48px, and a favicon that
resolves to a smudge is worse than one that reads as a honeycomb.
Legacy docs/docs/overrides/favicons/ is untouched -- it is unreferenced mkdocs
leftovers and removing it is a separate cleanup.
Signed-off-by: Andrew Anderson <andy@clubanderson.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
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.
Adopts the hivecommons.dev mark across the docs site. Hive the project keeps the bee; this repo represents Hive Commons, so it takes the new mark.
What changed
public/hive-commons-logo.png— was the bee, now the Hive Commons mark (512x512)public/hive-commons-og.png— new 1200x630 social cardsrc/app/icon.png— favicon, using a text-free crop of the markTwo pre-existing bugs fixed alongside
These were tightly coupled to the asset swap, so they are fixed here rather than left behind:
width: 1200, height: 630while the file it pointed at was actually 154x154. Now points at a real 1200x630 card.<Image>usages declared160x40for a square file, causing layout shift. Corrected to40x40.On the favicon
The sticker is detailed artwork and cannot reduce to a legible 16px favicon — the curved wordmark becomes a smudge. Verified this visually rather than assuming. The favicon therefore uses a text-free crop of the mark itself, which stays readable at small sizes.
Verification
npm run buildpasses;npm run lintexits 0. The two lint errors insrc/__tests__/are pre-existing and come from a missing rule definition in untouched files.Note: the build regenerates
docs/content/**andpublic/config/shared.jsonfrom source repos. Those were deliberately kept out of this commit — only the 9 branding files are staged.