[5.4] - mod_articles add h6 option for article title heading level#46037
[5.4] - mod_articles add h6 option for article title heading level#46037muhme merged 1 commit intojoomla:5.4-devfrom
Conversation
|
Or use the form field type="headertag" instead of all the Options? |
|
@chmst headertag is for the module title. This pr is for the article title |
|
I know, but we could use the form field everywhere - it simply generates the options for headings. |
|
I have tested this item ✅ successfully on c327cdd This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46037. |
|
@chmst I don't see how that's helpful in this use case. If we let you select h5 what's wrong with being able to select h6 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46037. |
|
Our global administrator language file has language strings JH1 to JH6: https://github.com/joomla/joomla-cms/blob/5.4-dev/administrator/language/en-GB/joomla.ini#L35-L40 I see no reason why not to use all 6. Only question is why not do the other modules, too?
The latter is legacy so maybe that should not be changed. But the former still seems to be state of the art (not legacy) so that could be changed, too. @brianteeman What do you think? Change these 2, too? Or only the former? |
|
For consistency you are correct they should be changed. I leave it to the maintainers to decide if the legacy module should be updated or not. |
|
I have tested this item ✅ successfully on c327cdd This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46037. |
|
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46037. |
|
Final test before merge with JBT graft nightly build and full package from PR Notes:
|
|
Thank you @travisrisner for your contribution. Thank you @brianteeman, @richard67 and @chmst for your comments. Thank you @brianteeman and @mothershedc for testing. |
Summary of Changes
Heading level selection for Article titles in mod_articles previously just went h1-h5. This adds h6 as an option.
Testing Instructions
Display Optionstab and selecth6as your value forHeader Level.Actual result BEFORE applying this Pull Request
No h6 was available in the list of options
Expected result AFTER applying this Pull Request
h6 is now an available option.
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