diff --git a/TEST_SCENARIOS.md b/TEST_SCENARIOS.md index d94f77f..b22110d 100644 --- a/TEST_SCENARIOS.md +++ b/TEST_SCENARIOS.md @@ -182,6 +182,9 @@ plus bas. - **ATTENDU** : le dialogue est toujours là, au premier plan de l'application, et répond. L'app n'est cliquable qu'à travers lui — c'est normal, une boîte native modale désactive sa fenêtre propriétaire tant qu'on n'a pas répondu. - **Historique** : le refus « busy » appelait `cancelAutofill()`, dont le chemin Delphi fait un `SetForegroundWindow` vers la fenêtre capturée. Le dialogue passait **derrière** l'autre application, et l'app paraissait figée : bloquée par une modale devenue introuvable. `refocus=0` distingue désormais **refuser** une demande (ne rien déplacer) de **terminer** une interaction utilisateur (rendre le focus à la cible). - **Généralisation** : un refus ne doit jamais déplacer une fenêtre. Tout nouveau garde qui annule une action doit passer par ce chemin. +- **Deuxième cause, côté Delphi** : `Ctrl+Shift+Q` et `Ctrl+Shift+A` appellent `RestoreWindowFromTray` **avant** que JS ne puisse refuser — un garde JS arrive trop tard par construction. Restaurer la fenêtre propriétaire d'une modale native laisse le dialogue en rade derrière tout le reste : l'app semble gelée alors qu'elle est simplement *désactivée* par une boîte devenue introuvable (au point qu'il fallait un utilitaire de capture d'écran pour la faire réapparaître). +- **Garde** : `NativeModalActive` teste `IsWindowEnabled` sur notre fenêtre — Windows la désactive tant qu'une modale native lui appartient. Posé en tête des **quatre** points d'entrée hotkey (quick search hotkey + menu tray, new entry, autofill). Il ne se contente pas de refuser : il **ramène le dialogue au premier plan** via `GetLastActivePopup`, ce qui est précisément ce dont l'utilisateur a besoin. +- **ATTENDU après correctif** : pendant l'« Enregistrer sous », n'importe quel hotkey **fait réapparaître le dialogue** au lieu de l'enterrer. ### B14 · dialogues natifs ouverts par un clic programmatique - **ACTION** : `A` ouverte + **dirty** → (a) Settings → *Import vault* ; (b) télécharger une pièce jointe (fallback web) ; (c) *Upload icon* ; (d) joindre un fichier. diff --git a/delphi-backend/UMainForm.pas b/delphi-backend/UMainForm.pas index 9aa9267..36229c5 100644 --- a/delphi-backend/UMainForm.pas +++ b/delphi-backend/UMainForm.pas @@ -178,6 +178,8 @@ type // stuck raised (see the implementation). // One-shot: hides the window the first time FMX shows it on a --tray // launch, then unhooks itself. + // True when a native modal dialog owns us — see the implementation. + function NativeModalActive: Boolean; procedure RestoreWindowFromTray; procedure WinStateMinimizeStart; procedure WinStateMinimizeEnd; @@ -470,6 +472,7 @@ begin // cmd://autofill/execute (fired after the user picks an entry) sends // the password into it. Then pop the modal in fill mode. if not FServer.Active then Exit; + if NativeModalActive then Exit; FAutofillTargetHWND := ATargetHWND; // Tray-hidden ONLY (Visible=false). A window merely minimised to the // taskbar must NOT count: hide-after would send it to the TRAY on close — @@ -494,6 +497,28 @@ begin [IntToHex(ATargetHWND, 8), ATitle])); end; +// A native modal dialog (Save As, folder picker) DISABLES its owner window for +// as long as it is up. Restoring / focusing that owner then strands the dialog +// behind everything: the app looks frozen, when it is merely disabled by a modal +// the user can no longer find. Every hotkey entry point checks this first — and +// rather than just refusing, it surfaces the dialog, which is what the user +// actually needs. +function TMainForm.NativeModalActive: Boolean; +var + LHwnd, LPopup: HWND; +begin + LHwnd := WindowHandleToPlatform(Self.Handle).Wnd; + Result := (LHwnd <> 0) and (not IsWindowEnabled(LHwnd)); + if not Result then Exit; + LPopup := GetLastActivePopup(LHwnd); + if (LPopup <> 0) and (LPopup <> LHwnd) and IsWindow(LPopup) then + begin + ShowWindow(LPopup, SW_SHOW); + SetForegroundWindow(LPopup); + LogLine('Hotkey ignored — a native dialog is open; brought it to front.'); + end; +end; + procedure TMainForm.BridgeQuickSearchRequest; var LWasHidden: Boolean; @@ -503,6 +528,7 @@ begin // see the modal, then ask JS to open it. JS handles the locked-vault // case (shows the auth screen with master-pw focused instead). if not FServer.Active then Exit; + if NativeModalActive then Exit; // Remember whether the window was TRAY-hidden BEFORE we restore — after // the user picks an entry the JS layer asks us to minimize back so they // can paste into the target app without an extra alt-tab. Iconic-only @@ -522,6 +548,7 @@ var EscapedTitle: string; begin if not FServer.Active then Exit; + if NativeModalActive then Exit; RestoreWindowFromTray; EscapedTitle := StringReplace(AWindowTitle, '\', '\\', [rfReplaceAll]); EscapedTitle := StringReplace(EscapedTitle, '"', '\"', [rfReplaceAll]); @@ -1804,6 +1831,7 @@ var LTitle, LKind: string; begin if not FServer.Active then Exit; + if NativeModalActive then Exit; FAutofillTargetHWND := ATargetHWND; // Tray-hidden only, like the quick-search path: a window merely minimised to // the taskbar must not come back as a tray icon it never was.