Bug 154019

Summary: Background of an absolutely positioned inline element inside text-indented parent is positioned statically
Product: WebKit Reporter: bbrinza <bbrinza>
Component: Layout and RenderingAssignee: alan <zalan>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, esprehn+autocc, glenn, hyatt, kondapallykalyan, simon.fraser, zalan
Priority: P2 Keywords: FromImplementor
Version: Safari 9   
Hardware: All   
OS: All   
Attachments:
Description Flags
Test reduction
none
Patch none

bbrinza@microsoft.com
Reported 2016-02-08 16:31:53 PST
(Copied from https://code.google.com/p/chromium/issues/detail?id=568851) Discovered on a live site (http://www.eset.com.br/download/free-trial/nod32-antivirus) - Edge and Firefox match on both live site and reduction. Reduction: https://jsfiddle.net/x4dohcak/ The text in the span is positioned with text-indent, but the background and border box looks to be statically positioned. Note: I don't have Safari available, but colleague of mine confirmed the repro on Safari.
Attachments
Test reduction (349 bytes, text/html)
2016-02-23 12:53 PST, alan 		no flags
Details
Patch (18.92 KB, patch)
2016-02-23 19:24 PST, alan 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
alan
Comment 1 2016-02-16 10:02:53 PST
It looks like that "list-style: none" throws us off of positioning the box properly. When the list-style is set to default (or square, etc), box is moved off to the left.
alan
Comment 2 2016-02-23 12:53:15 PST
Created attachment 272044 [details] Test reduction This is a more generic case where we end up ignoring the text-indent for the absolute positioned child (with static positions) when the line does not have any other content.
alan
Comment 3 2016-02-23 12:53:30 PST
Patch is coming up.
alan
Comment 4 2016-02-23 19:24:13 PST
Created attachment 272076 [details] Patch
alan
Comment 5 2016-02-23 20:22:40 PST
related bug 154628
WebKit Commit Bot
Comment 6 2016-02-24 09:13:28 PST
Comment on attachment 272076 [details] Patch Clearing flags on attachment: 272076 Committed r197030: <http://trac.webkit.org/changeset/197030>
WebKit Commit Bot
Comment 7 2016-02-24 09:13:32 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.