This is an archive of the discontinued LLVM Phabricator instance.

[analyzer][Rewrite] Fix boxes and shadows in HTML rewrites in Firefox.
ClosedPublic

Authored by NoQ on Aug 8 2016, 9:34 AM.

Details

Summary

Use the official CSS3 properties border-radius and box-shadow (not only -webkit-specific properties).

Fixes analyzer's diagnostic pieces in HTML diagnostics mode in Firefox and other non-webkit browsers.

Before:


After:

Not sure, do we need more reviewers? - it seems that HTMLRewrite is mostly used in the analyzer. Also, are there html/css gurus around? Cause the best thing i can do is trust the "a lot of green boxes" feeling at http://caniuse.com/#feat=border-radius and http://caniuse.com/#feat=css-boxshadow :)

Diff Detail

Repository
rL LLVM

Event Timeline

NoQ updated this revision to Diff 67182.Aug 8 2016, 9:34 AM
NoQ retitled this revision from to [analyzer][Rewrite] Fix boxes and shadows in HTML rewrites in Firefox..
NoQ updated this object.
NoQ added reviewers: dcoughlin, zaks.anna.
NoQ added subscribers: xazax.hun, a.sidorin, cfe-commits.
a.sidorin added inline comments.Aug 8 2016, 9:38 AM
lib/Rewrite/HTMLRewrite.cpp
304 ↗(On Diff #67182)

Should we remove '-webkit'-prefixed options as well? AFAIR, non-prefixed options should be supported by Webkit. Could you check with Chrome/Safari?

NoQ added inline comments.Aug 9 2016, 1:41 AM
lib/Rewrite/HTMLRewrite.cpp
304 ↗(On Diff #67182)

All modern browsers should support the unprefixed property and work well without the prefixed property, but i'm not sure if there ever existed older browsers that used to support only the prefixed property. So i think i'm playing safe.

Also, i read that they recommend putting prefixed properties before unprefixed, because prefixed properties should override unprefixed properties whenever supported; i'd fix that.

zaks.anna accepted this revision.Sep 12 2016, 10:53 AM
zaks.anna edited edge metadata.
This revision is now accepted and ready to land.Sep 12 2016, 10:53 AM
This revision was automatically updated to reflect the committed changes.