-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Dropdown won't open in IE if openOnFocus is false #1315
Comments
Thanks for the report. Can you please paste the full error or a screenshot of it? Also, a working codepen will help us debug more thoroughly. @layershifter I'm guessing this is due to different event handling on windows, could you confirm? |
I'll check it on my PC tomorrow. |
Sadly my Internet Explorer is locked to Swedish (work computer) so the error messages was not so usable. I will try to reproduce the bug on codepen tomorrow. |
So here is a pen http://codepen.io/cattermo/pen/zNeqpY I can't reproduce the javascript error, I will have to look into why that occurs in our app but not in the pen. |
Thanks, getting closer here. Could you list specifically what happens when you click each of those parts? Example:
Something like this will help us track down which event handlers need looked into. |
My results are:
In summary everything works fine when clicking on the whitespace (between placeholder text and arrow) but clicking the arrow or the placeholder text requires a double click to make the dropdown open. |
We still have this problem in IE11 if anyone has the time to take a look... |
@cattermo I lost this issue from my sight. I've checked Dropdown's docs, but can't reproduce issue. Can you make a codepen with mininal code that shows error in IE? |
The bug can still be reproduced in the pen here http://codepen.io/cattermo/pen/zNeqpY (updated with semantic-ui-react 0.68.2) @layershifter I still have the same results as my comment from feb 14 says. |
@cattermo Oh, I missed that codepen was already there. It's a known bug with PR is here (#1667), will be awesome to get a feedback. |
Steps
openOnFocus={false}
Expected Result
The dropdown should open when clicked.
Actual Result
I get a javascript error from this row in the code:
if (options[nextIndex].disabled) return _this.moveSelectionBy(offset, nextIndex);
saying that
options[nextIndex]
is undefinedThe dropdown don't open
Version
0.64.5
The text was updated successfully, but these errors were encountered: