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
fix(aio): group topbar into a single sidenav node when narrow #17001
Conversation
I like it (except for the fact that other than |
…creen The `TopBarNarrow` now only shows a single top level container, "About Angular", and the original `TopBar` items will be children of this container. The `TopBarNarrow` styling is changed to match the rest of the `SideNav`.
9f3a2de
to
1030a35
Compare
The reason for this is that those nodes also appear in the Footer nav view, and that seems to get precedence. So they don't get highlighted in the TopBarNarrow nav view. I created an issue - #17022 - and will fix in a subsequent PR. |
@IgorMinar - I have removed the "Docs" item from the "TopBarNarrow" view as requested. PTAL. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much better! thanks
…creen (angular#17001) The `TopBarNarrow` now only shows a single top level container, "About Angular", and the original `TopBar` items will be children of this container. The `TopBarNarrow` styling is changed to match the rest of the `SideNav`.
…creen (angular#17001) The `TopBarNarrow` now only shows a single top level container, "About Angular", and the original `TopBar` items will be children of this container. The `TopBarNarrow` styling is changed to match the rest of the `SideNav`.
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
No description provided.