WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#23536 closed defect (bug) (fixed)

Twenty Thirteen: Hover/focus style for buttons

Reported by: johnbillion Owned by: lancewillett
Priority: normal Milestone: 3.6
Component: Bundled Theme Version: 3.6
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

Buttons in Twenty Thirteen (eg. the "Post Comment" button) don't have a hover or focus style.

I played around with using the same light orange colour for the hover style that's used on links (#ea9629), but it's quite an abrupt change of colour for a hover. Maybe we could use an orange somewhere between #d94412 and #ea9629.

Attachments (2)

23536.patch​ (3.3 KB ) - added by lancewillett 14 years ago.
Patch from joen for hover/focus styles
23536.1.diff​ (3.2 KB ) - added by obenland 14 years ago.
Based off of Joen's work: a bit less intrusive, a bit more CSS style guideline conform

Download all attachments as: .zip

Change History (6)

#1 @lancewillett
14 years ago

  • Keywords needs-patch dev-feedback added
  • Milestone Awaiting Review → 3.6
  • Type enhancement → defect (bug)

@lancewillett
14 years ago

Patch from joen for hover/focus styles

#2 @lancewillett
14 years ago

  • Keywords has-patch needs-testing added; needs-patch dev-feedback removed

Added patch from joen -- needs testing.

Note: button / input styles might need better consolidation in the stylesheet. They are in two places right now.

@obenland
14 years ago

Based off of Joen's work: a bit less intrusive, a bit more CSS style guideline conform

#3 @lancewillett
14 years ago

  • Keywords commit added; needs-testing removed

New patch tests out well, thank you.

#4 @lancewillett
14 years ago

  • Owner set to lancewillett
  • Resolution → fixed
  • Status new → closed

In 23470:

Twenty Thirteen: add hover/focus style for buttons and organize button and input selectors a bit for a better cascade. Fixes #23536, props obenland.

Note: See TracTickets for help on using tickets.