-
Notifications
You must be signed in to change notification settings - Fork 92
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
Layout header does not re-render #452
Comments
Could you please provide a plunkr ? |
pls provide a plnkr that demonstrates your problem. Thx. |
I only managed to reproduce it on plnkr a few times. And only while in full preview mode. http://embed.plnkr.co/f860usb4eBdTAsfDH5he/ Edit: The issue is that the header acts as if the drawer is open. It happens 100% of the times on my current project. And as I said, only managed to reproduce it a few times at plnkr. :/ Edit2: This is what it looks like in my project after login. |
Found out what's causing it and updated the plnkr. The first login works fine. If you then click logout and try to login again the error appears. |
I was not able to reproduce, what browser are you using ? |
Chrome 55.0.2883.95 on macOS 10.12.2 Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.95 Safari/537.36 Edit: Just tried in IE, Safari and Firefox aswell. Same issue. |
@nikoTM Note that it only appears in mobile-view (drawer closed). |
@mseemann seems like is-small-screen is not being added for some reason. |
I've got a similar issue as I had with #89 but in the latest version.
I've got a full page login component. After you login I use
router.navigate(['/']);
to navigate to a component for logged in users (contains the layout). This does not trigger the media query listeners from the looks of it.The layout header has
margin-left: 240px;
and does not update even if I resize the page.The text was updated successfully, but these errors were encountered: