Skip to content

1.1.0#5

Open
exa18 wants to merge 8 commits intoconcrete5japan:mainfrom
exa18:2.x.DEV
Open

1.1.0#5
exa18 wants to merge 8 commits intoconcrete5japan:mainfrom
exa18:2.x.DEV

Conversation

@exa18
Copy link
Copy Markdown

@exa18 exa18 commented Aug 13, 2025

Fixes to version 2.0.0/1.0.2

  • fix block form.php div class for slider and testimonial carousel to tabs work
  • fix from issue Images get deformed on mobile #4
  • added image-noscale class to allow images have own original size
  • controller version bump, but also added XS and XL sizes, and all sizes rounded up to have little more sharp images (i use/prefer for large 1400 and for xlarge 2000)

controller.php Outdated
protected $appVersionRequired = '8.0.0';
protected $pkgVersion = '1.0.2';
protected $appVersionRequired = '9.0.0';
protected $pkgVersion = '2.0.2';
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@exa18

Thank you for your contribution! I'll test it, but the changes in this pull request do not require a minimum supported version change. Can you keep it, and change the package version to 1.1.0?

@exa18
Copy link
Copy Markdown
Author

exa18 commented Aug 19, 2025

sure, NP, I did 2.x while concrate market holds 2.0.0

in fact, changes require version 9.x while block majorca_slick_slider and majorca_testimonial_carousel inside FORM.php changin html tags which allow switch tabs while edit. And infact only this.

"
I thought to add IF for those html tags inside form.php to make them also for 8.x. Maybe if u wish then next PR.

@exa18 exa18 changed the title 2.0.2 1.1.0 Aug 19, 2025
Attempted to call an undefined method named "getBlockTypeToolsURL" of class "Concrete\Core\Application\Service\Urls".
Did you mean to call e.g. "getBlockTypeAssetsURL", "getBlockTypeCssURL" or "getBlockTypeIconURL"?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants