User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
Status | New | ⇒ | Pending |
Category | ⇒ | Modules Front End |
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-07-23 09:29:37 |
Closed_By | ⇒ | bembelimen | |
Labels |
Added:
Test instructions missing
Information Required
PR-5.3-dev
|
Thank you for your Pull Request, really appreciate that you contributed.
As Brian said, the suffix should be managed by the chrome and was changed here by purpose:
So I will close this PR but I hope you don't feel discouraged by this and continue contributing.
Thank you for your effort!
Ok, so the $params->get('moduleclass_sfx should be at the chrome implementation?
You can see an example here: https://github.com/joomla/joomla-cms/blob/5.3-dev/layouts/chromes/html5.php#L27
Cool, thanks!
This is not correct. The module class suffix is added by the module chrome. If you are not seeing this in your own template then you need to set the correct module chrome