Summary: | Unexplained 1px horizontal space between display:table-cell elements | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Rebert <webkit> | ||||||||
Component: | Tables | Assignee: | zalan <zalan> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | simon.fraser, zalan | ||||||||
Priority: | P2 | ||||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
URL: | http://jsbin.com/carilu/1/edit?html,css,output | ||||||||||
Bug Depends on: | 149366 | ||||||||||
Bug Blocks: | 159753 | ||||||||||
Attachments: |
|
Description
Chris Rebert
2015-10-08 14:42:43 PDT
Dangit, I meant http://jsbin.com/carilu/1/edit?html,css,output Created attachment 262713 [details]
Copy of JS Bin example
Created attachment 262714 [details]
Chrome screenshot demonstrating expected behavior
We suspect this may be related to why Bootstrap input groups with a button on the left have a double-wide horizontal border between the <button> and the <input> on Safari. (The borders are instead supposed to overlap since we set a negative margin.) Example: https://bootstrapdocs.com/v3.3.5/docs/components/#input-groups-buttons This is due to the integral space distribution between table cells vs. device pixel aligned drawing inside the cells (so we end up growing the cell more than needed and the pixelsnapped painting leaves gaps between the cells). If the input width is set to an integral value, the gap disappears. This will probably fixed by bug 149366. It works on Chrome since they don't do (device)pixelsnapped painting while FF has non-integral cell space distribution. This is fixed by https://trac.webkit.org/changeset/191623 (non-integral table space distribution) Do I need to worry about the other dependency, bug 150383, then? (In reply to comment #8) > Do I need to worry about the other dependency, bug 150383, then? Apparently not! (In reply to comment #9) > (In reply to comment #8) > > Do I need to worry about the other dependency, bug 150383, then? > Apparently not! I verified the linked test case on both retina and non-retina displays. We (r191623) render it the same as FF. (Chrome seems to be broken on non-retina though) Wonderful! Thanks for the fix and the double-check. I've removed this bug from Bootstrap's Wall (https://github.com/twbs/bootstrap/pull/18091 ) and will pass along your Chrome findings. (In reply to comment #11) > Wonderful! Thanks for the fix and the double-check. > I've removed this bug from Bootstrap's Wall > (https://github.com/twbs/bootstrap/pull/18091 ) > and will pass along your Chrome findings. Thanks for reporting it! |