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

Opening mat-menu for the first time in MS Edge causes scrollbars to show #12112

Closed
kygr1990 opened this issue Jul 9, 2018 · 3 comments · Fixed by #12141
Closed

Opening mat-menu for the first time in MS Edge causes scrollbars to show #12112

kygr1990 opened this issue Jul 9, 2018 · 3 comments · Fixed by #12141
Assignees
Labels
P4 A relatively minor issue that is not relevant to core functions

Comments

@kygr1990
Copy link

kygr1990 commented Jul 9, 2018

Bug, feature request, or proposal:

Bug

What is the expected behavior?

Mat-menu scrollbars shouldn't show on first-time render on MS Edge if styling is not specified.

What is the current behavior?

Scrollbars in mat-menus are shown on first time rendering on Edge. Subsequent opening of mat-menus do not render scrollbars. Disabling and enabling overflow styling does not re-render scrollbars.

What are the steps to reproduce?

mat menu scroll1
mat menu scroll2

What is the use-case or motivation for changing an existing behavior?

This would appear to be incorrect behaviour for the mat-menu component

Which versions of Angular, Material, OS, TypeScript, browsers are affected?

Microsoft Edge 38.14393.2068.0
Microsoft EdgeHTML 14.14393
"@angular/material": "^6.2.0"
"@angular/core": "^6.0.3"

Is there anything else we should know?

@crisbeto
Copy link
Member

crisbeto commented Jul 9, 2018

I wasn't able to reproduce the issue by adding extra content to the menu. Can you post an example that shows the issue? You can fork one of the examples from the docs.

@kygr1990
Copy link
Author

Hi, I was able to reproduce it. Kindly check the below link:
https://stackblitz.com/edit/angular-nq6s32?file=app/menu-overview-example.css

Basically by simply specifying
[mat-menu-item] { width: 230px; }

I was able to reproduce the issue with reference to the above browser version.

mat menu scroll3

@crisbeto crisbeto self-assigned this Jul 10, 2018
@crisbeto crisbeto added has pr P4 A relatively minor issue that is not relevant to core functions labels Jul 10, 2018
crisbeto added a commit to crisbeto/material2 that referenced this issue Jul 10, 2018
Fixes the menu panel showing scrollbars the first time it's opened on Edge, if the width of the menu items has been overwritten.

Fixes angular#12112.
jelbourn pushed a commit that referenced this issue Jul 11, 2018
Fixes the menu panel showing scrollbars the first time it's opened on Edge, if the width of the menu items has been overwritten.

Fixes #12112.
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P4 A relatively minor issue that is not relevant to core functions
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants