Skip to content

Add test for dialog attribute changed steps#50393

Closed
lukewarlow wants to merge 1 commit intomasterfrom
dialog-closedby-remove-open-attr
Closed

Add test for dialog attribute changed steps#50393
lukewarlow wants to merge 1 commit intomasterfrom
dialog-closedby-remove-open-attr

Conversation

@lukewarlow
Copy link
Copy Markdown
Member

This test ensures that when an open attribute is removed the element is removed from the document's open dialogs list and the close watcher is destroyed.

This test ensures that when an open attribute is removed the element is removed from the document's open dialogs list and the close watcher is destroyed.
Copy link
Copy Markdown
Member

@keithamus keithamus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lessavit

@keithamus keithamus enabled auto-merge (squash) January 30, 2025 20:11
@lukewarlow lukewarlow disabled auto-merge January 30, 2025 20:13
@lukewarlow
Copy link
Copy Markdown
Member Author

(Going to leave unmerged until the HTML PR is merged)

@lukewarlow lukewarlow closed this Oct 1, 2025
@keithamus keithamus deleted the dialog-closedby-remove-open-attr branch October 1, 2025 16:05
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.

4 participants