Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[css-view-transitions-1] When skipping transitions, call the DOM-update callback from a task #8827

Merged
merged 2 commits into from
May 11, 2023

Conversation

noamr
Copy link
Collaborator

@noamr noamr commented May 11, 2023

Amend assert to allow calling the callback from a "done" state

The "finished" promise is bound to the update callback promise, so it doesn't need to change (the update callback is async anyway).

See resolution.

Closes #7904

[css-spec-shortname-1] Brief description which should also include the #issuenum-or-URL and/or link to relevant CSSWG minutes.

Copy the above line into the Title and replace with the relevant details. Fill in any additional details here. See https://github.com/w3c/csswg-drafts/blob/master/CONTRIBUTING.md for more info.

Amend assert to allow calling the callback from a "done" state

The "finished" promise is bound to the update callback promise, so
it doesn't need to change (the update callback is async anyway).

See [resolution](w3c#7904 (comment)).

Closes w3c#7904
@noamr noamr requested a review from khushalsagar May 11, 2023 07:29
Copy link
Member

@khushalsagar khushalsagar left a comment

Choose a reason for hiding this comment

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

Can you add a line to the change list at the end of the spec, describing this edit.

@noamr
Copy link
Collaborator Author

noamr commented May 11, 2023

Can you add a line to the change list at the end of the spec, describing this edit.

Done

Copy link
Member

@khushalsagar khushalsagar left a comment

Choose a reason for hiding this comment

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

LGTM again.

@noamr
Copy link
Collaborator Author

noamr commented May 11, 2023

LGTM again.

Thanks! Do you have merge permissions or do we need to tag someone else?

@khushalsagar khushalsagar merged commit a55950d into w3c:main May 11, 2023
@noamr noamr deleted the 7904 branch May 11, 2023 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[css-view-transitions-1] Should the DOMChangeCallback be posted as a task from skip transition steps?
2 participants