Skip to content

fix(dialog, date-range-picker): sync the dialog on reconnect and commit after igcClosing - #2577

Merged
rkaraivanov merged 1 commit into
masterfrom
rkaraivanov/dialog-close-follow-ups
Nov 5, 2026
Merged

rkaraivanov merged 1 commit into
masterfrom
rkaraivanov/dialog-close-follow-ups

Conversation

@rkaraivanov

Copy link
Copy Markdown
Member

Description

  • Dialog: a dialog that moves, or is removed and appended again, keeps its state. A close also reaches the native dialog while the host is out of the document, and an append makes an open dialog modal again.
  • Date range picker: Done, Cancel, Escape and an outside click emit igcClosing first and commit or restore the range only when the close goes on. Done now emits igcChange after igcClosing.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)

Checklist

  • My code follows the project's coding standards
  • I have tested my changes locally
  • I have updated documentation if needed
  • I have added a CHANGELOG.md entry under [Unrelease]

…it after igcClosing

- Dialog: a dialog that moves, or is removed and appended again, keeps its
  state. A close also reaches the native dialog while the host is out of the
  document, and an append makes an open dialog modal again.
- Date range picker: Done, Cancel, Escape and an outside click emit igcClosing
  first and commit or restore the range only when the close goes on. Done now
  emits igcChange after igcClosing.
@rkaraivanov
rkaraivanov merged commit 2fd080f into master Nov 5, 2026
7 checks passed
@rkaraivanov
rkaraivanov deleted the rkaraivanov/dialog-close-follow-ups branch November 5, 2026 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant