Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#15507 closed defect (bug) (fixed)

Remove wp_admin_body_style()

Reported by: ocean90's profile ocean90 Owned by:
Milestone: 3.1 Priority: normal
Severity: normal Version: 3.1
Component: Administration Keywords: has-patch
Focuses: Cc:


I think we can remove the function. For admin we can use $admin_body_class and wp-admin.css and or themes can use body_class() .

Attachments (2)

14772.wp_admin_body_style.patch (2.1 KB) - added by ocean90 13 years ago.
admin_side_admin-bar_cleanup.patch (327 bytes) - added by WraithKenny 13 years ago.
Minor CSS fix for admin-bar

Download all attachments as: .zip

Change History (10)

#1 @ryan
13 years ago

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

(In [16600]) Remove wp_admin_body_style(). Props ocean90. fixes #15507

13 years ago

Minor CSS fix for admin-bar

#2 @WraithKenny
13 years ago

Places the padding-top: 28px rule on the first appropriate element that isn't set to height: 100%; (which is #wphead).

In 3.1-RC3, you can see that the height of body.admin-bar is larger than 100% (easier to see on pages where the window is taller then the page content) which creates scroll-bars where none are needed.

(Might also provide a hint to Theme designers who look to wp-admin for css support on layouts that have css-positioned elements.)

Tested on Firefox 4.

#3 @WraithKenny
13 years ago

  • Resolution fixed deleted
  • Severity changed from normal to trivial
  • Status changed from closed to reopened

#4 @WraithKenny
13 years ago

  • Severity changed from trivial to normal

#5 @SergeyBiryukov
13 years ago

Patch tested, removes unnecessary scrollbars.

#6 @ryan
13 years ago

Looking okay in Chrome on Mac.

#7 @ocean90
13 years ago

Opera and Firefox too.

#8 @ryan
13 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [17368]) Admin bar CSS fix; removes unnecessary scrollbars. Props WraithKenny. fixes #15507

Note: See TracTickets for help on using tickets.