[wp-trac] [WordPress Trac] #6453: Thickbox

WordPress Trac wp-trac at lists.automattic.com
Fri Jul 10 18:03:09 UTC 2009


#6453: Thickbox
--------------------------+-------------------------------------------------
 Reporter:  lmbbox        |        Owner:  anonymous
     Type:  defect (bug)  |       Status:  reopened 
 Priority:  low           |    Milestone:           
Component:  JavaScript    |      Version:  2.5      
 Severity:  normal        |   Resolution:           
 Keywords:  thickbox      |  
--------------------------+-------------------------------------------------
Changes (by chrishajer):

  * priority:  normal => low
  * resolution:  wontfix =>
  * status:  closed => reopened
  * component:  General => JavaScript
  * type:  enhancement => defect (bug)


Comment:

 Why was this closed as wontfix?  This issue still occurs on 2.8.1, and the
 theme is including the proper core /wp-includes/js/thickbox/thickbox.js,
 but in that file, the path to the images is hard-coded as ../wp-
 includes/js/thickbox/loadingAnimation.gif and ../wp-includes/js/thickbox
 /tb-close.png.  That makes the path relative to the current page, which
 results in a 404 error depending on the page slug (sometimes it works if
 you go ../ from the current page permalink.)  I can modify those two lines
 in the core file to make this work, but I think it should be fixed
 properly.

 Thank you.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/6453#comment:3>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list