ros/automaker
1
0
Fork 0
mirror of https://github.com/AutoMaker-Org/automaker.git synced 2026-02-04 09:13:08 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: streamline context menu behavior for project removal dialog

Browse Source 
- Ensure the context menu closes consistently after the confirmation dialog, regardless of user action.
- Reset confirmation state upon dialog closure to prevent unintended interactions.
This commit is contained in:
Shirone
2026-01-15 22:20:30 +01:00
parent 585ae32c32
commit ec042de69c
1 changed files with 7 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
apps/ui/src/components/layout/project-switcher/components/project-context-menu.tsx
View File

@@ -193,18 +193,16 @@ export function ProjectContextMenu({
const handleDialogClose = useCallback( const handleDialogClose = useCallback(
(isOpen: boolean) => { (isOpen: boolean) => {
setShowRemoveDialog(isOpen); setShowRemoveDialog(isOpen);
// Only close the context menu after dialog closes if removal was confirmed // Close the context menu when dialog closes (whether confirmed or cancelled)
// This prevents race condition where onClose unmounts the component // This prevents the context menu from reappearing after dialog interaction
// before ConfirmDialog finishes its internal state cleanup if (!isOpen) {
if (!isOpen && removeConfirmed) { // Reset confirmation state
setRemoveConfirmed(false);
// Always close the context menu when dialog closes
onClose(); onClose();
} }
// Reset confirmation state when dialog closes (for potential reopen)
if (!isOpen) {
setRemoveConfirmed(false);
}
}, },
[onClose, removeConfirmed] [onClose]
); );
return ( return (