Skip to content

Conversation

travisrisner
Copy link
Contributor

Pull Request for Issue #46056.

Summary of Changes

There is an item_heading parameter in the configuration of the module that is unused in the frontend.

Testing Instructions

  • Create new Articles Categories module
  • Swap to the Advanced tab and ensure there is no "Heading Style" option below layout

Actual result BEFORE applying this Pull Request

  • An option for "Heading Style" was displayed but was not used anywhere on the frontend.

Expected result AFTER applying this Pull Request

  • Unused option is now gone.

Link to documentations

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

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-5.4-dev labels Oct 14, 2025
@HLeithner
Copy link
Member

Thanks for your contribution, it's highly appreciated.

But this module is no longer maintained and subjected to be removed at some point. please use mod_articles.

Also removing a parameter could have impact of existing installations with layout overrides and might break them.

I'm closing this for now.

@travisrisner
Copy link
Contributor Author

Understood. Would there be any harm in me reworking this to just add in the h6 option even though it's going away at some point? If we allow h5, might as well allow h6?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language Change This is for Translators PR-5.4-dev

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants