Skip to content

Fix header spacing#108

Merged
kylewlacy merged 3 commits into
mainfrom
fix-header-spacing
Jun 28, 2026
Merged

Fix header spacing#108
kylewlacy merged 3 commits into
mainfrom
fix-header-spacing

Conversation

@kylewlacy

Copy link
Copy Markdown
Member

This is a small PR to fix the divider on the (non-Starlight) site header.

Before:
Top right corner of brioche.dev, showing the icons and theme selector (set to auto). There's a barely-visible vertical line flush with the icons.

(note the divider between the icons and theme selector is flush against the icons, and barely visible)

After:
Same as above, except the dividing line is taller and appropriately spaced between the icons and theme chooser.

(the divider is taller and at the midpoint between the icons and theme selector)

Starlight header for comparison:

Screenshot of top-right corner of Brioche docs page. Very similar to above, although the exact color and element sizes differ slightly.

(with thanks to @jaudiger for spotting this!)

@kylewlacy kylewlacy merged commit 39c084e into main Jun 28, 2026
1 check passed
@kylewlacy kylewlacy deleted the fix-header-spacing branch June 28, 2026 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant