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

Module overrides with core ModuleHelper #1803

Open
reilldesign opened this issue Jul 28, 2024 · 2 comments
Open

Module overrides with core ModuleHelper #1803

reilldesign opened this issue Jul 28, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@reilldesign
Copy link

Currently only two layouts are supported for the mod_jem (legacy & responsive layout). it would be very useful if other overrides were also possible with the help of the Joomla core ModuleHelper.

Old
require(JemHelper::getModuleLayoutPath($mod_name));

New
require ModuleHelper::getLayoutPath('mod_jem', $params->get('layout', 'default'));

@mckillo
Copy link
Collaborator

mckillo commented Aug 2, 2024

You can choose 2 layouts and you can do custom by override a module layout.
Each JEM module searches in these paths and order, for example mod_jem when responsive layouts is active in JEM setting:

/templates/cassiopeia/html/mod_jem/responsive/default.php
/modules/mod_jem/tmpl/responsive/default.php
/templates/cassiopeia/html/mod_jem/default.php
/modules/mod_jem/tmpl/default.php

JEM hasn't Joomla layout, the layouts are internal by global setting, but it's true that it could change to Joomla layout. In that case, there are to change code in several parts of the modules, but it could be viable (tested) to the improvement in low priority. You have to scale it to see the convert /modules/mod_jem/tmpl/responsive/default.php to /modules/mod_jem/tmpl/responsive.php

@Heklaterriol Heklaterriol added the enhancement New feature or request label Aug 5, 2024
@reilldesign
Copy link
Author

there is a pull request #1813

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants