-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Bug on layout when Dialog is open #1259
Comments
Hi @nckenn, please post your browser and OS with versions. |
I have the same issue, I was thinking it was something I did. Windows 10 and latest chrome |
for now i added this class in my styles.scss to make it work properly : |
This seems to be coming from angular/components#7512 and angular/components#7390. I started from @nckenn 's reply above and isolated the change to be:
I have tested this fix on Edge, FF and Chrome and it works as expected (on both @nnixaa @yggg what do you guys think? Should we do this override? If yes, where should this be best placed? |
@aefox I guess we should try and check it doesn't introduce any performance drawbacks. I believe https://github.com/akveo/nebular/blob/master/src/framework/theme/components/cdk/overlay/_overlay.theme.scss would be a good place since we already overwriting some styles there. |
What is the status on this issue? It adds top & left to html which creates issue. |
For anyone that is still on 3.5. This still works. Simply add this line to your index.html.
|
For me, overflow: hidden did not work but overflow: initial worked. |
For me |
This happened to me too This wasn't enough for me. .cdk-global-scrollblock {
position: initial;
overflow-y: hidden;
} Combination with html style worked. html {
position: fixed;
top: 0;
}
.cdk-global-scrollblock {
position: initial;
overflow-y: hidden;
} |
i dont know if this is a bug or not, currently clone the master branch, and when i try to open dialog this happen, but on your demo its seems ok, but on my local its not. hope someone can help me :) thanks
The text was updated successfully, but these errors were encountered: