Improve modal dialog UX and sizing

- Remove redundant success alert after presentation deletion
- Keep only confirmation modal before deletion
- Fix modal size classes CSS specificity
- Set reasonable min/max widths for modal sizes (small: 320-420px)
- Ensure responsive behavior on mobile devices

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Michael Mainguy 2025-08-21 15:24:18 -05:00
parent 018bfc8d9e
commit c0c080c6ac
2 changed files with 16 additions and 10 deletions

View File

@ -61,11 +61,6 @@ export const PresentationsList: React.FC = () => {
setDeleting(presentationId);
await deletePresentation(presentationId);
setPresentations(prev => prev.filter(p => p.metadata.id !== presentationId));
setAlertDialog({
isOpen: true,
message: 'Presentation deleted successfully',
type: 'success'
});
} catch (err) {
loggers.presentation.error('Failed to delete presentation', err instanceof Error ? err : new Error(String(err)));
setAlertDialog({

View File

@ -39,19 +39,22 @@
}
}
/* Modal Sizes */
.modal-small {
/* Modal Sizes - Applied to modal-content with size modifier */
.modal-content.modal-small {
width: 100%;
max-width: 400px;
min-width: 320px;
max-width: 420px;
}
.modal-medium {
.modal-content.modal-medium {
width: 100%;
min-width: 400px;
max-width: 600px;
}
.modal-large {
.modal-content.modal-large {
width: 100%;
min-width: 500px;
max-width: 800px;
}
@ -110,6 +113,14 @@
max-height: 95vh;
}
/* Override min-widths on mobile for better fit */
.modal-content.modal-small,
.modal-content.modal-medium,
.modal-content.modal-large {
min-width: auto;
width: 100%;
}
.modal-header {
padding: 1rem 1rem 0 1rem;
}