#15856 closed defect (bug) (fixed)
show_admin_bar(false) doesn't work
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.1 |
| Component: | Administration | Version: | |
| Severity: | normal | Keywords: | needs-patch |
| Cc: |
Description (last modified by nacin)
show_admin_bar(false) doesn't work. See the first comment for more.
My initial report:
Admin bar shouldn't show in theme preview iframe
It takes up space and looks weird. People are trying to preview a theme, and the admin bar is an unnecessary constant.
This should also be fixed.
Change History (8)
- Description modified (diff)
- Summary changed from Admin bar shouldn't show in theme preview iframe to show_admin_bar(false) doesn't work
- Type changed from enhancement to defect (bug)
This isn't show_admin_bar()'s fault.
Thickbox strips TB_iframe from the request
- Resolution set to fixed
- Status changed from accepted to closed
Note: See
TracTickets for help on using
tickets.

show_admin_bar() has apparently been crippled.
You should be able to call show_admin_bar( false ) before the admin bar is initiated, and it should kill the admin bar.
We should either eliminate this function, or fix it. I vote to fix it, otherwise putting this simple logic into preview_theme() doesn't work:
if ( isset( $_GET['TB_iframe'] ) ) show_admin_bar( false );And the call to show_admin_bar() in iframe_header() is useless.