-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add local nav to Team Updates and Project Updates if possible #34
Comments
Yeah I tend to agree. This should be a no-code change, as the local nav for each blog is derived from the primary menu for that site. I'll wait for @fcoveram to respond first. |
Local nav should always be present, as per this documentation in the Design Library. |
Ok, I'll add it. A bit of context: what's not obvious is that these are actually separate Make blog sites (like Design, Core, Meta, etc), so having the local nav on those blogs use the home local nav is unusual, when considering them as standalone sites. Given they are not linked to from anywhere else I don't think there are any issues with this. |
I've added the local nav to those 2 blogs. There is one inconsistency left in that the site title shows the name of that blog, eg. Project Updates looks like this: Changing the title of each of these blogs to 'Make WordPress' would fix this, but I'm not sure whether that would cause confusion in other ways. |
Thanks for the context @adamwoodnz. I see your point and understand the reason, but given that Project Updates and Team Updates have the layout and structure of make blogs, the navigation is consistent with most sections. Meeting looks more like an internal page, similar to the page listing all WordPress events, but I assume that "converting" that site into an "internal page" is out of the update scope.
To reduce confusion chances, we could keep the site link in local nav in active state (text in bold). So when visiting the Project Updates site, the "Project Updates" link in local nav is in bold all the time. This is visually consistent with most local nav updates. So when scrolling down, it should look like this What do you think of this? |
Meetings is actually an internal page, and I'm not sure why it's not showing an active state. I'll investigate and aim to have all the items in this local nav showing as active accordingly 👍 Rather than renaming the Team Updates and Project Updates blogs, I think I'll use a filter to change the site-title block in the local nav bar to display 'Make WordPress', so they also match the home site. |
I am not sure this is possible, but it feels like the local nav should be present on the Team Updates and Project Updates pages like it is for the Meeting page. I get why they don't, though.
What do you think @adamwoodnz @fcoveram
The text was updated successfully, but these errors were encountered: