Center the glyph in the tile, and align the README mark - #50
Merged
Conversation
Two defects, found by rendering the README through GitHub's own markdown API with GitHub's CSS and looking: The tile's glyph sat 6 units low — the transform took 18 as a translate value where it was meant as a margin, ignoring the glyph's own y-offset of 8. icon.png is regenerated centered, and icon.svg now lives beside it as the source of truth with the arithmetic written down. The README h1 can only be aligned with the legacy align attribute — GitHub strips CSS — and align="middle" anchors a few pixels below the title's visual center. The header now uses assets/logo-readme.svg, a cut whose glyph sits 6 units above true center to compensate, verified pixel-level against GitHub's renderer. The NuGet icon stays truly centered; only the README cut carries the compensation. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011ysVAfsbKMfGBaAkWvuRTf
The absolute raw URL pointed at main, where logo-readme.svg does not exist until this merges — so branch views rendered nothing. A relative path resolves against whichever ref is being viewed: this branch now, main after the merge, and forks. Nothing is lost on nuget.org, which strips HTML img tags either way; the package pages carry the mark through PackageIcon. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011ysVAfsbKMfGBaAkWvuRTf
Measured on the live branch page: GitHub renders align="middle" via Chrome's -webkit-baseline-middle mapping, anchoring the image box center 12-16px below the title's visual center — the harness CSS had understated the offset. No attribute value lands on the right anchor, so the compensation moves into the canvas: the glyph is pinned to the top of a taller transparent box (width 64, glyph 35px), putting its visible center exactly on the title's center. Verified in a local render using GitHub's markdown API and CSS. The cut also splits into light and dark variants behind GitHub's theme-aware <picture> support — the bare mark replaces the white tile, so dark mode gets the lifted colorway instead of a glaring chip. New file names double as a cache-bust for raw/camo. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011ysVAfsbKMfGBaAkWvuRTf
Uh oh!
There was an error while loading. Please reload this page.
Merged
ipjohnson added a commit
that referenced
this pull request
Aug 29, 2026
Version and changelog only. The icon fix and README header landed in #48-#50; this is the release that puts the corrected icon on nuget.org, since a package icon is per published version. Claude-Session: https://claude.ai/code/session_011ysVAfsbKMfGBaAkWvuRTf Co-authored-by: Claude Fable 5 <noreply@anthropic.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 freeto 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.
Follow-up to #49 — this commit was pushed to that branch just after the merge, so it
rides again here. Two defects, found by rendering the README through GitHub's own
markdown API with GitHub's CSS and looking at the result:
ignoring the glyph's own y-offset.
assets/icon.pngis regenerated truly centered(corrects the NuGet icon too, visible with the next release), and
assets/icon.svgis checked in beside it as the source of truth with the arithmetic written down.
only lever is the legacy
align="middle"attribute, which anchors a few pixels belowthe visual center — the header now uses
assets/logo-readme.svg, a cut whose glyphsits 6 units above true center to compensate, verified pixel-level against GitHub's
renderer. Its off-centering is deliberate; the comment in the file says so.
The header image URL points at
main, so the alignment shows on GitHub once thismerges.
Not merging — that's yours.
🤖 Generated with Claude Code
https://claude.ai/code/session_011ysVAfsbKMfGBaAkWvuRTf