Lightbox update - Image Gallery

[Barberousse]

Hello,

I did an update of the serendipity_event_lightbox plugin for my personnal purpose and you can now find my modifications in the official version in Spartacus.

Changelog:

Code: Select all

Version 1.7 (Barberousse):

    * Change graybox to greybox
    * Greybox upgraded to version 5.53
    * Lightbox 2 upgraded to version 2.04
    * Lightbox Plus upgraded to version 20061027
    * Pictures in the same entry are grouped in a gallery to use the "previous" and "next" button in Lightbox 2, Lightbox Plus, Greybox and Thickbox

Barberousse.

[ormus7577]

Thanks for your post. I've been working on the plugin as well and did similar things: http://board.s9y.org/viewtopic.php?t=12405

I'll download your changes and merge them where necessary, ok?

[Barberousse]

Thanks for your post. I've been working on the plugin as well and did similar things: http://board.s9y.org/viewtopic.php?t=12405

I'll download your changes and merge them where necessary, ok?

Of course !

Barberousse.

[konus]

Changelog:

Code: Select all

Version 1.7 (Barberousse):

    * Change graybox to greybox
    * Greybox upgraded to version 5.53
    * Lightbox 2 upgraded to version 2.04
    * Lightbox Plus upgraded to version 20061027
    * Pictures in the same entry are grouped in a gallery to use the "previous" and "next" button in Lightbox 2, Lightbox Plus, Greybox and Thickbox

Hello Barberousse, thank you for the update! I use Lightbox Plus and happy, that pictures are now grouped in a gallery. The only drawback I found out is, that several pictures of different entrys on the same page are all grouped together. Is this by design?

[Barberousse]

The only drawback I found out is, that several pictures of different entrys on the same page are all grouped together. Is this by design?

Oh no, it shouldn't, I thought I tested it correctly. What gallery are you using: Lightbox plus, Greybox, ... ?

Edit: sorry, you're right, I have only one entry by page on my blog, and I didn't test with more, I was too much confident with my code... I will try to correct it soon.

Barberousse.

[judebert]

I'm using lightbox plus, and I like that behavior. The condition of multiple entries only occurs on my frontpage or category pages; then users get a sort of "overview" of everything on the page.

Could you include an option to choose between the possible behaviors?

[Barberousse]

Done,

There is a configuration option now to choose between no gallery, gallery for the entry or gallery for the page.

Redownload from the same location and replace, this is now the version 1.18.

Barberousse.

[judebert]

Wow, that was fast! Thanks!

Did you update to SPARTACUS as well, or do you want me to do that after we've verified the correct operation?

[Barberousse]

Well,

I never tried to add something to Spartacus, so if you already have access rights, could you do it please?

Is there a documentation somewhere about Spartacus for plugin developpers?

Barberousse.

[konus]

I am really amazed about the speed and the helpfulness in this community. I love it! I would like to contribute the German translation. Please add to the leng_de.inc.php the following lines:

Code: Select all

@define('PLUGIN_EVENT_LIGHTBOX_GALLERY',  'Gallerie Erzeugung');
@define('PLUGIN_EVENT_LIGHTBOX_GALLERY_DESC',  'Steuert, ob das blättern zum nächsten Bild über Pfeile möglich ist.');
@define('PLUGIN_EVENT_LIGHTBOX_GALLERY_NONE', 'keine Gallerie');
@define('PLUGIN_EVENT_LIGHTBOX_GALLERY_ENTRY', 'nur Fotos eines Artikels');
@define('PLUGIN_EVENT_LIGHTBOX_GALLERY_PAGE', 'alle Fotos der Seite');

I invented the tag PLUGIN_EVENT_LIGHTBOX_GALLERY_DESC, just in case it will be needed in the future.

[garvinhicking]

Hi!

Thanks a lot for the contributions! I just added the update by Barberousse to Spartacus as well as the updated german translation by konus (slightly modified).

Barberousse, you changed some code that was in CVS regarding the regexp. Performancewise that's not the best way, so I changed it to the way it was before, bumping the version to 1.9.

(Updated files here: http://php-blog.cvs.sourceforge.net/php ... _lightbox/)

Regards,
Garvin

[judebert]

Thanks, Garvin. I've got too many steaks on the grill...

[Barberousse]

Barberousse, you changed some code that was in CVS regarding the regexp. Performancewise that's not the best way, so I changed it to the way it was before, bumping the version to 1.9.

You're right, I did it too quickly...

Barberousse.

[konus]

I have one small contribution to lightbox plus:

In the lightbox_plus.css I modified the last lines

Code: Select all

* html #overlay {
	background-color: #000;
	background-image: url(blank.gif);
	filter: alpha(opacity=50);

to

Code: Select all

* html #overlay {
	background-color: #000;
	background-image: url(blank.gif);
	filter: alpha(opacity=50);  /* IE */
	-moz-opacity: 0.50; /* Gecko */
	opacity: 0.50; /* Opera */
}

I belive it schould be even more cross browser compatible now (esprecially for older ones).

[konus]

I have two tips on using Lightbox plus:

Tip 1: If you add

Code: Select all

title="some caption"

inside your <img>-tag, Lightbox will show the caption below the image.

Tip 2: If you would like to display a logo or echoic word as an overlay over the image, change the last lines at lightbox_plus.js to

Code: Select all

effectclass:'serendipity_image_link'