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

Community: Breadcrumb should include "Community" #172

Closed
abjerner opened this issue Mar 5, 2018 · 2 comments
Closed

Community: Breadcrumb should include "Community" #172

abjerner opened this issue Mar 5, 2018 · 2 comments

Comments

@abjerner
Copy link
Contributor

abjerner commented Mar 5, 2018

Meetups is at the third level in the site structure, so the breadcrumb should also include Community (which is at level 2).

All the new community pages seems to be affected, so it probably only a single place that we have to fix.

@nul800sebastiaan If possible, feel free to assign this to me ;)

image

@nul800sebastiaan
Copy link
Member

It will do as soon as the community page is no longer hidden (naviHide is currently 0) :-)

@abjerner
Copy link
Contributor Author

abjerner commented Mar 6, 2018

Ah, I see. But should a breadcrumb really listen for umbracoNaviHide? I think this is specifically one of those moments where umbracoNaviHide should be ignored.

But as long as it works when the pages are made public ;)

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

No branches or pull requests

2 participants