Closed (duplicate)
Project:
Thickbox
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
15 Jun 2007 at 17:36 UTC
Updated:
31 Jul 2009 at 11:53 UTC
Hi, the theme I'm working on requires that thickboxes have the close link on the bottom. I modified thickbox.js a little some weeks ago but during the latest cvs update I lost my changes because default close link location is on top.
I've added a setting to thickbox module so that admins can choose if they want to use the close link on top or bottom, the setting is passed from PHP to Javascript thanks to drupal_add_js() and Drupal.settings.
An alternative could be passing the close link location as a part of the link calling the thickbox in HTML.
Patch is attached.
What you think?
| Comment | File | Size | Author |
|---|---|---|---|
| thickbox-close-location.patch | 4.73 KB | alexis |
Comments
Comment #1
frjo commentedSee http://drupal.org/node/163460