Skip to content

fix: docs favicon emoji, logo cropping, dark mode invert#33

Merged
RostiMelk merged 2 commits intomainfrom
fix/docs-polish
Mar 9, 2026
Merged

fix: docs favicon emoji, logo cropping, dark mode invert#33
RostiMelk merged 2 commits intomainfrom
fix/docs-polish

Conversation

@RostiMelk
Copy link
Member

  • Favicon now uses Twemoji 🍜 SVG paths (renders consistently everywhere)
  • Logo SVGs use the same Twemoji emoji with wider viewBox (fixes text cropping)
  • Demo logos get dark:invert for dark mode support

- Replace hand-drawn SVG icons with Twemoji 🍜 ramen emoji paths
- Widen logo viewBox to prevent text cropping
- Add dark:invert to demo logos for dark mode support
@vercel
Copy link

vercel bot commented Mar 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
logo-soup Ready Ready Preview, Comment Mar 9, 2026 3:26pm

Request Review

@github-actions
Copy link

github-actions bot commented Mar 9, 2026

Benchmark Comparison: main vs fix/docs-polish

Threshold: 5%+ change, >100us absolute delta, and statistically significant (p<0.05).

Benchmark main fix/docs-polish Change p-value Verdict
content detection (1 logo) 35.48 us 38.70 us +9.1% 0.188 unchanged
render pass (20 logos) 1.17 us 1.07 us -8.6% 0.003 ** unchanged
mount 20 logos (no detection) 3.29 us 3.55 us +7.9% 0.357 unchanged
mount 20 logos (defaults) 770.27 us 787.34 us +2.2% 0.115 unchanged

No regressions detected.

Feature cost breakdown

How expensive are individual features? Measured on this run's HEAD commit.

Feature On Off Cost Sig
densityAware: true vs false 33.02 us 33.42 us ~same 0.851
alignBy: visual-center-y vs bounds 1.53 us 46 ns 33× slower <0.001 ***
cropToContent: true vs false 2.21 ms 59 ns 2.21 ms <0.001 ***
layout update: full mount vs cached 848.14 us 2.04 us 416× slower <0.001 ***

Full benchmark output in the CI job logs.

@RostiMelk RostiMelk merged commit 1aa4a85 into main Mar 9, 2026
5 checks passed
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