Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#15856 closed defect (bug) (fixed)

show_admin_bar(false) doesn't work

Reported by: nacin Owned by: westi
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)

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.

This was crippled: [15938]

  • Owner set to westi
  • Status changed from new to accepted

This isn't show_admin_bar()'s fault.

Thickbox strips TB_iframe from the request

(In [17018]) Make note that this is a preview_iframe. See #15856

(In [17020]) If we are previewing don't show the admin bar. See #15856

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

(In [17021]) Ensure show_admin_bar( false ) works after init. Fixes #15856.

Note calling show_admin_bar( true ) late on is not going to evar evar evar be supported.

If you really want it ask early.

Note: See TracTickets for help on using tickets.