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

[Gecko Bug 1770729] Fix IntersectionObserver overflow: clip on zero-sized elements. #34166

Merged
merged 1 commit into from May 23, 2022

Conversation

moz-wptsync-bot
Copy link
Collaborator

I realized my fix in bug 1769512 (using IsEmpty()) wasn't quite correct
in this case, because IntersectionObserver preserves edge-inclusive
intersections.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1770729
gecko-commit: 6fb2ab75b12bbd9708b2476aa2c5e520b776ff02
gecko-reviewers: jwatt

I realized my fix in bug 1769512 (using IsEmpty()) wasn't quite correct
in this case, because IntersectionObserver preserves edge-inclusive
intersections.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1770729
gecko-commit: 6fb2ab75b12bbd9708b2476aa2c5e520b776ff02
gecko-reviewers: jwatt
Copy link
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 408d62a into master May 23, 2022
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1770729 branch May 23, 2022 21:45
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.

None yet

3 participants