WordPress.org

Make WordPress Core

#22825 closed defect (bug) (fixed)

Twenty Twelve: Buttons active status causes 2px shift

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

Description

There was one other location where I saw this, but unfortunately I can only find the behavior in the Twentytwelve buttons right now.

Here is a video of the bug and the fix:
http://dreamstarstudios.com/screencasts/2012-12-08_0226.swf

Attachments (1)

22825.patch (646 bytes) - added by ocean90 19 months ago.

Download all attachments as: .zip

Change History (14)

comment:1 SergeyBiryukov19 months ago

  • Component changed from UI to Bundled Theme

comment:2 ocean9019 months ago

  • Keywords has-patch added
  • Summary changed from Implementation of Bootstrap Buttons Causes 1px Shift to Twenty Twelve: Buttons active status causes 2px shift

ocean9019 months ago

comment:3 nacin19 months ago

  • Keywords commit added

comment:4 nacin19 months ago

I think this will need to wait for 3.6 as Twenty Twelve has already been released to the theme directory as 1.1. (It was supposed to wait for 3.5 final, but it got pushed early...)

comment:5 nacin19 months ago

  • Milestone changed from Awaiting Review to Future Release

comment:6 lancewillett19 months ago

@brianfeister Could you please add some more information?

  • Browser version
  • OS version
  • Example URL or steps to repeat in a test install

With display bugs that is essential info to help track this down, fix, and test thoroughly.

comment:7 follow-up: ocean9019 months ago

lancewillet, you saw the screencast?

  • Browser: Firefox, Chrome, Safari, Opera
  • OS: Win, Mac
  • Steps: Press the Post Comment button

comment:8 in reply to: ↑ 7 lancewillett19 months ago

Replying to ocean90:

lancewillet, you saw the screencast?

I didn't see any mention in the screencast of which permalink to test on or which browser / OS combinations. Crucial info! Also: s/lancewillet/lancewillett/ :)

comment:9 lancewillett19 months ago

  • Cc drewstrojny added
  • Keywords ui-feedback added; commit removed

Looking at this again, it *could* be intentional per the original design. I'll check with @drewstrojny.

comment:10 follow-up: drewstrojny19 months ago

Good catch! This is not intentional and I'm on board with the patch.

comment:11 in reply to: ↑ 10 lancewillett19 months ago

  • Keywords commit early added; ui-feedback removed

Replying to drewstrojny:

Good catch! This is not intentional and I'm on board with the patch.

Thanks Drew. I'll get it in after 3.5.

comment:12 lancewillett19 months ago

  • Milestone changed from Future Release to 3.6

comment:13 lancewillett17 months ago

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

In 23426:

Twenty Twelve: fix button active state to avoid a 2px shift, fixes #22825. Props ocean90.

Note: See TracTickets for help on using tickets.