Closed
Bug 1677642
Opened 4 years ago
Closed 4 years ago
Investigate whether the scrollable overflow area should use UnionRectEdges
Categories
(Core :: Layout, task)
Core
Layout
Tracking
()
RESOLVED
FIXED
85 Branch
Tracking | Status | |
---|---|---|
firefox85 | --- | fixed |
People
(Reporter: TYLin, Assigned: TYLin)
References
Details
Attachments
(2 files)
We have comments in nsOverflowAreas::UnionWith
and nsOverflowAreas::UnionAllWith
about using UnionRectEdges
for scrollable overflow rect.
This bug is to investigate whether these comments make sense or not. If yes, we should update the code, or the comments should be deleted.
Comment 1•4 years ago
|
||
I tend to think the comments should be deleted. Chrome agrees that empty boxes doesn't affect the scrollable overflow, so we should probably add a WPT to assert that.
Assignee | ||
Comment 2•4 years ago
|
||
Updated•4 years ago
|
Assignee: nobody → aethanyc
Status: NEW → ASSIGNED
Pushed by aethanyc@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/ccf878a677f0
Add a wpt testing an empty box doesn't contribute to parent's scrollable overflow area. r=emilio
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/26583 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot
Comment 6•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox85:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•