Admin Bar Problem

[louyx]

Well, this happened after the update to the revision [16600]... the admin bar is now all over the content!

[louyx]

Screenshot

[ocean90]

Does it works when you add define( 'SCRIPT_DEBUG', true ); to your wp-config.php file?
Must be a cache problem because ryan forget to update script-loader.php. :-)

[ramoonus]

what browser are you using?

[pszeinert]

Replying to ramoonus:

what browser are you using?

I noticed some overlap in my custom theme too. I checked to see if the issue appeared on the default configuration of TwentyTen and verified the overlap in Chrome, FF and IE.

* WordPress 3.1 Beta; Chrome; TwentyTen no modifications *

The overlap should be expected since the admin bar is the last to load and #wpadminbar uses { position: fixed; width:100%; top: 0; } to position it. Nothing in the code compensates for the bar.

This is not so much an unintended consequence as an issue that has not yet been accounted for.

Possible solutions:
--Use JavaScript to collapse the bar to the right, left, or up.
--Call the wpadminbar sooner.
--Place the bar as fixed to the bottom, though this would likely suffer from the same issues as before.

[pszeinert]

Side by side comparison of wpadminbar

[nacin]

#15613

[filosofo]

I can look into this if ocean90 isn't already.

[ocean90]

I can't see a bug here really. In Backend it should work because wp-admin.css is now up2date too.
As mentioned on the original ticket #15507, themes should/must use body_class() where we had added the class "admin-bar" see [16432]. I'm not a fan of the idea to add hundreds of themes to core, only to position the admin bar nicely in there theme, like we did it before, see [16600].
So my idea:

• add the CSS stuff to style.css from 2010 to show how it works
• (re-)add a filter/action but only for the frontend and with the padding-top: 28px stuff

[pszeinert]

.admin-bar shows up in my body class, and adding { padding-top:28px; } does the trick, but .admin-bar is not located in any of the files I have (WordPress 3.1-beta1-16590).

[ryan]

I think registering a default callback that can be changed by the theme will work, much like we do with _custom_background_cb().

By default, register _admin_bar_bump_cb() against wp_head. The theme can call add_theme_support('admin-bar', 'custom_callback) which will replace the default callback with a custom callback.

[pszeinert]

Replying to pszeinert:

.admin-bar shows up in my body class, and adding { padding-top:28px; } does the trick, but .admin-bar is not located in any of the files I have (WordPress 3.1-beta1-16590).

Disregard my previous statement. The most recent files are on my server as I said, but I was searching files on an older 3.0.1 local dev install. After updating my local files, body.admin-bar exists in wp-admin.css (which most already knew). Sorry for the confusion.

Still can't see why it would not display properly.

[pszeinert]

Replying to pszeinert:

Still can't see why it would not display properly.

Actually, I can. On the admin side, load-styles.php runs and includes the body.admin-bar{padding-top:28px;}. On the site side, while still logged in, there is no call to include the styles in wp-admin.css.

I thought it might be the custom theme I am using, but even twentyten fails to add the body.admin-bar style.

[ocean90]

pszeinert: The padding-top is only for the admin side not for "theme side". Working now on ryans idea.

[ryan]

(In [16636]) Add bump down for menu bar. Allow themes to add a custom bump callback. Props ocean90. fixes #15592