Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#17889 closed defect (bug) (fixed)

Browser nag with no JS has hard to read grey text

Reported by: duck_'s profile duck_ Owned by: azaozz's profile azaozz
Milestone: 3.2 Priority: normal
Severity: normal Version: 3.2
Component: General Keywords: has-patch
Focuses: Cc:

Description

See screenshot attached.

If you do not have JavaScript enabled then the browser nag metabox text is the default dark grey which is hard to read and looks strange alongside the white links.

The problem is caused by the ui-sortable class, which is specified in the CSS, not being applied.

Attachments (6)

browser-nag-no-js.png (16.1 KB) - added by duck_ 12 years ago.
17889.diff (451 bytes) - added by duck_ 12 years ago.
17889.2.diff (594 bytes) - added by duck_ 12 years ago.
It's possible to move the color declaration to a different block
17889.3.diff (502 bytes) - added by aaroncampbell 12 years ago.
17889.2.png (44.0 KB) - added by aaroncampbell 12 years ago.
17889-4.patch (1.7 KB) - added by azaozz 12 years ago.

Download all attachments as: .zip

Change History (12)

@duck_
12 years ago

@duck_
12 years ago

It's possible to move the color declaration to a different block

#1 @azaozz
12 years ago

+1 for 17889.2.diff.

#2 @aaroncampbell
12 years ago

17889.2.diff doesn't change the color of the header. You could drop the color definition in #dashboard_browser_nag.postbox h3 as well, but I just combined it with #dashboard_browser_nag a in 17889.3.diff and it looks fine with and without JS.

#3 @nacin
12 years ago

I've always wondered why the font color flickered on refresh. Yes, let's please get this fixed.

@azaozz
12 years ago

#4 @azaozz
12 years ago

17889-4.patch also fixes all other styles that use the .ui-sortable selector.

#5 @aaroncampbell
12 years ago

17889-4.patch seemed to clean things up for me (with and without JS).

#6 @azaozz
12 years ago

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

In [18347]:

Fix text color in the browser nag when no JS, props duck_, fixes #17889

Note: See TracTickets for help on using tickets.