WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#21473 closed defect (bug) (fixed)

Twenty Twelve: Style for comment form button

Reported by: obenland Owned by: 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 3 years ago.
searchform-button.png (8.5 KB) - added by obenland 3 years ago.
comment-form.png (36.2 KB) - added by obenland 3 years ago.
21473.2.diff (1.5 KB) - added by obenland 3 years ago.

Download all attachments as: .zip

Change History (8)

@obenland3 years ago

@obenland3 years ago

@obenland3 years ago

@obenland3 years ago

comment:1 @obenland3 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

comment:2 @DrewAPicture3 years ago

Screenshot following 21473.2.diff > http://cl.ly/image/442O3W300F3j

Version 0, edited 3 years ago by DrewAPicture (next)

comment:3 @lancewillett3 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.

comment:4 @SergeyBiryukov3 years ago

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