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

Modern Events Calendar carousel loads the wrong image size with 1.21.x #446

Closed
joe9663 opened this issue Aug 26, 2022 · 1 comment · Fixed by #457
Closed

Modern Events Calendar carousel loads the wrong image size with 1.21.x #446

joe9663 opened this issue Aug 26, 2022 · 1 comment · Fixed by #457

Comments

@joe9663
Copy link

joe9663 commented Aug 26, 2022

Describe the bug
Carousel images displayed by the Modern Events Calendar when using their MEC Fluent-view Layouts extension are using the thumbnail image files. No issues with the display size when using PPB version 1.20.2.

The settings for the carousel allow you to specify the thumbnail size.
image

With PPB 1.20.2 the size is correct and the images loaded in the carousel uses the correct image files (ex. /wp-content/uploads/ab_resized/2022-ECK-durango-whole-expo-1-mec-thumb-376-212.jpg).

When using version 1.21.x the wrong image file is grabbed (ex. /wp-content/uploads/ab_resized/2022-ECK-durango-whole-expo-1-mec-thumb-322-250.jpg).

The correct image file still exists in the ab_resized directory, but there seems to be some kind of interference from the latest version and the wrong image is gathered by the shortcode.

To Reproduce
Modern Events Calendar is available for free through their website, but the Fluent-view Layouts extension is a premium plugin. There are 2 CWP installations under @jamesros161 BG Central account for troubleshooting with all of the necessary plugins installed, one with PPB 1.20.2 and the other with 1.21.1

@jamesros161 jamesros161 added this to the 1.21.2 milestone Aug 26, 2022
@joe9663
Copy link
Author

joe9663 commented Sep 20, 2022

Acceptance testing complete

This was referenced Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants