Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#19923 closed defect (bug) (fixed)

admin colors-*.css needs foreground color in form inputs

Reported by: convissor's profile convissor Owned by: helenyhou's profile helenyhou
Milestone: 3.4 Priority: normal
Severity: normal Version: 3.3.1
Component: UI Keywords: has-patch commit
Focuses: Cc:

Description

The colors-fresh style sheet sets the background color of form inputs to white but does not set a foreground color. This leads to problems for people who have non-standard browser preferences. A patch is attached that colors the text black.

Attachments (4)

wp-colors-fresh-form-color.diff (328 bytes) - added by convissor 13 years ago.
wp--colors-admin-form-color.diff (653 bytes) - added by convissor 13 years ago.
Use 333 as color. Patch colors-classic too.
19923.diff (661 bytes) - added by helenyhou 13 years ago.
19923.2.diff (700 bytes) - added by helenyhou 13 years ago.

Download all attachments as: .zip

Change History (17)

#1 @SergeyBiryukov
13 years ago

  • Component changed from General to UI
  • Keywords has-patch ui-feedback added; form foreground background color removed

#2 @SergeyBiryukov
13 years ago

  • Milestone changed from Awaiting Review to 3.4

#3 @convissor
13 years ago

helenyhou suggested that #333 would be better than #000. Please adjust the patch accordingly if deemed more appropriate.

@convissor
13 years ago

Use 333 as color. Patch colors-classic too.

#4 @convissor
13 years ago

  • Summary changed from colors-fresh needs foreground color in form inputs to admin colors-*.css needs foreground color in form inputs

So ignore the first patch, please.

#5 @SergeyBiryukov
13 years ago

  • Keywords commit added; ui-feedback removed

#6 @nacin
13 years ago

Our select elements currently have #000. Inputs are #333. #333 looks good on the white input background, but #000 looks better on the gray select background. Going with the first patch. UI folks can duke it out in a future release.

#7 @nacin
13 years ago

  • Owner set to helenyhou
  • Status changed from new to reviewing

The first patch also applies to inputs, so some juggling will need to occur here.

#8 @convissor
13 years ago

If using an earlier patch, make sure to apply the same change to both the classic and fresh css files.

#9 @helenyhou
13 years ago

Further investigation indicates that inputs also ignore the body's declaration of color and use browser styles instead. Freaking forms. Let's go with #000 for all of the above as per browser default and to keep it simple. New patch for both colors-*.css incoming.

@helenyhou
13 years ago

#10 @helenyhou
13 years ago

Actually ran across a duplicate: #11645. Has similar patch, but specifies colors that won't end up working well for selects in Webkit.

#11 @ryan
13 years ago

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

In [20659]:

Set foreground color to #000 for form inputs. Props helenyhou, convissor. fixes #19923

@helenyhou
13 years ago

#12 @helenyhou
13 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Per koop, we should lighten up the foreground color for inputs to match the body, since the background is white. Selects need to stay black, as there's that pesky gray background in Webkit. 19923.2.diff

#13 @koopersmith
13 years ago

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

In [20748]:

Lighten foreground color for inputs to match the standard body text. Also lighten several other instances of pure black. props helenyhou, fixes #19923.

Note: See TracTickets for help on using tickets.