Make pikchrHex.svg a clean standalone logo (fix bottom clipping)#9
Merged
Conversation
The logo SVG carried a leftover inline style from the pikchr() HTML wrapper (width:100%; height:auto; margin:10px 0 10px 0). With height:auto and no intrinsic height, some viewers rendered it without a defined aspect ratio, clipping the bottom of the hexagon. Replace it with explicit width/height attributes matching the viewBox and a font-family presentation attribute, so the SVG has a defined intrinsic size and renders identically everywhere. Geometry unchanged (regular hexagon, 0.866 ratio). Regenerate logo.png from it. Co-Authored-By: Claude Opus 4.8 (1M context) <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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
man/figures/pikchrHex.svgcarried a leftover inline style from thepikchr()HTML wrapper:width:100%; height:auto; margin:10px 0 10px 0. Withheight:autoand no intrinsic height, some SVG viewers rendered it without a defined aspect ratio and clipped the bottom of the hexagon.This replaces that style with explicit
width/heightattributes (matching theviewBox) and afont-familypresentation attribute, giving the SVG a defined intrinsic size so it renders identically everywhere. The geometry is unchanged (regular hexagon, 0.866 ratio).logo.pngregenerated from it.🤖 Generated with Claude Code