Skip to content

refactor(ui): name dialogs use QueryaDialogActions for their footer - #1462

Merged
ZhuchkaTriplesix merged 1 commit into
devfrom
issue/1441-dialog-footers
Oct 11, 2026
Merged

ZhuchkaTriplesix merged 1 commit into
devfrom
issue/1441-dialog-footers

Conversation

@ZhuchkaTriplesix

Copy link
Copy Markdown
Member

Part of #1441.

Five small dialogs built the same Cancel + primary footer by hand (own container and divider, 12 px between the buttons): ERD group, note and view name, New folder, and Create database (Mongo). They now use QueryaDialogActions, wrapped in the ListenableBuilder that enables the primary action while the name is valid. Spacing becomes the standard 8 px, the divider comes from the kit and Esc cancels.

Mechanical change, found by a script that only rewrites the exact pattern (ghost Cancel, a gap, a ListenableBuilder with a primary button); the other footers (forms with Test connection, destructive confirmations) are for follow-up PRs under #1441.

ERD group / note / view name, New folder and Create database build a
Cancel + primary footer by hand (12 px gap, own divider). They now use
QueryaDialogActions: order, spacing 8, lg, one divider, Esc cancels.

Refs #1441
@github-actions github-actions Bot added bug Something isn't working frontend Theme parser epic label: frontend ui User interface components and widgets ux User experience, keyboard shortcuts and interactions P3 Low priority / Polish & Enhancements labels Oct 11, 2026
@ZhuchkaTriplesix
ZhuchkaTriplesix merged commit 4503e32 into dev Oct 11, 2026
15 checks passed
@ZhuchkaTriplesix
ZhuchkaTriplesix deleted the issue/1441-dialog-footers branch October 11, 2026 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working frontend Theme parser epic label: frontend P3 Low priority / Polish & Enhancements ui User interface components and widgets ux User experience, keyboard shortcuts and interactions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant