Bug 122878 - css rendering bug : fixed-position-element and 'left'
Summary: css rendering bug : fixed-position-element and 'left'
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BlinkMergeCandidate, BrowserCompat, InRadar
Depends on:
Blocks:
 
Reported: 2013-10-15 20:28 PDT by Ryosuke Niwa
Modified: 2023-09-17 16:44 PDT (History)
8 users (show)

See Also:


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

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 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.
Comment 1 Radar WebKit Bug Importer 2022-08-30 14:47:13 PDT
<rdar://problem/99352377>
Comment 2 Simon Fraser (smfr) 2022-08-30 14:51:11 PDT
Created attachment 462017 [details]
Test case
Comment 3 Ahmad Saleem 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.
Comment 4 Ahmad Saleem 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.