Bug 247869
| Summary: | Incorrect Static Position of Absolute Positioned Elements inside Rel-Positioned Containers | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | bfulgham, simon.fraser, webkit-bug-importer, zalan |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ahmad Saleem
Hi Team,
While going through Blink Commits, I came across two related commits to each other:
Blink Commit (1) - https://src.chromium.org/viewvc/blink?view=revision&revision=180488
Blink Commit (2) - https://chromium.googlesource.com/chromium/src.git/+/116b2f293d0c16017e63e45cc9a97ee5bfa9b161
We fail the test from Commit (1) - Test Failing - https://jsfiddle.net/bfnhpy0c/ and I think fixing it would be good for overall. We have code here:
https://github.com/WebKit/WebKit/blob/27705761a29a1f7a1625264d72425f9dffe49f58/Source/WebCore/rendering/RenderInline.cpp#L888
NOTE - We already have this Blink Commit (2) partially applied, we just need to update comments and get rid of the L892-894 as it was done in Blink Commit (1).
Since we are failing test cases, I will do PR to see if we have any regression or progress and will update, if I am not able to land it or not. So someone else can do it.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
EWS
Committed 256722@main (106eb5245b22): <https://commits.webkit.org/256722@main>
Reviewed commits have been landed. Closing PR #6451 and removing active labels.
Radar WebKit Bug Importer
<rdar://problem/102405478>