search.spec: Fehl-Klick entfernt — Reload IST der Weg zurück in den Lesemodus

Der Spec klickte nach page.reload() den Mode-Toggle „um den Edit-Modus zu verlassen" — nach dem Reload ist die Seite aber schon im Lesemodus (React- State resettet), der Klick wechselte also HINEIN. Das passierte jahrelang folgenlos, weil der Fokus auf dem Toggle-Button blieb und „/" die Suche öffnete. Seit dem Editor-Auto-Fokus (PR #140) landet der Fokus im Editor und „/" wird dort zu Text — je nach Ausgang des Rennens gegen den rAF- verzögerten Fokus mal grün (Läufe 381/385), mal rot (383/387/389/390). Fix: den Klick streichen; „/" läuft im Lesemodus als globaler Shortcut. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0155v2aT8AG1kZDQEZiCLBWC
#137 Feinjustierung: Checkbox im NodeView 3px höher (Text relativ 3px tiefer)
2026-07-19 21:31:53 +02:00 · 2026-07-19 21:31:53 +02:00 · 2026-07-19 17:54:02 +02:00
1 changed files with 11 additions and 10 deletions
--- a/apps/web/src/styles/base.css
+++ b/apps/web/src/styles/base.css
@ -1623,12 +1623,15 @@ ul[data-type='task_list'] li {
 }
 /* The checkbox is the top flex child; nudge it onto the first text line and
-   drop the leading/trailing margins of the item's paragraph so the text meets
+   drop the leading/trailing margins of the item's paragraphs so the text meets
-   the checkbox instead of dropping a line. Covers both DOM shapes: read-mode
+   the checkbox instead of dropping a line. DESCENDANT selectors on purpose
-   `li > input` + `li > p`, and editor `li > label > input` + `li > div > p`.
+   (issue #137, second regression): the DOM depth differs per surface —
-   Uses `:first-of-type`/`:last-of-type` (NOT `:first-child`): in the read-mode
+   public/docToHtml renders `li > input` + `li > p`, but the editor/auth read
-   markup the `<input>` is the first child, so the paragraph is never
+   view (TipTap ReactNodeView) nests the paragraph TWO wrappers deep:
-   `:first-child` and a `:first-child` reset would silently miss (issue #137). */
+   `li > div[data-node-view-content] > div > p`. Child-combinator chains keep
   missing one of the shapes; matching any `p` inside the task item is the
   robust form. `:first-of-type`/`:last-of-type` (NOT `:first-child`) because
   the `<input>`/`<label>` precedes the paragraph in the read-mode markup. */
 ul[data-type='task_list'] li > input[type='checkbox'],
 ul[data-type='task_list'] li > label {
  flex: none;
@ -1642,13 +1645,11 @@ ul[data-type='task_list'] li > label {
  margin-top: calc(0.25em - 3px);
 }
-ul[data-type='task_list'] li > p:first-of-type,
+ul[data-type='task_list'] li p:first-of-type {
 ul[data-type='task_list'] li > div > p:first-of-type {
  margin-top: 0;
 }
-ul[data-type='task_list'] li > p:last-of-type,
+ul[data-type='task_list'] li p:last-of-type {
 ul[data-type='task_list'] li > div > p:last-of-type {
  margin-bottom: 0;
 }
Author	SHA1	Message	Date
Claude Fable 5	60be198e51	search.spec: Fehl-Klick entfernt — Reload IST der Weg zurück in den Lesemodus All checks were successful CI / Build container images (pull_request) Successful in 1m8s Details CD / Promote to Int (push) Successful in 11s Details CI / Lint, typecheck, test (push) Successful in 4m35s Details CI / Build container images (push) Has been skipped Details CI / Lint, typecheck, test (pull_request) Successful in 4m25s Details CI / Auth e2e pack (pull_request) Successful in 6m49s Details CI / Import/export fidelity gate (pull_request) Successful in 54s Details Release / Build release images and notes (push) Successful in 1m12s Details Prod deploy / Deploy the released images to Prod (push) Successful in 15s Details CI / Import/export fidelity gate (push) Successful in 54s Details CD / Build and push images (push) Successful in 16s Details CD / Deploy to Test (push) Successful in 12s Details CD / Smoke tests against Test (push) Successful in 1m15s Details Release / Release-candidate operations QA (push) Successful in 45s Details CI / Auth e2e pack (push) Successful in 6m40s Details Der Spec klickte nach page.reload() den Mode-Toggle „um den Edit-Modus zu verlassen" — nach dem Reload ist die Seite aber schon im Lesemodus (React- State resettet), der Klick wechselte also HINEIN. Das passierte jahrelang folgenlos, weil der Fokus auf dem Toggle-Button blieb und „/" die Suche öffnete. Seit dem Editor-Auto-Fokus (PR #140) landet der Fokus im Editor und „/" wird dort zu Text — je nach Ausgang des Rennens gegen den rAF- verzögerten Fokus mal grün (Läufe 381/385), mal rot (383/387/389/390). Fix: den Klick streichen; „/" läuft im Lesemodus als globaler Shortcut. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0155v2aT8AG1kZDQEZiCLBWC	2026-07-19 21:31:53 +02:00
Claude Fable 5	d07f8bb8e1	#137 Feinjustierung: Checkbox im NodeView 3px höher (Text relativ 3px tiefer) Stefans Feedback nach dem Nachfahren-Selektor-Fix: „noch 3px weiter runter". Die Zeilenmetrik des Editor-/Auth-NodeViews (label-Wrapper) setzt die Checkbox ~3px tiefer als im öffentlichen docToHtml-Markup — daher NUR für den label-Pfad `margin-top: calc(0.25em - 3px)`; die öffentliche Ansicht (bare input, war korrekt) bleibt bei 0.25em. Live per Injektion auf Test vermessen: Versatz −9 → −6px, Checkbox mittig auf der Textzeile (Zoom-Screenshot). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0155v2aT8AG1kZDQEZiCLBWC	2026-07-19 21:31:53 +02:00
Claude Fable 5	3ed3cbb806	#137 Nachfix 2: Task-Item-Absätze per Nachfahren-Selektor treffen (NodeView-Tiefe) Some checks failed CI / Auth e2e pack (pull_request) Successful in 6m45s Details CI / Import/export fidelity gate (pull_request) Successful in 54s Details CD / Build and push images (push) Successful in 17s Details CD / Smoke tests against Test (push) Successful in 1m15s Details CD / Deploy to Test (push) Successful in 13s Details CD / Promote to Int (push) Successful in 12s Details CI / Lint, typecheck, test (push) Successful in 4m32s Details CI / Lint, typecheck, test (pull_request) Successful in 4m39s Details CI / Build container images (pull_request) Successful in 1m13s Details CI / Build container images (push) Has been skipped Details CI / Auth e2e pack (push) Failing after 5m28s Details CI / Import/export fidelity gate (push) Has been skipped Details Stefan sah die Checkbox-Verschiebung weiterhin — in der ANGEMELDETEN Lese-/Bearbeiten-Ansicht. Dort rendert der TipTap-ReactNodeView das <p> ZWEI Wrapper tief (`li > div[data-node-view-content] > div > p`), die bisherige Kind-Kette `li > div > p` griff also nur im flachen docToHtml-Markup der öffentlichen Ansicht. Fix: Nachfahren-Selektoren (`li p:first-of-type` / `li p:last-of-type`) — robust gegen die Wrapper-Tiefe beider Renderpfade. Live am echten NodeView-DOM verifiziert (Injektion auf Test: p-marginTop 16px→0, Checkbox bündig; öffentlicher Pfad unverändert ok). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0155v2aT8AG1kZDQEZiCLBWC	2026-07-19 17:54:02 +02:00