Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#15510 closed defect (bug) (fixed)

Thickbox images can't be loaded in network admin

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

Description

The reason is:

if ( typeof tb_pathToImage != 'string' ) {
	var tb_pathToImage = "../wp-includes/js/thickbox/loadingAnimation.gif";
}
if ( typeof tb_closeImage != 'string' ) {
	var tb_closeImage = "../wp-includes/js/thickbox/tb-close.png";
}

Image link for network admin => wp.local/wp-admin/wp-includes/js/thickbox/tb-close.png

See also #6453.

Attachments (2)

15510.patch (815 bytes) - added by ocean90 14 years ago.
15510.2.patch (1.2 KB) - added by ocean90 14 years ago.

Download all attachments as: .zip

Change History (8)

#1 @nacin
14 years ago

Where are we using thickbox in the network admin?

If we are, then we should define those variables and inject them into the head when we need it. (That's the purpose of the check, which was added by us into TB.)

#2 @ocean90
14 years ago

Theme preview or plugin (upgrade) infos are usin thickbox.

@ocean90
14 years ago

#3 @ocean90
14 years ago

  • Keywords has-patch added; needs-patch removed

#4 follow-up: @nacin
14 years ago

Can the add_action() be put directly into add_thickbox if is_network_admin()? Seems cleaner.

@ocean90
14 years ago

#5 in reply to: ↑ 4 @ocean90
14 years ago

Replying to nacin:

Can the add_action() be put directly into add_thickbox if is_network_admin()? Seems cleaner.

Sure, why not. Patch attached.

#6 @nacin
14 years ago

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

(In [16963]) Plugin install and thickbox for the network admin. props ocean90, fixes #15510, fixes #15750.

Note: See TracTickets for help on using tickets.