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

md-menu not click able #6109

Closed
emps opened this issue Dec 6, 2015 · 8 comments
Closed

md-menu not click able #6109

emps opened this issue Dec 6, 2015 · 8 comments
Assignees
Milestone

Comments

@emps
Copy link

emps commented Dec 6, 2015

ng-click on buttons inside md-menu-item not working
( md-menu-item md-button ) im on master

btw demo of (codepen works)

@ttruongatl
Copy link

I had the same error.

@jayeshanandani
Copy link
Contributor

+1

@GoulwenBZH
Copy link

I had the same problem in rc6 and master.

@WolfSoko
Copy link

WolfSoko commented Dec 7, 2015

Did you check if it's a problem with z-index stacking? The pointer-events are catched by the md-backdrop.

For me, the md-backdrop that gets attached to the body, was in front of the opened menu.

The problem is that the md-open-menu-container is in RC6 a child of md-menu. If the md-menu is inside a different z-index stacking branch the backdrop can be above it.

<md-backdrop> --> z-index: 94
<any-element> 
  <another element> --> z-index: < 94
    <md-menu/> --> z-index: 95 but it is always under the backdrop because the z-index: 94 of the parent is handled as independent stacking branch.
  </another element>
</any-element>

I hope this helps a little bit. When you increase the z-index of the another-element you'll loose that click anywhere to close menu behavior (above the area of the another element).

@brianppoole
Copy link

I'm noticing the same thing as @SuperheroicCoding . Here's a short demo of some evidence that at least some of this could be z-index related:

menu-item-disabled

Also, I'm noticing the font color changed to white after updating to RC6, but that might be unrelated since I tend to play with the themes a lot.

If I have time soon, I'll create a CodePen isolating the issue.

@WolfSoko
Copy link

WolfSoko commented Dec 7, 2015

@brianppoole exactly, thanks for the demo. Yes the font color is maybe from a custom style. Perhaps on md-menu. I had the same issue.

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
@sathishvj
Copy link

@SuperheroicCoding Is there a workaround z-index we can use? Also on which element? I've been trying a bunch of combinations without success.

@WolfSoko
Copy link

WolfSoko commented Dec 8, 2015

@sathishvj
I don't see a quick fix for that.

But you can make the menu working again. Try to reduce the md-backdrop z-index until your open menu receives the clicks again. The menu will work, but the closing of the menu when you click outside will not work above the toolbar.

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

10 participants