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

Option "Disable on Mobile" for Menu particle missing #2574

Closed
hacki65 opened this issue Dec 4, 2019 · 2 comments
Closed

Option "Disable on Mobile" for Menu particle missing #2574

hacki65 opened this issue Dec 4, 2019 · 2 comments
Labels
Milestone

Comments

@hacki65
Copy link

hacki65 commented Dec 4, 2019

As in #2538 suggested, this PR should be in the last Gantry v5.4.31. But it isn't. I have searched in the downloaded files for this, but nothing found. Seems that this was not completely merged or is there another problem?

In the changelog under "Common" it is listed as IMPROVED, but it's not in the downloads.

Can this be checked please.

@thexmanxyz
Copy link
Contributor

thexmanxyz commented Dec 5, 2019

@hacki65 I guess you are searching the in the PR proposed fix/setting on the wrong spot, as there was a long discussion where to apply the setting. Please check under Styles:

grafik

and then scroll down and you will find it on the Menu panel. It is named Hide on Mobile

grafik

You did not find it in the downloaded files because it is named Hide on Mobile and not Disable on Mobile. So yes, it was merged and there is/should be no other problem involved because it is visible on my installation on v5.4.31. If you're still not finding the setting report back, otherwise this issue can be closed!

@hacki65
Copy link
Author

hacki65 commented Dec 5, 2019

Thx, found it. I have customized the menu.yaml file with extra animation settings, so it was not visible for me. Sorry for wrong alert. Can be closed.

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

No branches or pull requests

3 participants