-
Notifications
You must be signed in to change notification settings - Fork 39
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
Comments
You can choose 2 layouts and you can do custom by override a module layout. /templates/cassiopeia/html/mod_jem/responsive/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 |
there is a pull request #1813 |
The fix of this issue is in PR #1813. |
in installed the mod_jem: it's very nice and opens new possibilities! Well done |
Then, open this model with Joomla layout in JEM modules. |
This issue come from #1813 This layout model for the module in Joomla needs to be exported to another JEM module. JEM basic module is finished. |
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'));
The text was updated successfully, but these errors were encountered: