The app's HTML results in a full-height Sidenav that covers the toolbar instead of staying below it
mtevrst opened this issue · 8 comments
Is no one addressing the issues in this starter app?
I think the issue comes from angular-material original source...
Check the screenshot of original website:
But, anyway - you can find other solution. I like how it looks when you adding md-toolbar to slide-nav:
Take a look at my example: http://codepen.io/anon/pen/ZOVvWX
@topherfangio - seems like a worthwhile fix.
@ThomasBurleson Correct me if I'm wrong, but
- On Desktop - Sidenav should be locked open and show the toolbar above the sidenav
- On Mobile - Sidenav should act as a standard sidenav which overlays the toolbar
The README never actually shows what should happen on mobile, but I think it is correct as-is.
@mtevrst - Is your sidenav hiding/showing even on desktop? If so, then that is definitely a bug, but should be fixed with the changes I'm making over in the wip/es6-tutorial-updated branch.
Agree with @topherfangio,
if we wanted the sidenav to be toggled on desktop it should be beneath the toolbar for sure, but on mobile it should overlay the toolbar.
will not fix.