#43220 closed defect (bug) (fixed)
CSS Coding standards: Use double quotes for values
| Reported by: | afercia | Owned by: | afercia |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.1 |
| Component: | Administration | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
Description
As per the CSS Coding standards:
Values:
Use double quotes rather than single quotes, and only when needed, such as when a font name has a space.
Specifically, for CSS generated content (especially icons) there are good reasons to always use double quotes, see [34011] / #33786. Even when the icons don't have to be swapped for RTL, using double quotes it's always a good practice.
I'd also propose to update the Coding Standards page to explicitly mention values for the content property.
Attachments (2)
Change History (7)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
43220.2.diff fixes also some indentation and double spaces in the touched files.