Summary: | Webkit doesn't calculate padding-top/-bottom in % correctly when parent has padding | ||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | ik | ||||||||||||||||||||||
Component: | Layout and Rendering | Assignee: | Rob Buis <rbuis> | ||||||||||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||||||||||
Severity: | Normal | CC: | bfulgham, changseok, clopez, esprehn+autocc, ews-watchlist, glenn, ik, koivisto, kondapallykalyan, listtad, pdr, rbuis, rego, simon.fraser, smoley, thorton, webkit-bug-importer, youennf, zalan | ||||||||||||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||||||||||||
Version: | Safari 14 | ||||||||||||||||||||||||
Hardware: | All | ||||||||||||||||||||||||
OS: | All | ||||||||||||||||||||||||
Attachments: |
|
Description
ik
2021-02-01 05:43:19 PST
Created attachment 418863 [details]
Result of test-case in Safari 14.0.2 on Big Sur
Created attachment 418864 [details]
Result of test-case in Chrome 88
Created attachment 418865 [details]
Test case v2
Updated test case where the first box's css was incorrect.
It looks like `height: 100%` *is* correctly calculated but only `padding-bottom: 100%` is not.
Thanks for filing, I'm seeing this issue reproduce on Safari 13.1.3 as well as TOT. Created attachment 427894 [details]
Patch
Created attachment 427915 [details]
Patch
This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess Created attachment 427923 [details]
Patch
Created attachment 427975 [details]
Patch
Created attachment 427976 [details]
Patch
Comment on attachment 427976 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=427976&action=review r=me > Source/WebCore/ChangeLog:9 > + https://www.w3.org/TR/CSS2/visudet.html#containing-block-details (4.2). Nit: Maybe you can link the new CSS specs here, as this is quite an old reference: https://drafts.csswg.org/css-position-3/#absolute-positioning-containing-block Created attachment 430000 [details]
Patch
Committed r278194 (238237@main): <https://commits.webkit.org/238237@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 430000 [details]. *** Bug 133049 has been marked as a duplicate of this bug. *** |