Skip to content

[Gecko Bug 1834876] Part 4: Fix the case when changing the display from none.#45976

Merged
moz-wptsync-bot merged 1 commit intomasterfrom
gecko/1834876
Apr 30, 2024
Merged

[Gecko Bug 1834876] Part 4: Fix the case when changing the display from none.#45976
moz-wptsync-bot merged 1 commit intomasterfrom
gecko/1834876

Conversation

@moz-wptsync-bot
Copy link
Copy Markdown
Collaborator

Add one extra branch if we have before-change style but its display
is none, and the new style is not display:none. Also, we add an extra
subtest if we use the container query to change the display property.

Differential Revision: https://phabricator.services.mozilla.com/D208572

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1834876
gecko-commit: 4a70e54db9b5ed46ae8fc610bc4efed17a590c4f
gecko-reviewers: layout-reviewers, firefox-style-system-reviewers, emilio

Add one extra branch if we have before-change style but its display
is none, and the new style is not display:none. Also, we add an extra
subtest if we use the container query to change the display property.

Differential Revision: https://phabricator.services.mozilla.com/D208572

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1834876
gecko-commit: 4a70e54db9b5ed46ae8fc610bc4efed17a590c4f
gecko-reviewers: layout-reviewers, firefox-style-system-reviewers, emilio
Copy link
Copy Markdown
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Firefox project.

@moz-wptsync-bot moz-wptsync-bot merged commit 873ff0c into master Apr 30, 2024
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1834876 branch April 30, 2024 16:48
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.

3 participants