Repository navigation
Web 0.14.23: verschachtelte Verzweigungen kompakter, Spalten füllen die Verzweigung - #250
Merged
Merged
Conversation
…iagramme scrollen Zweiarmige Verzweigung macht beide Spalten nur gleich breit, wenn keine Spalte eine weitere Verzweigung/Fallauswahl enthält; sonst verdoppelte sich die Breite je Ebene (Lektion 10 Schritt 2, Getränkeautomat). Lektionsvorschau verkleinert breite Diagramme höchstens auf 60 % und scrollt dann seitlich. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Ugz28U6UTNEkKmTMxnAW3k
Spalten einer Verzweigung/Fallauswahl wurden beim Strecken (z. B. im Schleifenrumpf) nicht mitskaliert, weil frame.width schon vorher auf die neue Breite gesetzt war. Das if in Lektion 14 Schritt 3 sah dadurch wie eine Schleife aus. Alte Breite jetzt aus den Spalten; neuer Test prüft alle Lektionsdiagramme. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Ugz28U6UTNEkKmTMxnAW3k
code4teaching
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by Holger · project thread
Probleme
frame.widthwar schon auf die neue Breite gesetzt, Skalierung = 1). Das bestand schon vor diesem PR, 37 Lektionsdiagramme betroffen.Änderung
web/src/core/layout.ts: gleiche Spaltenbreite nur noch, wenn keine Spalte selbst eine Verzweigung/Fallauswahl enthält. Einfache if/else bleiben unverändert.stretchMultiwayNodeToWidthnimmt die alte Breite aus den Spalten. Neuer TestlayoutColumns.test.tsprüft alle Lektionsdiagramme.app.ts,styles.css).website/appneu gebaut.Getränkeautomat: 3072 → 1453 px (de). Alle 521 Tests grün.
🤖 Generated with Claude Code
https://claude.ai/code/session_01Ugz28U6UTNEkKmTMxnAW3k