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

[Layover] / [Menu] Invalid positioning of hidden Layovers in a dialog window #852

Closed
showme79 opened this issue Dec 10, 2019 · 0 comments
Closed

Comments

@showme79
Copy link

Describe the bug In

To Reproduce Steps to reproduce the behavior:

  1. Create a Dialog
  2. Add two containers: one is display: block, other is display: hidden
  3. Add one-one SelectField into the two containers (simplified mode false)
  4. Opening the SelectField in the first (block) container, the DropDown shows correctly
  5. Turn the container visible (using a state for example) and open the SelectField

Expected behavior A clear and concise description of what you expected to
The DropDowns must be rendered to the proper position, below the TextField part of SelectField.

Screenshots
Good one in the first container:
image
Bad one in the second container after making it visible:
image

Desktop (please complete the following information):

  • OS: any
  • Browser: any
  • Version: any
  • React MD version: 1.12.3

Smartphone (please complete the following information):
Not tested

Additional context
I have found that the problem is originated using node = node.offsetParent; in parent dialog popup. The fix node = node.offsetParent || node.parentNode; fixed the problem.

@showme79 showme79 added the bug label Dec 10, 2019
@mlaursen mlaursen added this to the v12.3.4 milestone Dec 12, 2019
mlaursen pushed a commit that referenced this issue Dec 12, 2019
…a dialog window #852 (#853)

* Fixed invalid positioning of hidden Layovers in a dialog window

----------------------
branch [[v1]]

* Fixed invalid positioning of hidden Layovers in a dialog window

----------------------
branch [[v1]]
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