Lightbox problems - now working - with fixed theme.inc (works for me!)

evilzenscientist

Joined: 2006-11-22
Posts: 29
Posted: Thu, 2007-09-27 04:59

Hey,

I've got WordPress 2.3 and Gallery2 2.2 installed and working great; I've got the WPG2 3.0 RC1 installed - and that also seems to be working.

I can't get lightbox to fire.

Any ideas?

 
ClimbingMonkey

Joined: 2005-03-18
Posts: 62
Posted: Thu, 2007-09-27 05:21

Have you have enabled the internal lightbox script from the WPG2 options tab? If so have you turned "on" the WPG2 Tags and sidebar support?

 
ozgreg
ozgreg's picture

Joined: 2003-10-18
Posts: 1378
Posted: Thu, 2007-09-27 05:24

Is that you MB??
____________________________________
Wordpress / Gallery2 (WPG2) Author

 
capt_kirk

Joined: 2006-01-24
Posts: 492
Posted: Thu, 2007-09-27 10:14

If you have RC1, you'll want to upgrade to RC Final: http://wordpress.org/extend/plugins/wpg2/

Kirk
____________________________________
G2Image Documentation, G2Image Demo Page, My Family Website

 
evilzenscientist

Joined: 2006-11-22
Posts: 29
Posted: Thu, 2007-09-27 15:13

Um - yes..

 
evilzenscientist

Joined: 2006-11-22
Posts: 29
Posted: Thu, 2007-09-27 15:31

Ok - I've upgraded (delete all of the old 3.0 release candidate; reinstall 3.0 rc final) and I still can't get this to fire.

Reading the docs and the changelogs etc - there is no underlying dependency on another set of lightbox js; yes it's enabled on the plugin page; yes all of the js are in <foo>/plugins/wpgs/js

Is the embedded theme from here lightbox enabled: http://www.galleryembedded.com/forums/viewtopic.php?t=7352

If I use another lightbox enabled theme; with the non WPG2 lightbox pieces, turning off the lightbox support - then things look ok.

 
capt_kirk

Joined: 2006-01-24
Posts: 492
Posted: Thu, 2007-09-27 16:30

Please try the latest SVN snapshot. I just made some changes that should remove a case sensitivity on the WPG2 tags. See this thread.

My first guess is that this may be the same problem you're seeing.

EDIT: It turned out that the other problem was not a case sensitivity problem, it was just a bad link. But could you try the latest SVN snapshot anyway. Also, check your wpg2header.php file to see if there are any "unneeded" lines, as lebaige noted here.

Kirk
____________________________________
G2Image Documentation, G2Image Demo Page, My Family Website

 
evilzenscientist

Joined: 2006-11-22
Posts: 29
Posted: Thu, 2007-09-27 20:30

capt_kirk

No joy.

Do you (or ozgreg) have an updated wordpressembedded theme - and just to confirm - does that directly support lightbox?

 
capt_kirk

Joined: 2006-01-24
Posts: 492
Posted: Thu, 2007-09-27 21:16

Evilzenscientist,

I think I just corrected another error in the lightbox URL generation. Please try the latest SVN snapshot again.

I don't have a lightbox enabled theme availble. Ozgreg may. He should be online soon, since it's 7:15 am in Melbourne.

Kirk
____________________________________
G2Image Documentation, G2Image Demo Page, My Family Website

 
ozgreg
ozgreg's picture

Joined: 2003-10-18
Posts: 1378
Posted: Thu, 2007-09-27 22:56

WordpressEmbedded Theme is Lightbox aware, however it has a problem with the detection of mimetypes and blindly applies lightbox tags to everything. I not solved this issue yet as it is a lower priority to bug fixing and documentation
____________________________________
Wordpress / Gallery2 (WPG2) Author

 
evilzenscientist

Joined: 2006-11-22
Posts: 29
Posted: Fri, 2007-09-28 00:17

Latest from svn worked. :)

Now the theme only throws up the largest version of the photo. Any pointers - I'm happy to go fix this one myself in the theme.

