No Code Attached Yet
avatar muhme
muhme
9 Sep 2025

Steps to reproduce the issue

  • Create some sub-categories below one parent category.
  • Create a site module 'Articles - Categories' with this parent category.
  • See four bugs/improvements for tab 'Advanced' respective the frontend HTML code.

Expected result

  1. The 'Heading Style' setting is used in the HTML code of the the frontend in the same way as in the 'Articles' module
  2. The 'Heading Style' offers selection 'h1' ... 'h6' and 'None' as in the 'Articles' module
  3. The items style selection is similar named in the 'Articles - Categories' and the 'Articles' module
  4. Inline help is given

Actual result

  1. The 'Heading Style' setting is irrelevant for frontend site HTML code, there is always 'li' used
  2. The 'Heading Style' offers selection 'h1' ... 'h5'
  3. Items style selection is named 'Heading Style' and thus differs from 'Header Level' in the 'Articles' module
  4. No inline help is given (this hurts as we have 'Heading Style' and 'Header Tag' selections in the 'Advanced' tab

System information (as much as possible)

  • Found during review of #46037
  • Tested in 5.4-dev, but I suspect it has been around for a while

Additional comments

  • Missing h6 found by @richard67 👍
  • If the 'Heading Style' selection is not needed (the categories should always only be 'li' entries) then the selection should be removed.
  • I find both the label 'Heading Style' and 'Header Tag' inappropriate. How about something like 'Item Style'?
avatar muhme muhme - open - 9 Sep 2025
avatar joomla-cms-bot joomla-cms-bot - change - 9 Sep 2025
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 9 Sep 2025
avatar brianteeman
brianteeman - comment - 9 Sep 2025

header tag is for the header of the module not any of the content in the module

Image
avatar muhme
muhme - comment - 12 Sep 2025

header tag is for the header of the module not any of the content in the module

Yes, my fault, the different names are 'Heading Style' and 'Header Level'. Corrected.

Add a Comment

Login with GitHub to post a comment