Rundgang-Hinweis raus, Topbar benannt, DatenGraf-Brücke aufgeräumt (v68) - #65
Merged
Merged
Conversation
…aeumt (v68) Nachdem v67 den Rundgang dauerhaft in den Hero gestellt hat, stand der wegklickbare Hinweis als zweite Einladung darunter - und die Topbar bot "Loslegen" an, das dasselbe tat wie "Sofort loslegen" zwei Zeilen tiefer. Der Hinweis ist weg, der Topbar-Knopf heisst jetzt "Daten-Kompass" und benennt sein Ziel. Mit dem Hinweis entfiel sein einziger Leser: `datenlotse_tour` wurde danach nur noch geschrieben und von niemandem mehr gelesen. Der Schluessel, tourSeen(), markTourSeen() und refreshTourHint() sind deshalb mit entfernt worden, statt als toter Zustand liegen zu bleiben. DatenGraf-Bruecke: das DatenGraf-Zeichen (fa-circle-nodes, dasselbe wie in Topbar und Seitenleiste) ersetzt das CSV-Zeichen, der lila Rand links ist weg. Das echte DatenGraf-Logo war nicht zu beschaffen - die Netzwerkrichtlinie sperrt datengraf.nozilla.net und der Repo-Zugriff reicht nicht weiter. Beim Herausschneiden der .tour-hint-Regeln nahm der Schnitt die daneben- stehende Abdunkel-Regel body.tour-on::before mit; der Test aus v66 machte es rot, weil er die Farbe misst statt die Regel zu suchen. Behoben. 1 neuer Test (379 gesamt), eine Mutation geprueft. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011QxDJcLg9h8zdL9ZpTTty7
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
2 participants
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.
Aufräumen nach v67 — zwei Einladungen für dieselbe Sache, zwei Knöpfe mit demselben Ziel.
Der Rundgang-Hinweis ist entfallen
Seit v67 bietet der Hero den Rundgang dauerhaft an;
#tour-hintstand als zweite Einladung direkt darunter.Mit ihm entfiel sein einziger Leser.
datenlotse_tourwurde danach nur noch geschrieben — niemand hat ihn mehr gelesen. Der Schlüssel,tourSeen(),markTourSeen()undrefreshTourHint()sind deshalb mit entfernt worden, statt als toter Zustand liegen zu bleiben. Dieselbe Lehre wie beiREQUIRED_FIELDSin v65, diesmal gleich mit aufgeräumt.Ein Test hält fest, dass nach einem vollständigen Rundgang nichts mehr in den Speicher geschrieben wird.
Topbar: „Loslegen" → „Daten-Kompass"
Der Knopf tat dasselbe wie „Sofort loslegen" zwei Zeilen tiefer im Hero und hieß fast genauso. Er benennt jetzt sein Ziel.
DatenGraf-Brücke
fa-circle-nodes) ersetzt das CSV-Zeichen — dasselbe Zeichen, das die App schon in Topbar und Seitenleiste für DatenGraf benutzt.Ein eigener Fehler, den der Test gefunden hat
Beim Herausschneiden der
.tour-hint-Regeln nahm mein Schnitt die direkt danebenstehende Abdunkel-Regelbody.tour-on::beforemit — der Rundgang wäre ohne Abdunkeln erschienen.Der Test aus v66 machte es rot, weil er die Farbe misst statt die Regel zu suchen. Ein Test, der nur nach dem Selektor gegriffen hätte, wäre hier genauso rot geworden — aber einer, der die Klasse am
bodygeprüft hätte, wäre grün geblieben.Tests
1 neuer Test, 379 gesamt, alle grün. Eine Mutation geprüft: den Schlüssel wieder schreiben, ohne dass ihn jemand liest → macht genau den neuen Test rot.
node tools/generate-wissen.js --check→ 9 Seiten aktuell.npm run iconsmitgelaufen — 88 → 87 Icons, weilfa-playmit dem Hinweis entfallen ist.🤖 Generated with Claude Code
https://claude.ai/code/session_011QxDJcLg9h8zdL9ZpTTty7
Generated by Claude Code