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

Support for RN 0.30 zIndex #7

Closed
sodik82 opened this issue Jul 21, 2016 · 2 comments
Closed

Support for RN 0.30 zIndex #7

sodik82 opened this issue Jul 21, 2016 · 2 comments

Comments

@sodik82
Copy link
Contributor

sodik82 commented Jul 21, 2016

RN 0.30 now supports zIndex property that can be used to override default "zIndex strategy" in previous RN versions. This can lead to layout problems.

Pls add new config property to MenuContext that can influence zIndex of rendered backdrop and menu options. However for backward compatibility with previous RN versions there will be no default zIndex and will be activated only if user explicitly define desired zIndex.

madox2 added a commit that referenced this issue Jul 22, 2016
@madox2
Copy link
Collaborator

madox2 commented Jul 22, 2016

Release of this feature is blocked by react native issues: facebook/react-native#8970, facebook/react-native#8968

@sodik82
Copy link
Contributor Author

sodik82 commented Dec 17, 2016

closing with won't fix since RN "z-index" is somehow different from web and it does not seem to have sense to use it in our situation ( see facebook/react-native#8970 for more discussion)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants