Skip to content

perf(playground): narrow font range for loading states#4

Merged
wiyco merged 1 commit into
mainfrom
perf/playground-font-range-code-ligatures
Jun 4, 2026
Merged

perf(playground): narrow font range for loading states#4
wiyco merged 1 commit into
mainfrom
perf/playground-font-range-code-ligatures

Conversation

@wiyco

@wiyco wiyco commented Jun 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • Replace loading-state ellipses with ASCII periods and remove U+2026 from the Geist unicode ranges.
  • Disable font ligatures in playground code surfaces so JSX spread syntax renders as {...props}.
  • Document the typography rationale for keeping Geist in the stack while narrowing runtime glyph usage.

Tests

  • pnpm vitest run apps/playground/src/design-token-sync.test.ts apps/playground/src/core/svg-playground/ui/SvgPlayground.test.tsx apps/playground/src/core/svg-playground/ui/SvgPlayground/panels/CodeSurface.test.tsx
  • pnpm --filter ./apps/playground build
  • pnpm run test:browser:playground -- src/playgrounds/svgo-plugin-hoist-stroke-width/App.browser.test.tsx

Replace loading-state ellipsis glyphs with ASCII dots so Geist no longer needs U+2026 in its runtime unicode ranges.

Disable code-surface ligatures to keep JSX spread syntax visually faithful while preserving the copied source.
@wiyco wiyco force-pushed the perf/playground-font-range-code-ligatures branch from 16cf81c to e0a4ffd Compare June 4, 2026 21:23
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown

Code Metrics Report

Download coverage report: lcov-report

main (a7fb939) #4 (f7e73e0) +/-
Coverage 100.0% 100.0% 0.0%
Code to Test Ratio 1:1.8 1:1.8 +0.0
Test Execution Time 1s 1s 0s
Details
  |                     | main (a7fb939) | #4 (f7e73e0) | +/-  |
  |---------------------|----------------|--------------|------|
  | Coverage            |         100.0% |       100.0% | 0.0% |
  |   Files             |             63 |           63 |    0 |
  |   Lines             |           1323 |         1323 |    0 |
  |   Covered           |           1323 |         1323 |    0 |
+ | Code to Test Ratio  |          1:1.8 |        1:1.8 | +0.0 |
  |   Code              |           4261 |         4265 |   +4 |
+ |   Test              |           7939 |         7948 |   +9 |
  | Test Execution Time |             1s |           1s |   0s |

Code coverage of files in pull request scope (100.0% → 100.0%)

Files Coverage +/- Status
apps/playground/src/core/svg-playground/ui/SvgPlayground/panels/CodeSurface.editor.tsx 100.0% 0.0% modified
apps/playground/src/core/svg-playground/ui/SvgPlayground/panels/OptimizedPanel.tsx 100.0% 0.0% modified
apps/playground/src/core/svg-playground/ui/SvgPlayground/panels/PreviewPanel.tsx 100.0% 0.0% modified
apps/playground/src/core/svg-playground/ui/SvgPlayground/panels/ReactSourcePanel.tsx 100.0% 0.0% modified
apps/playground/src/core/svg-playground/ui/SvgPlaygroundPage.tsx 100.0% 0.0% modified

Reported by octocov

@wiyco wiyco merged commit 441a7a4 into main Jun 4, 2026
4 checks passed
@wiyco wiyco deleted the perf/playground-font-range-code-ligatures branch June 4, 2026 21:24
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