No Code Attached Yet bug
avatar brianteeman
brianteeman
27 May 2021

Steps to reproduce the issue

Step 1

Go to the global options for a menu item and set a value for all 4 options
chrome_2021-05-27_18-32-30

Step 2

Create a new menu item and check the settings in the page display tab - expected behaviour would be to see the values from step 1

chrome_2021-05-27_18-34-03

Step 3

Now we know that the page heading and page class global settings are ignored - what about the browser title that is at least displayed in the menu item options. Nope thats ignored

chrome_2021-05-27_18-38-32

Summary

The only one of the four options that actually does anything is the third one "Show Page Heading" but even that doesnt really doing anything as it is meaningless without a value for "Page Heading"

Conclusion

There is zero point in having any of these global options for a menu item as they dont work as their is no code to use them and they can be removed - unless I have missed something

System information (as much as possible)

Additional comments

avatar brianteeman brianteeman - open - 27 May 2021
avatar joomla-cms-bot joomla-cms-bot - change - 27 May 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 27 May 2021
avatar tonypartridge
tonypartridge - comment - 27 May 2021

They used to work in J3, so I’d expect them to work in J4... we have actually be using them for meta management and page H1’s.

And page class to define custom classes within the theme...

Wouldn’t it make more sense to add the support rather than remove it?

avatar brianteeman
brianteeman - comment - 27 May 2021

are you sure they work in j3? The broken behaviour seems identical to me.

avatar chmst
chmst - comment - 28 May 2021

They don't work in J3 either, at least in content. @tonypartridge magine that all sites have the same title? a11y and seo ....

avatar chmst
chmst - comment - 31 May 2021

Finally found what happens and where.

Before a PR I would like to get your opinion if we can remove the site title form gobal params. Equal title for all pages is an accessibility issue if it works. @brianteeman


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34248.
avatar brianteeman
brianteeman - comment - 31 May 2021

I agree equal titles is wrong. Hard to confirm that this param creates that though as I dont see where it can be used

avatar chmst
chmst - comment - 31 May 2021

It does. I will prepare a draft PR.and let the param, then you can see it.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34248.

avatar brianteeman
brianteeman - comment - 31 May 2021

great

avatar brianteeman brianteeman - change - 1 Feb 2022
Status New Closed
Closed_Date 0000-00-00 00:00:00 2022-02-01 23:03:48
Closed_By brianteeman
Labels Added: No Code Attached Yet
Removed: ?
avatar brianteeman brianteeman - close - 1 Feb 2022
avatar richard67 richard67 - change - 30 Oct 2022
Status Closed New
Closed_Date 2022-02-01 23:03:48
Closed_By brianteeman
avatar richard67 richard67 - reopen - 30 Oct 2022
avatar richard67
richard67 - comment - 30 Oct 2022

Re-opening this issue since the PR has been closed without merging.

avatar chmst chmst - change - 20 Feb 2023
Labels Added: bug
avatar chmst chmst - labeled - 20 Feb 2023

Add a Comment

Login with GitHub to post a comment