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

Remove Layer - 2 (L2) .NET navigation header #2343

Closed
wadepickett opened this issue Apr 29, 2020 · 4 comments · Fixed by #2344
Closed

Remove Layer - 2 (L2) .NET navigation header #2343

wadepickett opened this issue Apr 29, 2020 · 4 comments · Fixed by #2344
Assignees

Comments

@wadepickett
Copy link
Contributor

wadepickett commented Apr 29, 2020

A new navigation header has been already published for all docs.microsoft.com It duplicates items already in the .NET layer 2 navigation header. We are removing the .NET layer 2 navigation header across all published dotnet content repos.

  • Remove the L2 .NET Header from docfx.json

Also reviewed:

  • Do we add a "Download .NET" and/or Community link to the EF Hub page or elsewhere?
    Conclusion: Do not include them.
@wadepickett
Copy link
Contributor Author

wadepickett commented Apr 29, 2020

@JeremyLikness, does this proposal sound good to you? The L2 Navigation that I am referring to is the ".NET About Learn Docs Downloads Community" navigation layer at the top of all the EF content

@JeremyLikness
Copy link
Member

I don't see a lot of community clicks and "download .NET" feels out of place on the Entity Framework Core hub. I recommend we just remove L2 navigation.

@JeremyLikness
Copy link
Member

Pinging @ajcvickers for his thoughts.

@ajcvickers
Copy link
Contributor

Agree with @JeremyLikness

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

Successfully merging a pull request may close this issue.

3 participants