#19923 closed defect (bug) (fixed)

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

Reported by: convissor Owned by: helenyhou
Priority: normal Milestone: 3.4
Component: UI Version: 3.3.1
Severity: normal Keywords: has-patch commit
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 16 months ago.
wp--colors-admin-form-color.diff (653 bytes) - added by convissor 16 months ago.
Use 333 as color. Patch colors-classic too.
19923.diff (661 bytes) - added by helenyhou 13 months ago.
19923.2.diff (700 bytes) - added by helenyhou 13 months ago.

Download all attachments as: .zip

Change History (17)

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

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

Use 333 as color. Patch colors-classic too.

  • 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.

  • Keywords commit added; ui-feedback removed

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.

  • 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.

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

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.

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

  • 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

  • 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

  • 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.