Fixed font spec files #511

Merged
johnsturgeon merged 1 commit from fix-font-spec-files-510 into main 2026-09-11 15:24:12 +02:00
Owner
No description provided.
Fixed font spec files
All checks were successful
Claude Code Review / claude-review (pull_request) Successful in 43s
Python FastAPI Jinja Linting / build (pull_request) Successful in 32s
Tests / pytest (pull_request) Successful in 14s
Release on merge / release (pull_request) Successful in 1s
6104559161
Author
Owner

Summary

This PR corrects three malformed @font-face declarations in the Tailwind source (app/styles/v3.src.css) — quoting the src URLs and removing the stray spaces from the Figtree unicode-range tokens (U+0000 -00FFU+0000-00FF) — and checks in the regenerated minified v3.css, along with the routine patch version bump. The changes are correct, self-consistent, and low-risk.

I verified that the committed v3.css matches what the source would minify to (all three fonts now share the identical, valid unicode-range; the previously-broken Figtree range like U0 -0FF is gone), that the referenced font files (figtree-var-latin.woff2, inter-var-latin.woff2, rock-salt-400-latin.woff2) exist under app/static/fonts/, and that the version bump is applied consistently in both pyproject.toml:3 and uv.lock per the versioning convention.

The change looks good — nothing blocking, and no correctness, performance, or security concerns for a static-asset fix like this.

Nits

  • app/styles/v3.src.css:46 still ends without a trailing newline (the diff shows \ No newline at end of file). Not introduced by this PR and cosmetic, but worth normalizing if you touch the file again.
<!-- claude-code-review --> ## Summary This PR corrects three malformed `@font-face` declarations in the Tailwind source (`app/styles/v3.src.css`) — quoting the `src` URLs and removing the stray spaces from the Figtree `unicode-range` tokens (`U+0000 -00FF` → `U+0000-00FF`) — and checks in the regenerated minified `v3.css`, along with the routine patch version bump. The changes are correct, self-consistent, and low-risk. I verified that the committed `v3.css` matches what the source would minify to (all three fonts now share the identical, valid `unicode-range`; the previously-broken Figtree range like `U0 -0FF` is gone), that the referenced font files (`figtree-var-latin.woff2`, `inter-var-latin.woff2`, `rock-salt-400-latin.woff2`) exist under `app/static/fonts/`, and that the version bump is applied consistently in both `pyproject.toml:3` and `uv.lock` per the versioning convention. The change looks good — nothing blocking, and no correctness, performance, or security concerns for a static-asset fix like this. ### Nits - `app/styles/v3.src.css:46` still ends without a trailing newline (the diff shows `\ No newline at end of file`). Not introduced by this PR and cosmetic, but worth normalizing if you touch the file again.
johnsturgeon deleted branch fix-font-spec-files-510 2026-09-11 15:24:12 +02:00
Sign in to join this conversation.
No description provided.