WordPress.org

Make WordPress Core

Opened 21 months ago

Closed 21 months ago

Last modified 21 months 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 21 months ago.
searchform-button.png (8.5 KB) - added by obenland 21 months ago.
comment-form.png (36.2 KB) - added by obenland 21 months ago.
21473.2.diff (1.5 KB) - added by obenland 21 months ago.

Download all attachments as: .zip

Change History (8)

obenland21 months ago

obenland21 months ago

obenland21 months ago

obenland21 months ago

comment:1 obenland21 months 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 DrewAPicture21 months ago

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

Last edited 21 months ago by DrewAPicture (previous) (diff)

comment:3 lancewillett21 months 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 SergeyBiryukov21 months ago

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