From a17c1b7604f12704f9dbfc48cc867ba81740a454 Mon Sep 17 00:00:00 2001 From: le king fu Date: Tue, 20 Oct 2026 19:43:18 -0400 Subject: [PATCH] docs: placeListbox test header and CLAUDE.md module count (#429, #384) Review 25921, points 4 and 6: the header of placeListbox.test.ts still listed the re-read after a render among what needs a DOM, though CategoryCombobox.placement.test.ts pins it; only the paint and React's skipped render remain. CLAUDE.md announced 10 command modules and listed twelve: 12 modules, 10 of them with commands, 45 commands. Co-Authored-By: Claude Opus 5.5 --- CLAUDE.md | 2 +- src/components/shared/placeListbox.test.ts | 10 +++++----- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/CLAUDE.md b/CLAUDE.md index 5307267..c0a870f 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -65,7 +65,7 @@ src/ src-tauri/ ├── src/ -│ ├── commands/ # 10 modules, 45 commandes Tauri (enregistrées dans generate_handler!) +│ ├── commands/ # 12 modules, dont 10 avec des commandes : 45 commandes Tauri (enregistrées dans generate_handler!) │ │ ├── fs_commands.rs # Système de fichiers (7 commandes) │ │ ├── export_import_commands.rs # Export/import chiffré (5 commandes) │ │ ├── profile_commands.rs # Gestion des profils (8 commandes) diff --git a/src/components/shared/placeListbox.test.ts b/src/components/shared/placeListbox.test.ts index dc0127d..8a1dacd 100644 --- a/src/components/shared/placeListbox.test.ts +++ b/src/components/shared/placeListbox.test.ts @@ -1,11 +1,11 @@ // placeListbox — where the open list of a category combobox goes (#429). // The portal itself needs a DOM (manual check); its placement rule, the // listeners that keep it on its input and the state update that keeps an -// unchanged position do not, and are pinned here. What needs a DOM: that the -// combobox reads the position again after a render that moved its input -// (`useLayoutEffect` without dependencies, `getBoundingClientRect`), and that -// this ends, React skipping the render when `keepPlacement` hands back the -// same object; the harness refuses `useLayoutEffect` and `useId`. +// unchanged position do not, and are pinned here. The combobox reading its +// position again after a render that moved its input is pinned under the +// harness by CategoryCombobox.placement.test.ts. What still needs a DOM: the +// paint, and React skipping the render when `keepPlacement` hands back the +// same object, which is what ends that reading. import { describe, it, expect, vi } from "vitest"; import { LISTBOX_GAP,