Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#21473 closed defect (bug) (fixed)

Twenty Twelve: Style for comment form button

Reported by: obenland's profile obenland Owned by: lancewillett's profile lancewillett
Milestone: 3.5 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

The commentform submit button should receive some Theme styling.

Attachments (4)

21473.diff (1.4 KB) - added by obenland 12 years ago.
searchform-button.png (8.5 KB) - added by obenland 12 years ago.
comment-form.png (36.2 KB) - added by obenland 12 years ago.
21473.2.diff (1.5 KB) - added by obenland 12 years ago.

Download all attachments as: .zip

Change History (8)

@obenland
12 years ago

@obenland
12 years ago

#1 @obenland
12 years ago

21473.2.diff also makes button height browser compatible.

It seems the Firefox bug isn’t really a bug at all, but a “feature”: that is, it’s a deliberate decision by Firefox to limit line heights on buttons. This is evidenced by this line of CSS in Firefox’s default CSS:

button, input[type="reset"], input[type="button"], input[type="submit"] {
        line-height:normal !important;
}

Basically, Firefox is setting the line-height to “normal” on buttons and is enforcing this decision with an !important declaration.

Source

#2 @DrewAPicture
12 years ago

Screenshot following 21473.2.diff (FF 14) > http://cl.ly/image/442O3W300F3j

Last edited 12 years ago by DrewAPicture (previous) (diff)

#3 @lancewillett
12 years ago

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

In [21438]:

Twenty Twelve: better button styles for consistent look across browsers. Props obenland, fixes #21473.

#4 @SergeyBiryukov
12 years ago

  • Milestone changed from Awaiting Review to 3.5
Note: See TracTickets for help on using tickets.