-
Notifications
You must be signed in to change notification settings - Fork 179
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
Overflow Menu not working in IE #833
Comments
Analyzed the issue and found out that the usage of window.scrollX and window.scrollY causing the issue in IE. Seems the properties scrollX and scrollY doesn't support the IE browser. https://developer.mozilla.org/en-US/docs/Web/API/Window/scrollY Using window.pageXOffset and window.pageYOffset resolved the issue for me. Can we get this fix in next version? |
Sure @Vinoth-LearningThings will be a couple of days before I can take a look. Feel free to submit a PR which checks for the existence of scrollX/scrollY and defering to pageXOffset/pageYOffset which I may be able to review tomorrow. Include interactive tooltip in any fix. |
@Vinoth-LearningThings let me know if that works for you canary just published. |
@lee-chase Yup. Looks good to me. Thanks! |
Overflow menu is not showing up in IE
Detailed description
Steps to reproduce the issue
Additional information
The text was updated successfully, but these errors were encountered: