Skip to content

fix(ui): open a recent clip from the status page - #133

Merged
Palawizard merged 4 commits into
devfrom
fix/open-recent-clips
Oct 28, 2026
Merged

Palawizard merged 4 commits into
devfrom
fix/open-recent-clips

Conversation

@Palawizard

Copy link
Copy Markdown
Owner

Branches

  • feature/, fix/, chore/ → dev

Phase / tâche

v1.4.1 — retour de Palawi : on ne peut pas ouvrir un clip depuis « Derniers clips » sur État

Ce qui change

  • Cartes d'État : double-clic ou Entrée ouvre le clip dans le lecteur de la Galerie (clic simple : rien, convention de la galerie confirmée par Palawi). Chaque carte est un bouton focusable (anneau de focus + bordure à l'encre, survol), nom accessible « jeu, date, durée, pistes », aria-keyshortcuts="Enter". La colonne n'a plus son propre arrêt de tabulation (les cartes suffisent pour défiler au clavier).
  • Destination.clip (chemin) dans App.tsx, passé par guard.leave comme les autres destinations ; lib/destination.ts (demande d'ouverture et page de retour).
  • Galerie : ouvre le clip demandé (get_clip), et fermer le lecteur (bouton ou Échap) ramène à la page d'origine. Ouvrir une carte de la galerie revient toujours à la galerie. Mettre à la Corbeille depuis le lecteur reste dans la galerie (l'annulation y est).
  • Fichier disparu : État appelle get_clip avant de partir ; file_missing → message sur État (« Ce fichier n'existe plus… »), la bibliothèque retire le clip et la carte disparaît.
  • Moteur : Library::clip(path) ; shell : commande get_clip { path } (même contrôle known_clip que les autres commandes de clip).
  • La carte du premier lancement (clip de test) reste non cliquable.

Critères d'acceptation

  • Double-clic et Entrée ouvrent le clip ; focus visible, nom lisible
  • Chemin du clip dans la destination, garde de sortie inchangée
  • Fermer le lecteur ramène à État
  • Fichier disparu : message, carte retirée
  • Pas de clic simple

Comment ça a été testé

  • Automatique : cargo clippy --workspace --all-targets -- -D warnings, cargo test (nouveau a_clip_is_found_by_its_path), npm test (3 tests destination.test.ts), npm run build
  • App isolée : pwsh scripts/cdp-run.ps1 -Script scripts/recent-open-check.mjs -Clips 3 → 6 sur 6 (nom et raccourci ; clic simple sans effet ; double-clic → ce clip, retour à État ; Tab → focus visible, Entrée → lecteur, Échap → État ; galerie par la barre → pas de lecteur et retour à la galerie ; fichier supprimé juste avant le double-clic → message sur État, carte retirée).

Docs mises à jour

  • SPEC GAL-03, ARCHITECTURE § IPC (CHANGELOG dans la PR de version)

@Palawizard
Palawizard merged commit b67ef8b into dev Oct 28, 2026
1 check passed
@Palawizard
Palawizard deleted the fix/open-recent-clips branch October 28, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant