fix: follow-ups of the lot-v018 stack reviews #482

Merged
maximus merged 25 commits from review-followups-lot-v018 into main 2026-10-21 00:31:46 +00:00
2 changed files with 6 additions and 6 deletions
Showing only changes of commit a17c1b7604 - Show all commits

View file

@ -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)

View file

@ -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,