WordPress.org

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#10110 closed defect (bug) (fixed)

wp-admin/css/media.css rule for labels

Reported by: hailin Owned by:
Milestone: 2.9 Priority: normal
Severity: normal Version: 2.8
Component: UI Keywords: has-patch
Focuses: Cc:

Description

I remember Andrew added this rule to address multiple labels display case in media library.

.describe label {

display: inline;

}

One remaining issue is that those labels inherited font-weight: bold, so they are kind of ugly when the string is long.
I think we should make this more precise - it applies to "td" only, and make it font-weight: normal
Looks better this way:

.describe td label {

display: inline;
font-weight: normal;

}

Attachments (2)

media-css.jpg (53.3 KB) - added by hailin 12 years ago.
screenshot
10110_media_css.diff (431 bytes) - added by hailin 12 years ago.
patch

Download all attachments as: .zip

Change History (8)

@hailin
12 years ago

screenshot

@hailin
12 years ago

patch

#1 @Denis-de-Bernardy
12 years ago

  • Component changed from General to Accessibility
  • Milestone changed from Unassigned to 2.8.1
  • Version set to 2.8

#2 @Denis-de-Bernardy
12 years ago

  • Component changed from Accessibility to UI

#3 @Denis-de-Bernardy
12 years ago

  • Keywords has-patch added

#4 @hailin
12 years ago

Discussed with Andrew. It's better to clean up the css tree a bit and fix from the root.

#5 @azaozz
12 years ago

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

(In [11557]) CSS tweaks for the uploader and Media Library, props hailin, fixes #10110

#6 @azaozz
12 years ago

  • Milestone changed from 2.8.1 to 2.9
Note: See TracTickets for help on using tickets.