Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

filter hidden menu items #2398

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Chrissi2812
Copy link
Contributor

This is another approach to the issue I've tried to fix in #2348, but this will only affect Joomla.
It will change the menu editor behavior slightly, because hidden menu items won't show up anymore.
But that's the same for the editor in Grav.

@Chrissi2812
Copy link
Contributor Author

Any news on this? It's been a year since I opened it.

Joomla's mod_menu is doing something similar in it's mod_menu/helper.php where this method original comes from.

Maybe it's better to just port everything from there over?

@mahagr
Copy link
Member

mahagr commented Oct 11, 2019

The check should not be in this level, the same code is used in admin to edit pages.

Our code is delivered from some older version (J3.4 I think) of Joomla helper class and template file. I will need to take a look into this as I'm currently removing support for older Joomla versions.

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

Successfully merging this pull request may close these issues.

None yet

2 participants