Skip to content

Category Top Navigation -> Maximal Depth is broken in > 2.1 #6925

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

Closed
pyostrike opened this issue Oct 7, 2016 · 5 comments
Closed

Category Top Navigation -> Maximal Depth is broken in > 2.1 #6925

pyostrike opened this issue Oct 7, 2016 · 5 comments

Comments

@pyostrike
Copy link

pyostrike commented Oct 7, 2016

It's said at http://docs.magento.com/m2/ce/user_guide/catalog/navigation-top.html that this value controls maximum number of levels that are available in top navigation. Actually it's not.

Preconditions

Steps to reproduce

Create a 3 level category tree. Set Maximal Depth to 1.

Expected result

Menu in top navigation will not show sub categories.

Actual result

Menu in top navigation shows all categories.

@pyostrike pyostrike changed the title Category Top Navigation -> Maximal Depth is broken in 2.1 Category Top Navigation -> Maximal Depth is broken in > 2.1 Oct 7, 2016
@veloraven
Copy link
Contributor

@pyostrike thank you for your feedback.
Please, format this issue according to the Issue reporting guidelines: with steps to reproduce, actual result and expected result.

@veloraven
Copy link
Contributor

According to contributor guide, tickets without response for two weeks should be closed.
If this issue still reproducible please feel free to create the new one: format new issue according to the Issue reporting guidelines: with steps to reproduce, actual result and expected result and specify Magento version.

@pyostrike
Copy link
Author

Хохлы ) Лишь бы не работать.

@dnadle
Copy link

dnadle commented Dec 8, 2016

Please reopen this. It is a fact that setting Maximal Depth -> 1 in v 2.1.2 does not prevent multiple menu levels from displaying. Also it looks like the original post was edited to conform to your guidelines.

@PascalBrouwers
Copy link
Contributor

Continue in #7543

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

No branches or pull requests

4 participants