From e3e015bca402a8d6cd82bb4cff03d9bae8808e97 Mon Sep 17 00:00:00 2001 From: Dogfather Date: Thu, 17 Sep 2026 18:59:19 +0200 Subject: [PATCH] Kopfleiste: ein Name, der zweimal vergeben war MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Filipe zu einem Bildschirmfoto: "was soll das jetzt, wieso sieht das in gewissen seiten so scheisse aus?!" Der Teilen-Knopf stand doppelt so hoch wie seine Nachbarn, mit dem Zeichen UEBER dem Wort statt daneben. Es war kein Gestaltungsfehler, sondern ein Name: `.teilen` gab es ZWEIMAL -- gate.css Knopf der Kopfleiste inline-flex, waagerecht bereich.css Layout der Teilen-Seite flex column, 22px, 44rem Auf den drei Seiten, die beide Dateien laden (bereich, bewerben, teilen), gewann die spaeter geladene. Der Knopf bekam das Layout einer ganzen Seite. "Gewisse Seiten" war exakt richtig beobachtet. DERSELBE NAME KOSTETE AUSSERDEM DEN KNOPF: kopf.js baut ihn nur ein, wenn `getElementById('teilen')` nichts findet -- ein Schutz gegen doppelten Einbau. Auf teilen.html steht `
` aber schon im HTML. Dort fehlte der Knopf ganz, ohne Spur, ohne Meldung. REPARIERT ALS REGEL, NICHT ALS EINZELFALL: - Was kopf.js in die Leiste haengt, traegt jetzt `kopf-` davor (kopf-teilen, kopf-sicht-wahl, kopf-sicht-feld, kopf-fremd-band). Eine Seite darf dann heissen wie sie will -- auch "teilen", "chat" oder "suchen", und genau diese Woerter braucht als naechstes eine Seite. - Die Teilen-Seite bekommt eigene Klassennamen (tl-*). NEU: server/pruef-namen.mjs (12 Pruefungen, ohne Server, <1 s). Es ist der ZWEITE Namenskonflikt an einem Tag -- der erste war `.t-schritt` auf der Talente-Seite, gefunden nur durch Zaehlen im Browser. Beide waren gruen in jeder bestehenden Prueflinie. Gemessen, um den Schnitt zu finden, der etwas taugt: 2872 Klasse in zwei geladenen Dateien erwaehnt -> Rauschen 1887 davon mit eigener Blockdefinition -> Rauschen 12 davon mit widersprechendem `display` 6 davon ohne `display: none` -> geprueft Mit Gegenprobe, die aus den Dateien ABGELEITET wird (die erste, fest abgeschriebene Fassung fand nichts, weil die gewaehlte Klasse dort gar kein `display` setzt -- eine Gegenprobe, die selbst danebenliegt, beweist nichts). pruef-kopf-messen: 3 -> 16 Messungen, und drei eigene Fehler behoben, die alle dieselbe Form hatten -- zu frueh hingesehen: 1. Sie mass NUR uebersicht.html. Die laedt bereich.css nicht und war fuer diesen Fehler blind. 2. Sie mass nur 320/390/430px. Unter 720px versteckt gate.css das Wort "Teilen"; der Knopf bleibt klein, auch falsch angeordnet. Filipes Bild war 1366px breit. 3. Sie wartete 300ms. kopf.js haengt Teile erst nach `/api/ich` ein und hat ein Netz bei 1200ms. Zweiter Versuch: warten, bis `#wer` Text hat -- SOFORT wahr, dort steht `…` als Platzhalter. Dritter: auf Stabilitaet warten -- nach 0ms und 500ms gleich gross, also "fertig". Erst die Mindestwartezeit (1200 aus kopf.js plus Luft) davor machte die Messung echt. Bewiesen an der Gegenprobe: mit dem alten Fehler meldet sie jetzt "kopf-teilen ist 77px -- die anderen im Schnitt 43px", vorher blieb sie gruen. Statt einer festen Hoehe wird gefragt, ob EIN Teil aus der Reihe faellt (> 1,6x der Schnitt seiner Nachbarn). Das waechst mit, wenn die Leiste groesser wird, und muss bei keinem neuen Knopf nachgezogen werden. Gruen: namen, kopf-messen, sicht, css-klassen, struktur. Co-Authored-By: Claude Opus 5 --- server/bild-sicht.mjs | 8 +- server/pruef-handy.mjs | 2 +- server/pruef-kopf-messen.mjs | 135 +++++++++++++- server/pruef-namen.mjs | 299 +++++++++++++++++++++++++++++++ server/pruef-sicht.mjs | 20 +-- workspace/app.webmanifest | 8 +- workspace/assets/css/bereich.css | 38 ++-- workspace/assets/js/kopf.js | 14 +- workspace/assets/js/teilen.js | 8 +- workspace/aufgaben.html | 24 +-- workspace/automation.html | 26 +-- workspace/befinden.html | 24 +-- workspace/bereich.html | 36 ++-- workspace/bewerben.html | 30 ++-- workspace/bewerbungen.html | 28 +-- workspace/calls.html | 28 +-- workspace/chat.html | 26 +-- workspace/content.html | 30 ++-- workspace/crew-index.html | 10 +- workspace/crew.webmanifest | 8 +- workspace/dateien.html | 26 +-- workspace/entwicklung.html | 24 +-- workspace/index.html | 8 +- workspace/kalender.html | 28 +-- workspace/leistung.html | 26 +-- workspace/personen.html | 26 +-- workspace/profil.html | 28 +-- workspace/rechte.html | 24 +-- workspace/report.html | 28 +-- workspace/scouting.html | 26 +-- workspace/start.html | 24 +-- workspace/startcheck.html | 28 +-- workspace/steckbrief.html | 28 +-- workspace/talente.html | 26 +-- workspace/team.html | 26 +-- workspace/teamlage.html | 24 +-- workspace/teilen.html | 48 ++--- workspace/treff-moderation.html | 24 +-- workspace/treff-regeln.html | 26 +-- workspace/uebersicht.html | 26 +-- workspace/wissen.html | 26 +-- 41 files changed, 892 insertions(+), 460 deletions(-) create mode 100644 server/pruef-namen.mjs diff --git a/server/bild-sicht.mjs b/server/bild-sicht.mjs index c24ecdcf..875ed389 100644 --- a/server/bild-sicht.mjs +++ b/server/bild-sicht.mjs @@ -57,14 +57,14 @@ await seite.click("button[type=submit]"); await seite.waitForURL("**/start.html", { timeout: 25000 }); await seite.waitForTimeout(1500); -const feld = await seite.$("#sicht-wahl"); +const feld = await seite.$("#kopf-sicht-wahl"); if (feld) { await feld.screenshot({ path: "sicht-eigene.png" }); console.log("sicht-eigene.png geschrieben"); } /* Jetzt auf Lunas Sicht umschalten. */ -await seite.selectOption("#sicht-feld", String(idLuna)).catch(async () => { +await seite.selectOption("#kopf-sicht-feld", String(idLuna)).catch(async () => { /* wahl.js ersetzt das -