Aus #214 („Befund:700 wird geladen, aber vom Theme nirgends benutzt. Nicht angefasst.“).
Befund
src/ui/fonts.ts:5 lädt source-sans-3-latin-700-normal.woff2 (15 596 Bytes) zusammen mit 400, 500 und 600.
Der Editor wartet beim Start auf alle Schnitte (loadEditorFonts, fonts.ts:20), also auch auf diesen.
Als Schriftgewicht kommen im Code nur 400, 500 und 600 vor (src/ui/Theme.ts:118, :145, :163, src/editor/graph/AnnotationView.ts:59). bold oder 700 setzt niemand. Button.ts:177 fragt zwar >= 600 ab, das deckt aber schon 600 ab.
Zu bauen (Vorschlag)
Schnitt 700 aus FACES entfernen, die Datei aus src/ui/fonts/ löschen.
tests/ui/fonts.test.ts prüft heute, dass jedes Gewicht des Themes geladen ist. Dazu kommt die Umkehrung: Jedes geladene Gewicht kommt in src/ als fontWeight vor. Der Test liest src/ für die Zeichenprüfung schon heute. Ein unbenutzter Schnitt fällt dann künftig von selbst auf.
Offen
Keine Rückfrage.
Aus #214 („**Befund:** `700` wird geladen, aber vom Theme nirgends benutzt. Nicht angefasst.“).
## Befund
- `src/ui/fonts.ts:5` lädt `source-sans-3-latin-700-normal.woff2` (15 596 Bytes) zusammen mit 400, 500 und 600.
- Der Editor wartet beim Start auf alle Schnitte (`loadEditorFonts`, `fonts.ts:20`), also auch auf diesen.
- Als Schriftgewicht kommen im Code nur `400`, `500` und `600` vor (`src/ui/Theme.ts:118`, `:145`, `:163`, `src/editor/graph/AnnotationView.ts:59`). `bold` oder `700` setzt niemand. `Button.ts:177` fragt zwar `>= 600` ab, das deckt aber schon 600 ab.
## Zu bauen (Vorschlag)
- Schnitt 700 aus `FACES` entfernen, die Datei aus `src/ui/fonts/` löschen.
- `tests/ui/fonts.test.ts` prüft heute, dass jedes Gewicht des Themes geladen ist. Dazu kommt die Umkehrung: Jedes geladene Gewicht kommt in `src/` als `fontWeight` vor. Der Test liest `src/` für die Zeichenprüfung schon heute. Ein unbenutzter Schnitt fällt dann künftig von selbst auf.
## Offen
Keine Rückfrage.
meleki
added the new label 2026-09-29 18:26:30 +02:00
Done in f2282a8, wiki 0b54e14 (Architecture, Development).
What changes
The editor loads Source Sans 3 in 400, 500 and 600, plus Source Code Pro 400. The 700 face is no longer loaded or waited for, and source-sans-3-latin-700-normal.woff2 (15 596 bytes) is gone.
Nothing draws above 600. Besides the four fontWeight settings listed above I searched for bold, font-weight and HTML headings in src/, web/*.html and the player, with no hits. bodyBold is 500, accent is 600.
Under the hood
tests/ui/fonts.test.ts now checks both ways:
as before, every weight set in src/ has a loaded face
new: every loaded Sans face is set somewhere in src/
new: every .woff2 in the font folder is in FACES, so a forgotten file that the glyph test would still read shows up as well
Reading the weights out of src/ moved into the helper usedWeights(), since two tests need it.
Tests
gate (melektaus-atx) 197 files, 2856 tests, CHECK EXIT 0 (2855 + 1)
e2e 1-14 ALL PASSED - 492 checks, loaded faces ["400","500","600"]
bundle no 700 file in dist/assets
e2e 14 alone fails its two tab width checks, a known dependency: it needs the documents 7-13 open. Its font check was green there too.
Counter-checks work: loading 700 again with its file fails "loads no face the editor never sets". Putting only the file back, without an entry in FACES, fails "... and no other" and names the file.
The e2e total stays at 1603, no new e2e check. CI is switched off for now, so there is no CI run for f2282a8.
Done in `f2282a8`, wiki `0b54e14` (*Architecture*, *Development*).
**What changes**
- The editor loads Source Sans 3 in 400, 500 and 600, plus Source Code Pro 400. The 700 face is no longer loaded or waited for, and `source-sans-3-latin-700-normal.woff2` (15 596 bytes) is gone.
- Nothing draws above 600. Besides the four `fontWeight` settings listed above I searched for `bold`, `font-weight` and HTML headings in `src/`, `web/*.html` and the player, with no hits. `bodyBold` is 500, `accent` is 600.
**Under the hood**
- `tests/ui/fonts.test.ts` now checks both ways:
- as before, every weight set in `src/` has a loaded face
- new: every loaded Sans face is set somewhere in `src/`
- new: every `.woff2` in the font folder is in `FACES`, so a forgotten file that the glyph test would still read shows up as well
- Reading the weights out of `src/` moved into the helper `usedWeights()`, since two tests need it.
**Tests**
```
gate (melektaus-atx) 197 files, 2856 tests, CHECK EXIT 0 (2855 + 1)
e2e 1-14 ALL PASSED - 492 checks, loaded faces ["400","500","600"]
bundle no 700 file in dist/assets
```
- e2e 14 alone fails its two tab width checks, a known dependency: it needs the documents 7-13 open. Its font check was green there too.
- Counter-checks work: loading 700 again with its file fails "loads no face the editor never sets". Putting only the file back, without an entry in `FACES`, fails "... and no other" and names the file.
- The e2e total stays at 1603, no new e2e check. CI is switched off for now, so there is no CI run for `f2282a8`.
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Aus #214 („Befund:
700wird geladen, aber vom Theme nirgends benutzt. Nicht angefasst.“).Befund
src/ui/fonts.ts:5lädtsource-sans-3-latin-700-normal.woff2(15 596 Bytes) zusammen mit 400, 500 und 600.loadEditorFonts,fonts.ts:20), also auch auf diesen.400,500und600vor (src/ui/Theme.ts:118,:145,:163,src/editor/graph/AnnotationView.ts:59).boldoder700setzt niemand.Button.ts:177fragt zwar>= 600ab, das deckt aber schon 600 ab.Zu bauen (Vorschlag)
FACESentfernen, die Datei aussrc/ui/fonts/löschen.tests/ui/fonts.test.tsprüft heute, dass jedes Gewicht des Themes geladen ist. Dazu kommt die Umkehrung: Jedes geladene Gewicht kommt insrc/alsfontWeightvor. Der Test liestsrc/für die Zeichenprüfung schon heute. Ein unbenutzter Schnitt fällt dann künftig von selbst auf.Offen
Keine Rückfrage.
Done in
f2282a8, wiki0b54e14(Architecture, Development).What changes
source-sans-3-latin-700-normal.woff2(15 596 bytes) is gone.fontWeightsettings listed above I searched forbold,font-weightand HTML headings insrc/,web/*.htmland the player, with no hits.bodyBoldis 500,accentis 600.Under the hood
tests/ui/fonts.test.tsnow checks both ways:src/has a loaded facesrc/.woff2in the font folder is inFACES, so a forgotten file that the glyph test would still read shows up as wellsrc/moved into the helperusedWeights(), since two tests need it.Tests
FACES, fails "... and no other" and names the file.f2282a8.