Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrading to 2.9.0 breaks all functionality #964

Closed
kaiyoma opened this issue Aug 30, 2022 · 15 comments
Closed

Upgrading to 2.9.0 breaks all functionality #964

kaiyoma opened this issue Aug 30, 2022 · 15 comments

Comments

@kaiyoma
Copy link

kaiyoma commented Aug 30, 2022

Description of Problem

I've used Simple Lightbox for years without any issues, but 2.9.0 seems problematic. After upgrading, the lightbox functionality stopped working entirely. Clicking on images in my blog posts would navigate to the full image instead of displaying the JS modal. I was also seeing some subtle artifacts (a red horizontal line) when hovering over images.

I tried clearing the browser cache and that didn't help. I used a rollback plugin to roll back to version 2.8.1 and now everything is working correctly again, so 2.9.0 is definitely the problem. I'm happy to help you debug this, if you let me know what you need.

Details

  • URL of Page exhibiting problem: https://www.kylegetz.me/2021/10/10/fleet-week-air-show/ (not visible currently as the plugin has been rolled back)
  • Browser(s) exhibiting problem (Firefox, Chrome, etc.): Chrome, Edge, Firefox
  • Version of SLB Installed: 2.9.0
  • WordPress Version: 6.0.1
  • PHP Version: 7.4.3
  • Other plugins installed: Classic Editor, Favicon by RealFaviconGenerator, Jetpack, Open External Links in a New Window, WP Rollback
@archetyped
Copy link
Owner

Hi, the standard Support Ticket template contains all the fields necessary to evaluate issues, so I've updated your post with that template. Please edit your original post and fill in the fields, and I would be glad to take a look.

Thanks.

@kaiyoma
Copy link
Author

kaiyoma commented Aug 30, 2022

@archetyped Updated.

@archetyped
Copy link
Owner

archetyped commented Aug 30, 2022

Thank you for the details, however I need to be able to see the actual issue in order to evaluate it. SLB 2.9.0 has been tested on several sites without issue prior to release, so the issue may be specific to your site.

Please provide a link to a page that exhibits the issue you are experiencing (with all caching functionality disabled), and I would be glad to take a look.

Thanks.

@kaiyoma
Copy link
Author

kaiyoma commented Aug 30, 2022

@archetyped I just upgraded the plugin again, so you can see it live on the URL above.

@archetyped
Copy link
Owner

Thanks. Can you temporarily disable Jetpack to see if that resolves the issue?

@kaiyoma
Copy link
Author

kaiyoma commented Aug 30, 2022

Disabled Jetpack and your plugin is working again (yay! ... kind of). I've left the plugins in that state so you can look for yourself.

@archetyped
Copy link
Owner

archetyped commented Aug 31, 2022

Thanks. Looks like this is due to Jetpack's "Image Accelerator" feature, which modifies image URLs to serve them from Jetpack's own servers.

This will be looked into, but in the meantime, you can disable Jetpack's "Image Accelerator" feature to keep the image links from being modified.

@kaiyoma
Copy link
Author

kaiyoma commented Aug 31, 2022

@archetyped I disabled the image accelerator, and I'm still seeing the same problem. I don't think it's that setting: I think it's actually the lazy loading. Toggling that setting and refreshing the page changes the behavior.

@paroj
Copy link

paroj commented Sep 3, 2022

I think it's actually the lazy loading.

I can confirm that

@OldGrumpy-de
Copy link

OldGrumpy-de commented Sep 3, 2022

I observed the same issues as @kaiyoma but I don't run any caching plugin nor Jetpack. Disabling the built-in lazy load of WordPress did not help either. By disabling plugins systematically I found out that Nextgen Gallery (https://wordpress.org/plugins/nextgen-gallery/) seems to clash with Simple Lightbox 2.9.0. Simple Lightbox 2.9.0 works properly when Nextgen Gallery is disabled, so maybe there is some overlapping with internal shortcodes? Rolling back to Simple Lightbox 2.8.1 resolves the issue too. Nextgen Gallery is at the most recent version (3.28) for all tests. Did shortcode handling change between 2.8.1 and 2.9.0? Were new shortcodes introduced? I'm quite busy with other projects at the moment (including water in the basement after recent heavy rain, sigh...), but I'll gladly test or help debug, just let me know.

Edit: One detail I forgot to mention: I can see the shortcode [slb_group][/slb_group] around the gallery images, so it apparently doesn't get processed properly. Might be a valuable hint.

Edit 2: My galleries are standard WordPress galleries, using the [gallery][/gallery] shortcode. Nextgen is a legacy remnant being phased out. A number of old articles still needs to be revised to use the WordPress gallery instead of Nextgen, so for the moment I'm stuck with using both.

@archetyped
Copy link
Owner

I can see the shortcode [slb_group][/slb_group] around the gallery images, so it apparently doesn't get processed properly. Might be a valuable hint.

@OldGrumpy-de Can you please clarify what "gallery images" you're referring to? Standard WordPress galleries, Nextgen-created galleries, a different plugin?

Thanks.

@OldGrumpy-de
Copy link

OldGrumpy-de commented Sep 6, 2022

I can see the shortcode [slb_group][/slb_group] around the gallery images, so it apparently doesn't get processed properly. Might be a valuable hint.

@OldGrumpy-de Can you please clarify what "gallery images" you're referring to? Standard WordPress galleries, Nextgen-created galleries, a different plugin?

Oh, my bad. Sorry! My galleries are standard WordPress galleries, using the [gallery][/gallery] shortcode. Nextgen is a legacy remnant being phased out. A number of old articles still needs to be revised to use the WordPress gallery instead of Nextgen, so for the moment I'm stuck with using both.

@archetyped
Copy link
Owner

v2.9.1 contains the fix for this issue. Please clear WordPress and browser cache after updating plugin to ensure your site is loading the updated code.

@OldGrumpy-de the issues you reported (NextGen gallery, shortcodes displaying) could not be reproduced. If you are still experiencing this issue after updating to 2.9.1, please create a new ticket for each issue so that they can be fully evaluated.

Thanks all.

@kaiyoma
Copy link
Author

kaiyoma commented Sep 9, 2022

Just upgraded and the problem is fixed for me. Thanks @archetyped

@OldGrumpy-de
Copy link

@archetyped Just did some testing and the issue is fixed with the update to 2.9.1 for me as well. Thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants