When using url() in CSS files, you don't need to add the optional ' or ". More info.

Most of core doesn't use them, some does, this patch removes them all since they aren't needed. it is also best practice not to use them, most top CSS people don't. This was a habit that was carried over as including them lead to some browser bugs. Those no longer exist, but the practice still holds.

Simple patch, should be good.

CommentFileSizeAuthor
css_22.patch7.3 KBm3avrck

Comments

jjeff’s picture

+1

I know that I've seen some browser inconsistencies with this. We should try to be uniform throughout Drupal.

Here's the W3C ref: http://www.w3.org/TR/REC-CSS1#url

jjeff’s picture

Status: Needs review » Reviewed & tested by the community
quicksketch’s picture

Nice. +1

dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)