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

md-select inside md-dialog has incorrect position #6030

Closed
lmignot opened this issue Dec 2, 2015 · 10 comments
Closed

md-select inside md-dialog has incorrect position #6030

lmignot opened this issue Dec 2, 2015 · 10 comments

Comments

@lmignot
Copy link

lmignot commented Dec 2, 2015

As per this codepen: http://codepen.io/anon/pen/vLBeBy
When an md-select is placed inside an md-dialog the menu is not positioned correctly and the options are not clickable. keyboard navigation of the options still works.

I believe this is a similar issue to the one described in #6023 however that specifies that it's in the docs whereas the issue affects md-select anywhere

@DenisLaboureyras
Copy link

I've reproduced the same issue

@FabVillegas
Copy link

Checked that as well on my app.
IMO theres a quick fix: set the z-index to 80 of the class md-backdrop.md-select-backdrop

md-backdrop.md-select-backdrop {
    z-index: 80;
    transition-duration: 0;
}

This could restore functionality as a provisional way

@haochending
Copy link

I have the same issue, and the options are not clickable.

@matthiasg
Copy link

@FabVillegas the z-index fixes the backdrop click problem but the position is obviously still off

@matthiasg
Copy link

its helpful in the codepen to move the select to the right to see the error better.

http://codepen.io/matthiasg/pen/zrOeYQ

@matthiasg
Copy link

fixed in rc6

@villelahdenvuo
Copy link
Contributor

Confirm fixed in RC6.

@lmignot
Copy link
Author

lmignot commented Dec 4, 2015

confirm fixed in RC6

@mckenzielong
Copy link
Contributor

This is happening again.

7edda11: http://codepen.io/mckenzielong/pen/xZbqMQ <- broken
One commit before: http://codepen.io/mckenzielong/pen/qbErvv <- fine

rschmukler added a commit that referenced this issue Dec 7, 2015
rschmukler added a commit that referenced this issue Dec 8, 2015
@imamtauhid
Copy link

+1

rschmukler added a commit that referenced this issue Dec 8, 2015
rschmukler added a commit that referenced this issue Dec 8, 2015
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