Standardize single and double quotes in CSS url()s

[TobiasBg]

As per ​http://www.w3.org/TR/CSS2/syndata.html#value-def-uri (and other locations), single quotes ' and double quotes " in url() values in CSS like

body { background: url("../img/image,gif");

are optional. Most of core's CSS is already written without such quotes. The patch 22476-remove-quotes-in-url.diff removes the remaining ones, for consistency and to save us some bytes again, even in the minified CSS.

Patch removes optional ' and " in CSS url()s

Our ​CSS coding standards have double quotes listed as the style. Personal preferences aside, these decisions came out of long and arduous group discussions. Also, there's already a ticket for CSS standards cleanup: #20570

Personally, I don't actually prefer the without-quotes version either. I justed wanted some consistency, and as quotes are optional, and as recent commits with new CSS also left them out, additionally to a lot of url()s already being without them, I decided to go that way in the patch.

It might indeed be better to add " everywhere, according to the Coding Standards, and then maybe hope that the CSS minifier will someday automatically strip them to save the bytes in the .min.css version.

Add double quotes for consistency

+1 for MikeHansenMe's patch and coding standards.

updated for new inconsistent styles

Added a new patch because there has been new css added that is also inconsistent. Hope this can go into 3.6

updated patch

I think now would be a good time to get this patch in since all the features are now in beta. Once all the css has a standard it will be easier to monitor new patches for this standard.

Hate to do this to you - we (I?) ended up changing the coding standards after all about a month ago: ​http://make.wordpress.org/core/handbook/coding-standards/css/#values

Use double quotes rather than single quotes, and only when needed, such as when a font name has a space.

I can't quite recall what prompted that change - will link if I have a chance to dig it up.

patch without quotes

So it seems like the preferred way is without quotes unless necessary. I really do not mind which we use, only that it is consistent. However, the coding standard has been double quotes up until recently. Any users who have adopted double will now be non-standard.

Rechecked and refreshed for current trunk(3.6-beta2-24227)