Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

mdMenu does function properly inside an mdSidenav or mdList #6089

Closed
bgoerdt opened this issue Dec 4, 2015 · 6 comments
Closed

mdMenu does function properly inside an mdSidenav or mdList #6089

bgoerdt opened this issue Dec 4, 2015 · 6 comments
Assignees
Milestone

Comments

@bgoerdt
Copy link

bgoerdt commented Dec 4, 2015

In 1.0.0-rc6 the styling and click functionality of the mdMenu component is broken when inside a sidenav or list. The menu is sometimes cut off when it would normally go outside the parent container, and the intended click action does not fire.

Here is a CodePen demonstrating the issues:
http://codepen.io/bgoerdt/pen/pgoWBm

@nervestaple
Copy link

+1

Similar problems with mdToolbar.

http://codepen.io/anon/pen/pgodJM

@cranesandcaff
Copy link

Went back through the versions on bower, this issue starts after v1.0.0-rc5-master-08b5af5

@MilosStanic
Copy link

With 1.0.0-RC4 my menu looks like this
mdrc5

And with 1.0.0-RC6 my menu looks like this, and menu items are not clickable:
mdrc6

In my case md-menu is nested into md-toolbar.
Looks pretty much the same as @nervestaple's codepen

@MilosStanic
Copy link

Same as #6073

@corneschouten
Copy link

+1

@WolfSoko
Copy link

WolfSoko commented Dec 7, 2015

+1
Same as #6109

@ThomasBurleson ThomasBurleson added this to the 1.0-rc7 milestone Dec 7, 2015
rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089
rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089, fixes #6116
rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089, fixes #6116
rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089, fixes #6116
rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089, fixes #6116
rschmukler added a commit that referenced this issue Dec 8, 2015
fixes #6131, fixes #6109, fixes #6049, fixes #6073, references #6080,
fixes #6089, fixes #6116
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants