Bug 122878

Summary: css rendering bug : fixed-position-element and 'left'
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: ahmad.saleem792, bdakin, dino, hyatt, karlcow, sgill26, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: BlinkMergeCandidate, BrowserCompat, InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Test case
none
GitHub Patch Screenshot none

Ryosuke Niwa
Reported 2013-10-15 20:28:50 PDT
Consider merging https://chromium.googlesource.com/chromium/blink/+/771dc814b28496295ded1ac67346d774f4ebd608 Fixed position element was rendered as normal flow irrespective of it's relative-position-parents. if fixed position element does not have left and/or right position then it should render as normal flow with respect to it's parent. So fixed position element is shifted with the value of relative position of it's parent.
Attachments
Test case (3.88 KB, text/html)
2022-08-30 14:51 PDT, Simon Fraser (smfr) 		no flags
Details
GitHub Patch Screenshot (1.53 MB, image/png)
2023-07-06 12:56 PDT, Ahmad Saleem 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2022-08-30 14:47:13 PDT
<rdar://problem/99352377>
Simon Fraser (smfr)
Comment 2 2022-08-30 14:51:11 PDT
Created attachment 462017 [details] Test case
Ahmad Saleem
Comment 3 2023-07-06 12:56:50 PDT
Created attachment 466955 [details] GitHub Patch Screenshot I manage to make this Blink patch work but I am sure, I did error because we start regressing some other tests as well. If someone can help or point me to right direction, it would be great. It although also progresses few failing test cases (Case 11 & Case 12) as well.
Ahmad Saleem
Comment 4 2023-09-17 16:44:12 PDT
It has follow-up bug as well: https://src.chromium.org/viewvc/blink?view=revision&revision=185274 So I think we need to merge both. Hence, my patch might have not worked.
Ahmad Saleem
Comment 5 2024-05-23 18:28:44 PDT
(In reply to Ahmad Saleem from comment #4) > It has follow-up bug as well: > https://src.chromium.org/viewvc/blink?view=revision&revision=185274 > > So I think we need to merge both. Hence, my patch might have not worked. GitHub link - https://github.com/chromium/chromium/commit/137fc51c2db76f02cf6b02febc9a537b8af0a0c6 and https://github.com/chromium/chromium/commit/6e7527943cc3f1e7306e42435708d7ac54cb3171
Note You need to log in before you can comment on or make changes to this bug.