Remove unneeded CSS prefixes and length unit symbols

[valtlai]

• Remove -webkit- prefixes from transition and flexbox-related properties
• Remove -webkit- prefix inside unprefixed @keyframes
• Add missing unprefixed transforms
• Remove px unit from 0px values

(+ Remove whitespace from end of lines)

[petetnt]

LGTM

[redmunds]

@valtlait

The original email notification I received said this:

(+ Remove whitespace from end of lines and empty lines)

But message in PR now says this:

(+ Remove whitespace from end of lines)

It looks like it was edited to remove "and empty lines". But, I still see lines with only whitespace that were edited to remove whitespace in this PR. Please backout those changes and rebase into a single commit. Thanks.

In general:

Yes, please remove whitespace from the end of non-empty lines.

Please do not remove whitespace from empty lines. This is not part of the Brackets coding conventions. It's optional based on developer tastes. It doesn't do any good to remove it because the next dev might put it back. Worst of all, the change will show up as a merge conflict for the next poor soul that edits a line near that change, so it causes unnecessary pain.

Exception: whitespace on empty lines should be at correct indentation level, so it's ok to correct that. But no need to scan files for this -- just correct it in code blocks that you are editing for some other reason.

[petetnt]

FWIW There was discussion about this on #11998 too and @peterflynn suggested (for the code) "no-trailing-spaces": [2, { "skipBlankLines": true }] which I think makes some sense. The thing is that when no-trailing-spaces is enabled you can't really use an automatic whitespace sanitizer, without doing tons of silly diffs all over the documents. But then again CodeMirror likes to put whitespace allover the place so there's you have either to endure the manual removal of trailing spaces, let them be or just remove them all, all of which have their downsides.

[valtlai]

Reverted whitespace changes for empty lines.

[redmunds]

@valtlait Thank you!

[marcelgerber]

Please note that the Linux build of Brackets Shell is sadly still based on CEF 1547 (~ Chrome 29), so you still need these prefixes (and probably others, too) according to caniuse.com

[valtlai]

• Restored -webkit- prefix to transform
• Removed -webkit- prefix inside unprefixed @keyframes
• Added missing unprefixed transforms

[MiguelCastillo]

Curious. Why did you removed the units from all 0s?

[valtlai]

@MiguelCastillo There’s a practice to leave off the unit from zero length values. That makes the code look slightly cleaner.

[ficristo]

@valtlai could you update the PR?
It LGTM but I prefer if someone else could do the last check.

[valtlai]

@ficristo Updated.

[ficristo]

@valtlai the merge didn't end well, can you check?

[valtlai]

Replaced with #12648.