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

ISBAT ...Carousel block doesn't allow images to be enlarged #2571

Closed
vlaisavljevic opened this issue Oct 7, 2023 · 5 comments · Fixed by #2589
Closed

ISBAT ...Carousel block doesn't allow images to be enlarged #2571

vlaisavljevic opened this issue Oct 7, 2023 · 5 comments · Fixed by #2589
Labels
[Priority] Low This issue/pull request is not immediate [Type] Bug Something that is not working as expected

Comments

@vlaisavljevic
Copy link

vlaisavljevic commented Oct 7, 2023

Describe the bug:

Images from the Carousel block cannot be enlarged on CoBlocks 3.1.0+

To reproduce:

  • Install and activate version 3.1.0+
  • Create a simple Carousel block (Lightbox variation)
  • Save the page and try to enlarge images on the front

Expected behavior:

The images should be enlargeable

Screenshots:

https://ibb.co/YW4jMjj

@vlaisavljevic vlaisavljevic added [Priority] Low This issue/pull request is not immediate [Type] Bug Something that is not working as expected labels Oct 7, 2023
@vlaisavljevic vlaisavljevic changed the title ISBAT ... ISBAT ...Carousel block doesn't allow images to be enlarged Oct 7, 2023
@chrisjreber
Copy link

I'm having the same issue. I've verified that the lightbox code isn't even outputting the lightbox HTML on the page. If I straight up copy the coblocks-lightbox.js into the console it works just fine.

@AnthonyLedesma
Copy link
Member

This is resolved with CoBlocks 3.1.6

@AnthonyLedesma
Copy link
Member

Related #2589

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Priority] Low This issue/pull request is not immediate [Type] Bug Something that is not working as expected
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants
@chrisjreber @AnthonyLedesma @vlaisavljevic and others