WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#20382 closed defect (bug) (fixed)

When the admin menu is folded, the icons aren't centered

Reported by: azaozz's profile azaozz Owned by: azaozz's profile azaozz
Milestone: 3.4 Priority: normal
Severity: normal Version:
Component: UI Keywords: ui-feedback
Focuses: Cc:

Description

Noticed that when adding the retina display 2x sized icons.

Attachments (4)

before.jpg​ (11.0 KB) - added by azaozz 13 years ago.
after.jpg​ (11.2 KB) - added by azaozz 13 years ago.
20382.patch​ (5.1 KB) - added by azaozz 13 years ago.
20382.diff​ (4.7 KB) - added by helenyhou 13 years ago.

Download all attachments as: .zip

Change History (10)

@azaozz
13 years ago

@azaozz
13 years ago

#1 @azaozz
13 years ago

Seems most are 1px off center, the "posts" icon 2px.



@azaozz
13 years ago

#2 @azaozz
13 years ago

Not sure if the best way to fix this is by tweaking the sprite or in the css. 20382.patch corrects it from the css.

#3 @azaozz
13 years ago

  • Keywords ui-feedback added

#4 @helenyhou
13 years ago

Haven't figured out what caused that to shift, since the regular sprite didn't change but still seems affected, but a CSS change seems sane enough. The post icon actually looks like it was 1px too far to the left in 3.3.1 to begin with :( 20382.patch​ looks good to me, testing on both regular desktop and new iPad.

@helenyhou
13 years ago

#5 @helenyhou
13 years ago

Actually, the users menu icon seemed 1px to the right. 20382.diff​ for that.

#6 @azaozz
13 years ago

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

In [20400]:

Center the menu icons when the menu is folded. For proper display in RTL the width of .wp-menu-image has to be 30px. Props helenyhou, fixes #20382

Note: See TracTickets for help on using tickets.