Drop-shadow filter does not work when 'overflow' property is set to 'hidden'
Reported by
insensi...@gmail.com,
Aug 18 2016
|
|||||
Issue descriptionChrome Version : 52.0.2743.116 (Official Build) m (32-bit) URLs (if applicable) : https://jsfiddle.net/6baweyro/8/ Other browsers tested: Firefox/Microsoft edge Add OK or FAIL, along with the version, after other browsers where you have tested this issue: Safari: Firefox: OK IE: OK What steps will reproduce the problem? (1) Open https://jsfiddle.net/6baweyro/8/ What is the expected result? All three boxes on the page should have shadow, even if element 'overflow' property is set to 'hidden' What happens instead? Middle box having 'overflow' property set to 'hidden' does not have shadow I tested the same page in latest Firefox/Microsoft Edge - all three boxes have shadow in these browsers. I am not sure what says W3C filter specification about this.
,
Aug 19 2016
I think this falls on the paint side of things. If not please sent it back my way.
,
Aug 20 2016
Thanks for taking the time to file this, I do think it's a bug. box-shadow uses filters internally so I'm not sure why we're applying the clip early in the filter cases but not the box-shadow one. Possibly due to how ComputedStyle::hasBoxDecorationBackground() works? Testcase: http://jsbin.com/rotuhi
,
Sep 8 2016
I just updated jsfiddle example, removed -webkit prefix since Chrome 53 is released https://jsfiddle.net/6baweyro/9/ allows unprefixed filters. This issue has priority 2 "want (for current milestone)" and labeled "M-53" Chrome 53 is released and I am not familar with Chrome release cycle, so I wonder is there an approximate release I can expect this issue to be fixed? Many thanks.
,
Aug 2 2017
This issue seems to be already fixed. I am unable to reproduce it in Chrome 60.
,
Aug 2
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Aug 2
Fixed no doubt by the refactoring we have done. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by rnimmagadda@chromium.org
, Aug 19 2016Components: Blink>Layout
Labels: M-53 OS-Linux OS-Mac OS-Windows
Status: Untriaged (was: Unconfirmed)
2.2 MB
2.2 MB Download