Summary: | ::before/::after elements not filling their grid cell when container has display: contents | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Michał Gołębiowski-Owczarek <m.goleb+bugzilla> | ||||||||||
Component: | CSS | Assignee: | Nobody <webkit-unassigned> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | 709922234, allan.jensen, commit-queue, esprehn+autocc, ews-watchlist, glenn, gyuyoung.kim, koivisto, macpherson, menard, rniwa, saam, simon.fraser, webkit-bug-importer, zalan | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | Safari Technology Preview | ||||||||||||
Hardware: | Mac | ||||||||||||
OS: | iOS 12 | ||||||||||||
Bug Depends on: | |||||||||||||
Bug Blocks: | 148695 | ||||||||||||
Attachments: |
|
Description
Michał Gołębiowski-Owczarek
2019-01-18 00:35:17 PST
Adding `display: block` to the ::before/::after styles fixes the issue. The original test case works fine in Chrome 71 & Firefox 64. Edge doesn't support `display: contents` so it won't work there. This broke a cool Slack logo on Grid demo by Ana Tudor she posted on Twitter: https://codepen.io/thebabydino/pen/rogQZb https://twitter.com/anatudor/status/1085994803173699588 Created attachment 359460 [details]
Expected output (Chrome, Firefox)
Created attachment 359461 [details]
Buggy output (Safari)
Created attachment 382299 [details]
patch
Comment on attachment 382299 [details] patch Clearing flags on attachment: 382299 Committed r251780: <https://trac.webkit.org/changeset/251780> All reviewed patches have been landed. Closing bug. |