We no longer need to prefix border-radius or box-shadow.
http://caniuse.com/border-radius
http://caniuse.com/css-boxshadow

Files: 
CommentFileSizeAuthor
#20 remove-prefixed-css-2005520-20.patch8.92 KBechoz
PASSED: [[SimpleTest]]: [MySQL] 57,667 pass(es).
[ View ]
#12 remove-prefixed-css-2005520-12.patch8.37 KBechoz
PASSED: [[SimpleTest]]: [MySQL] 56,749 pass(es).
[ View ]
#10 remove-prefixed-css-2005520-10.patch8.37 KBechoz
FAILED: [[SimpleTest]]: [MySQL] 56,131 pass(es), 2 fail(s), and 0 exception(s).
[ View ]
#8 remove-prefixed-css-2005520-8.patch5.36 KBechoz
PASSED: [[SimpleTest]]: [MySQL] 55,431 pass(es).
[ View ]
#3 remove-prefixed-css-2005520-3.patch6.12 KBechoz
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch remove-prefixed-css-2005520-3.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#1 remove-prefixed-css-2005520.patch6.12 KBechoz
FAILED: [[SimpleTest]]: [MySQL] 56,670 pass(es), 1 fail(s), and 0 exception(s).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new6.12 KB
FAILED: [[SimpleTest]]: [MySQL] 56,670 pass(es), 1 fail(s), and 0 exception(s).
[ View ]

This patch removes remaining prefixed border-radius and prefixed box-shadow found in Seven and views-admin.theme.css

Issue tags:+d8mux-css-cleanup

tagging

StatusFileSize
new6.12 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch remove-prefixed-css-2005520-3.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Re-submitting since stuck at "postponed" after 7hrs, where later patches were queued normally.

Issue tags:-d8mux-css-cleanup

#3: remove-prefixed-css-2005520-3.patch queued for re-testing.

Status:Needs review» Needs work

The last submitted patch, remove-prefixed-css-2005520-3.patch, failed testing.

Status:Needs work» Needs review

#3: remove-prefixed-css-2005520-3.patch queued for re-testing.

Status:Needs review» Needs work
Issue tags:+d8mux-css-cleanup

The last submitted patch, remove-prefixed-css-2005520-3.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new5.36 KB
PASSED: [[SimpleTest]]: [MySQL] 55,431 pass(es).
[ View ]

Re-roll

Status:Needs review» Needs work

I found a few more instances of prefixed properties. I don't think we can touch the joyride.css, you could submit an issue in the Github repo. I have no idea about simpletest but I'm sure we can loose it there as well.

/core/modules/tour/css/joyride-2.0.3.css:
/core/modules/simpletest/files/css_test_files/css_input_without_import.css.unoptimized.css
/core/modules/simpletest/files/css_test_files/css_input_without_import.css.optimized.css
/core/modules/simpletest/files/css_test_files/css_input_without_import.css
/core/themes/seven/ie.css // This one is quite funny

Status:Needs work» Needs review
StatusFileSize
new8.37 KB
FAILED: [[SimpleTest]]: [MySQL] 56,131 pass(es), 2 fail(s), and 0 exception(s).
[ View ]

Thanks Lewis. This patch adds the 3 files you noted from simpletest. I also deleted the MS filter lines as IE9 supports box shadow, MS filter no longer needed.

That ie.css (shudder), I spotted that recently, and it can go. Needs another issue, although unrelated to this one.

Status:Needs review» Needs work

The last submitted patch, remove-prefixed-css-2005520-10.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new8.37 KB
PASSED: [[SimpleTest]]: [MySQL] 56,749 pass(es).
[ View ]

Ooops, patching that optimized file got me.

Status:Needs review» Needs work
Issue tags:-d8mux-css-cleanup

The last submitted patch, remove-prefixed-css-2005520-12.patch, failed testing.

Status:Needs work» Needs review
Issue tags:+d8mux-css-cleanup

#12: remove-prefixed-css-2005520-12.patch queued for re-testing.

is the optimized file, css_input_with_import.css.optimized.css, a generated file so perhaps we're not supposed to patch it?

I spoke to Berdir about this on IRC, it sounds like we should be patching the optimised file. The CSS files are tests to check to see if the unoptimised CSS comes back correctly once it's optimised. If we change the unoptimised we should change the optimised to match.

Ok, thanks for checking on that. It is patched in #12 and it came back green.

Status:Needs review» Reviewed & tested by the community

Fantastic, may I suggest we add a section on prefixed properties to our coding standards?

Status:Reviewed & tested by the community» Needs work

We still have some usages of -webkit-box-shadow and -moz-box-shadow in ckeditor.admin.css and seven's ie.css (why they're in ie.css is kinda beyond me :D )

Status:Needs work» Needs review
StatusFileSize
new8.92 KB
PASSED: [[SimpleTest]]: [MySQL] 57,667 pass(es).
[ View ]

Gee, I don't know what process I began searching for these properties with, but it sure needed some work, thanks @LewisNyman and @alexpott. This patch adds ckeditor.admin.css. I believe we could follow up with prefixed gradients http://lea.verou.me/2013/04/can-we-get-rid-of-gradient-prefixes/ which might need discussion where our line is.

I'm creating another issue to delete seven's ie.css.

Status:Needs review» Needs work
Issue tags:-d8mux-css-cleanup

The last submitted patch, remove-prefixed-css-2005520-20.patch, failed testing.

Status:Needs work» Needs review

#20: remove-prefixed-css-2005520-20.patch queued for re-testing.

Status:Needs review» Needs work

The last submitted patch, remove-prefixed-css-2005520-20.patch, failed testing.

Status:Needs work» Needs review
Issue tags:+d8mux-css-cleanup

#20: remove-prefixed-css-2005520-20.patch queued for re-testing.

Status:Needs review» Reviewed & tested by the community

Thanks Alex, I thought the ckeditor CSS was part of the ckeditor plugin.

Did a global grep for -webkit- prefixes. The only places that -webkit-border-radius or -webkit-box-shadow show up are in third party libs.

Status:Reviewed & tested by the community» Fixed

Committed 6d8aacf and pushed to 8.x. Thanks!

Status:Fixed» Active

Why were lines with those properties in core/modules/simpletest/files/css_test_files/* removed? They were there to guarantee that CSS aggregation wouldn't break them, and it's entirely possible that sites which need to support older browsers still want to use these.

@Wim Leers, I assume you mean the "filter" and "-ms-filter". I noted removing them in #10. They are obsolete proprietary MS code, although I don't know the nuances of simpletest. Do we need to revert and re-patch leaving them in? If that's the case, it seems like we don't want to remove anything from simpletest, as it's more likely someone would still use the much more modern -webkit-border-radius.