Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 16 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: Focuses:

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)

#1 @nacin
16 years ago

  • Description modified (diff)
  • Summary Admin bar shouldn't show in theme preview iframe → show_admin_bar(false) doesn't work
  • Type enhancement → 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.

#2 @nacin
16 years ago

This was crippled: [15938]

#3 @westi
16 years ago

  • Owner set to westi
  • Status new → accepted

#4 @westi
16 years ago

This isn't show_admin_bar()'s fault.

Thickbox strips TB_iframe from the request

#5 @westi
16 years ago

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

#6 @westi
16 years ago

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

#7 @westi
16 years ago

  • Resolution → fixed
  • Status accepted → closed

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

#8 @westi
16 years ago

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.