Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#21847 closed defect (bug) (fixed)

Align the custom fields metabox

Reported by: ocean90's profile ocean90 Owned by: nacin's profile nacin
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.5
Component: UI Keywords: has-patch commit
Focuses: Cc:

Description

See screenshots.

Attachments (10)

21847.patch (2.2 KB) - added by ocean90 12 years ago.
before.png (23.3 KB) - added by ocean90 12 years ago.
after.png (22.9 KB) - added by ocean90 12 years ago.
21847.2.patch (2.8 KB) - added by SergeyBiryukov 12 years ago.
21847.after.2.png (13.7 KB) - added by SergeyBiryukov 12 years ago.
after.body.png (40.6 KB) - added by ocean90 12 years ago.
after.side.png (36.7 KB) - added by ocean90 12 years ago.
21847.3.patch (3.7 KB) - added by ocean90 12 years ago.
21847.4.patch (4.4 KB) - added by SergeyBiryukov 12 years ago.
21847.after.4.png (43.9 KB) - added by SergeyBiryukov 12 years ago.

Download all attachments as: .zip

Change History (19)

@ocean90
12 years ago

@ocean90
12 years ago

@ocean90
12 years ago

#1 follow-up: @scribu
12 years ago

It looks aligned to me (Firefox 15).

#2 in reply to: ↑ 1 @ocean90
12 years ago

Replying to scribu:

It looks aligned to me (Firefox 15).

Hmm. Not for me.

Opera 12.02: http://cl.ly/JL5x

Firefox 14/15: http://cl.ly/JKm6

#3 @scribu
12 years ago

Yeah, if I use Firebug's rulers, I can see a few pixels difference too.

#4 @SergeyBiryukov
12 years ago

Tested the patch in Chrome 21 and IE 7/8 as well, looks good.

'small' class should be appended, not prepended (#21852).

21847.2.patch also properly aligns "Enter new" link in Opera, "Add Custom Field" button in all browsers (attachment:21847.after.2.png), and removes a redundant line (#21833).

#5 @SergeyBiryukov
12 years ago

  • Component changed from Administration to UI

@ocean90
12 years ago

@ocean90
12 years ago

@ocean90
12 years ago

#6 @ocean90
12 years ago

Patch refreshed. Metabox is now optimized for #side-sortables too.

#7 @SergeyBiryukov
12 years ago

More tweaks and cleanup in 21847.4.patch. Screenshot: 21847.after.4.png.

A separate section for RTL is no longer needed, replaced with existing comment:
http://core.trac.wordpress.org/browser/tags/3.4.2/wp-admin/css/wp-admin-rtl.dev.css#L1274

Tested in Firefox 15, Chrome 21, IE 7, IE 8, Opera 12.

#8 @ocean90
12 years ago

  • Keywords commit added

#9 @nacin
12 years ago

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

In [21955]:

Better alignment in the ustom fields metabox. props SergeyBiryukov, ocean90, fixes #21847.

Note: See TracTickets for help on using tickets.