Thickbox module extended

Roberto Gerola - April 25, 2008 - 09:44
Project:Thickbox
Version:5.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:patch (code needs review)
Description

LeftRightMinds (Vancouver - Canada) has sponsored some great improvements for Drupal Thickbox module :

  • support for flashplayers (at now Flowplayer and Mediaplayer are supported)
    <a href="/files/audio/myaudiofile.mp3?width=200&height=20&shownavigation=true&showdigits=true&showstop=false&usefullscreen=false" class="thickbox-flashplayer" title="Audio link">Audio link</a>

    <a href="/files/videos/myvideofile.flv?width=400&height=300&usefullscreen=false" class="thickbox-flashplayer" title="Video link">Video link</a>

Width and height are optional. If not provided, the default values inserted in settings will be used.
The url must be absolute and you can pass on the url the parameters for the player you decided to use in the settings.

  • support for node body :
    To use it create an url like this :
    <a href="node/1?width=100&height=100" class="thickbox-nodebody">Launch popup</a>

    Instead of node/1, you can use also the url alias for the node :
    <a href="home?width=100&height=100" class="thickbox-nodebody">Launch popup</a>

    Width and height are optional. If not provided, the default values inserted in settings will be used.
  • For details and download see here : http://www.speedtech.it/drupal-thickbox-module

    Roberto

    #1

    JBrauer - May 15, 2008 - 23:41
    Status:active» active (needs more info)

    Roberto,

    Will there be a patch forthcoming with your changes?

    #2

    Tanc - May 19, 2008 - 23:33

    Yes, it would be good to see a patch. Also, when you say node body, does this updated module fetch the content from the database or does it run it through the templating system first? I'm wondering how much style control you get over the node content?

    #3

    christefano - June 25, 2008 - 00:45

    I'm also hoping these features will be contributed as a patch. Announcing these features here (and in the handbook pages) and hosting a forked version somewhere else breaks my heart a little.

    #4

    Roberto Gerola - June 25, 2008 - 17:05

    Ok, I'll provide a patch.

    #5

    Roberto Gerola - June 26, 2008 - 07:39
    Status:active (needs more info)» patch (code needs review)

    Hi.

    I created a patch against the last development version. See the attachment.

    Roberto

    AttachmentSize
    thickbox.patch12.39 KB

    #6

    christefano - June 26, 2008 - 10:45

    Thanks, Roberto! I'll be testing this soon.

     
     

    Drupal is a registered trademark of Dries Buytaert.