Feel free to mail me.

 
ozgreg
ozgreg's picture

Joined: 2003-10-18
Posts: 1378
Posted: Fri, 2007-09-28 01:23

It is way Gallery2 stores the resized images, I have a fix in WPG2 for this but not applied the fix yet to the WordpressEmbedded Gallery2 theme. I have a bug ticket open against this, but it is a little down the list..
____________________________________
Wordpress / Gallery2 (WPG2) Author

 
evilzenscientist

Joined: 2006-11-22
Posts: 29
Posted: Fri, 2007-09-28 01:33

ozgreg

let me know if there's anything I can do on this; give me some pointers and I'm happy to look at it.

 
ozgreg
ozgreg's picture

Joined: 2003-10-18
Posts: 1378
Posted: Fri, 2007-09-28 01:51

unfortunately it is code based, basically the selection logic in theme.inc cannot handle resized images being out of order and needs a mime type detection, unfortunately it is not a drop in and replace from WPG2 change as some of the function calls are either different or in the case of mimetype, not supported..
____________________________________
Wordpress / Gallery2 (WPG2) Author

 
evilzenscientist

Joined: 2006-11-22
Posts: 29
Posted: Fri, 2007-09-28 14:18

How about cheating until the rest of it is fixed on the gallery2 side.

I'm sure most people want a lightbox image of say around 1024x1024-ish. I can see the size selection logic in theme.inc. I'm going to hack around with this to pop up a mid-sized image for my stuff in the interim.

I'll post changes back.

ozgreg wrote:
unfortunately it is code based, basically the selection logic in theme.inc cannot handle resized images being out of order and needs a mime type detection, unfortunately it is not a drop in and replace from WPG2 change as some of the function calls are either different or in the case of mimetype, not supported..
____________________________________
Wordpress / Gallery2 (WPG2) Author

 
evilzenscientist

Joined: 2006-11-22
Posts: 29
Posted: Fri, 2007-09-28 15:32

Quick and dirty hack; here's an updated theme.inc that 'works for me'.

 
evilzenscientist

Joined: 2006-11-22
Posts: 29
Posted: Fri, 2007-09-28 16:24

... and for my images I used:

$lboxMaxEdge = 1025;

to make sure it worked for the resize options in the gallery.

Hope this helps a few people.

 
ozgreg
ozgreg's picture

Joined: 2003-10-18
Posts: 1378
Posted: Sat, 2007-09-29 00:26

Unfortunately your attachment is coming back 404, can you pop it in a zip and post it again, Thanks!! :)
____________________________________
Wordpress / Gallery2 (WPG2) Author

 
evilzenscientist

Joined: 2006-11-22
Posts: 29
Posted: Sat, 2007-09-29 01:12

theme.zip

contains theme.inc from the wordpressembedded theme for g2.

 
ClimbingMonkey

Joined: 2005-03-18
Posts: 62
Posted: Sat, 2007-09-29 03:43

evilzenscientist glad you got it fixed up...

Yep, ozgreg that's me... :) MonkeyBoy was already taken 2 1/2 years ago so on these forums I am a Climbing Monkey.. lol...

 
evilzenscientist

Joined: 2006-11-22
Posts: 29
Posted: Sat, 2007-09-29 04:46

Yeah - a nice afternoon of tracking down stuff. I've now got a working lightbox and g2image works for me..

All this for a family blog ;) www.allertonadventure.com

I don't write code!| www.evilzenscientist.com | www.gonetoutah.com |

 
ozgreg
ozgreg's picture

Joined: 2003-10-18
Posts: 1378
Posted: Sun, 2007-09-30 00:40

MB, Damn I have to check to see if you been Mod Rights then, I have a bad feeling I told Valiant Monkey?Boy :(

Evil -> Grab the latest WordpressEmbedded Snapshot -> http://www.wpg2next.com/snapshot/wesnapshot.zip and give it a try..

____________________________________
Wordpress / Gallery2 (WPG2) Author