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

md-subheader don't allow clicks or hover events for md-button inside #3932

Closed
mattiarigo-humus opened this issue Jul 28, 2015 · 4 comments
Closed
Assignees
Labels
pr: merge ready This PR is ready for a caretaker to review
Milestone

Comments

@mattiarigo-humus
Copy link

Hello,
I'm using 0.9.8 because 0.10.0 still has some little bugs.
When an md-button is inside an md-subheader there is no HOVER event for css rules neither click event.
I've tried this css solution

.md-subheader .md-subheader-content > *{
    position: relative;
    z-index: 10;
}

this rule works but it give me this on the page when scrolling

schermata 2015-07-28 alle 17 49 52

I've no hover event or click event for any of element inside md-subheader.
How can I fix it?

Thanks in advance,
\m

@marcysutton
Copy link
Contributor

A Codepen or Plunker would help diagnose this issue.

@pratyushmittal
Copy link

@marcysutton Here is a Codepen demo: http://codepen.io/anon/pen/xGQPyw

@topherfangio
Copy link
Contributor

@pratyushmittal I need to do some more testing to make sure it doesn't break anything else, but it appears you can add the following CSS to fix the issue:

.md-subheader .md-subheader-content {
  display: block;
}

Can you give it a try? If all looks good, we'll get this added/fixed in 0.12.0.

@pratyushmittal
Copy link

@topherfangio That seems to be working fine. Thanks.

topherfangio added a commit that referenced this issue Aug 21, 2015
Previously, the inner DIVs of the subheader component were not set
to use `display: block`, so you could potentially have contents
that were inaccessible.

fixes #3932
@topherfangio topherfangio added the pr: merge ready This PR is ready for a caretaker to review label Aug 21, 2015
kennethcachia pushed a commit to kennethcachia/material that referenced this issue Sep 23, 2015
Previously, the inner DIVs of the subheader component were not set
to use `display: block`, so you could potentially have contents
that were inaccessible.

fixes angular#3932. closes 4280.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants