Skip to content

Queue confirm dialog#213

Merged
slhr merged 5 commits intomasterfrom
queue-confirm-dialog
Nov 10, 2025
Merged

Queue confirm dialog#213
slhr merged 5 commits intomasterfrom
queue-confirm-dialog

Conversation

@slhr
Copy link
Copy Markdown
Collaborator

@slhr slhr commented Nov 7, 2025 •
edited
Loading

  • Added the ability to call confirmDialog in a queue;
  • Removed useEffect (which removed the "p-overflow-hidden" class from body) from Dialog component;
  • Added the ability to pass all props in the Dialog and confirmDialog components.

@slhr slhr requested review from elenik72 and nnsense91 November 7, 2025 04:59
@slhr slhr self-assigned this Nov 7, 2025
elenik72
elenik72 reviewed Nov 7, 2025
View reviewed changes
Comment thread src/components/Dialog/index.js Outdated
@slhr slhr merged commit dfeb9ce into master Nov 10, 2025
4 checks passed
@slhr slhr deleted the queue-confirm-dialog branch November 10, 2025 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@elenik72 elenik72 elenik72 approved these changes

@nnsense91 nnsense91 nnsense91 approved these changes

Assignees

@slhr slhr

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@slhr @elenik72 @nnsense91