Bug 90281
Summary: | Paddings are rounded to whole pixels | ||
---|---|---|---|
Product: | WebKit | Reporter: | Behdad Esfahbod <behdad> |
Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED CONFIGURATION CHANGED | ||
Severity: | Normal | CC: | ahmad.saleem792, eae, rbuis, rjkroege |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Bug Depends on: | 90097 | ||
Bug Blocks: |
Behdad Esfahbod
Test with Chrome that has SUBPIXEL_LAYOUT enabled. While testing for bug 90097, I came across this bug:
According to https://trac.webkit.org/wiki/LayoutUnit paddings are supposed to have subpixel precision, but this sample shows that they are being rounded:
http://jsfiddle.net/behdad/ALd5F/5/
Output:
span[position:static; padding-right:7.5px]: rect.width=7
span[position:fixed; padding-right:7.5px]: rect.width=7
span[position:absolute; padding-right:7.5px]: rect.width=7
span[position:relative; padding-right:7.5px]: rect.width=7
Applying patches in bug 90097 fixes the two middle cases, but no the other two:
span[position:static; padding-right:7.5px]: rect.width=7
span[position:fixed; padding-right:7.5px]: rect.width=7.5
span[position:absolute; padding-right:7.5px]: rect.width=7.5
span[position:relative; padding-right:7.5px]: rect.width=7
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Rob Buis
This works these days.
Ahmad Saleem
*** Safari Technology Preview 160 ***
span[position:static; padding-right:7.5px]: rect.width=7.5
span[position:fixed; padding-right:7.5px]: rect.width=7.5
span[position:absolute; padding-right:7.5px]: rect.width=7.5
span[position:relative; padding-right:7.5px]: rect.width=7.5
*** Chrome Canary 110 ***
span[position:static; padding-right:7.5px]: rect.width=7.5
span[position:fixed; padding-right:7.5px]: rect.width=7.5
span[position:absolute; padding-right:7.5px]: rect.width=7.5
span[position:relative; padding-right:7.5px]: rect.width=7.5
*** Firefox Nightly 110 ***
span[position:static; padding-right:7.5px]: rect.width=7.5
span[position:fixed; padding-right:7.5px]: rect.width=7.5
span[position:absolute; padding-right:7.5px]: rect.width=7.5
span[position:relative; padding-right:7.5px]: rect.width=7.5
_______
Since all browsers are matching each other, I am going to mark this as "RESOLVED CONFIGURATION CHANGED". Thanks!