WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 20 months ago

#20587 closed defect (bug) (fixed)

Invalid CSS with 'border-right-color: none'

Reported by: bananastalktome Owned by: markjaquith
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)

20587.diff (1.5 KB) - added by sabreuse 22 months ago.
20587.2.diff (420 bytes) - added by SergeyBiryukov 22 months ago.
20587.3.diff (408 bytes) - added by SergeyBiryukov 20 months ago.
Refreshed after [21592]

Download all attachments as: .zip

Change History (15)

comment:1 bananastalktome2 years ago

Another oddball border issue: ./wp-includes/css/editor.dev.css:1992 sets property border-bottom-right-left (should this be border-bottom-left-radius maybe?)

comment:2 helenyhou2 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.

comment:3 helenyhou2 years ago

  • Keywords 3.5-early needs-patch added

comment:4 coffee2code2 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.

sabreuse22 months ago

comment:5 sabreuse22 months ago

20587.diff fixes the border-color: none instances mentioned.

comment:6 coffee2code22 months ago

  • Keywords has-patch added; needs-patch removed

comment:7 SergeyBiryukov22 months ago

  • Keywords 3.5-early removed
  • Milestone changed from Awaiting Review to 3.5

comment:8 markjaquith22 months ago

  • Owner set to markjaquith
  • Resolution set to fixed
  • Status changed from new to closed

In [21188]:

border-right-color: none; should be border-right-color: transparent; props sabreuse. fixes #20587

SergeyBiryukov22 months ago

comment:9 SergeyBiryukov22 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:10 lessbloat20 months ago

20587.2.diff is a no-brainer. :-) Looks good.

comment:11 helenyhou20 months ago

  • Keywords commit added

SergeyBiryukov20 months ago

Refreshed after [21592]

comment:12 nacin20 months ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In [21661]:

Fix typo in CSS. props SergeyBiryukov. fixes #20587.

Note: See TracTickets for help on using tickets.