Opened 13 years ago
Closed 13 years ago
#20587 closed defect (bug) (fixed)
Invalid CSS with 'border-right-color: none'
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.5 | Priority: | normal |
Severity: | minor | Version: | 3.3 |
Component: | UI | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
I came across several instances where we had 'border-right-color: none' in the stylesheets. Because 'none' is not a valid color value, the style appears to be ignored. I am not sure what (or if?) to change these to, but they appear as follows:
./wp-admin/css/colors-classic.dev.css:2356: ./wp-admin/css/colors-classic.dev.css:2390: ./wp-admin/css/colors-classic.dev.css:2421: ./wp-admin/css/colors-fresh.dev.css:1962: ./wp-admin/css/colors-fresh.dev.css:1996: ./wp-admin/css/colors-fresh.dev.css:2027:
Attachments (3)
Change History (15)
#2
@
13 years ago
- Keywords dev-feedback removed
Let's make a note to discuss this and #20586 (and invalid CSS in general) today in #wordpress-ui.
The value you'd want instead of none
is transparent
.
#4
@
13 years ago
- Version set to 3.3
The border-right-color: none
lines were merged in from the formerly-separate rtl CSS files (see r18579). They only apply for rtl.
#8
@
13 years ago
- Owner set to markjaquith
- Resolution set to fixed
- Status changed from new to closed
In [21188]:
#9
@
13 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
20587.2.diff fixes comment:1.
Per line 1797, it should be border-bottom-left-radius
:
http://core.trac.wordpress.org/browser/tags/3.4.1/wp-includes/css/editor.dev.css#L1792
Another oddball border issue:
./wp-includes/css/editor.dev.css:1992
sets propertyborder-bottom-right-left
(should this beborder-bottom-left-radius
maybe?)