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

[Dropdown] Search In-Menu repeatedly opens/closes in IE11 #4237

Closed
Async0x42 opened this issue Jul 7, 2016 · 9 comments
Closed

[Dropdown] Search In-Menu repeatedly opens/closes in IE11 #4237

Async0x42 opened this issue Jul 7, 2016 · 9 comments
Milestone

Comments

@Async0x42
Copy link

Async0x42 commented Jul 7, 2016

http://jsfiddle.net/j9nup1kv/2/

I can't get the jsfiddle to work on my IE11, but the code is the example here: http://semantic-ui.com/modules/dropdown.html#search-in-menu

In IE11 the dropdown will continue to 'bounce' open and shut repeatedly. The class alternates from visible to active over and over.

Browser: IE11
OS: Win10 x64

@jlukic
Copy link
Member

jlukic commented Jul 29, 2016

Wow IE, thats pretty epic bad behavior. Confirmed

@jlukic jlukic added this to the 2.2.4 milestone Jul 29, 2016
@christianklemp-imt
Copy link

christianklemp-imt commented Sep 14, 2016

+1 for a fix

@joeweale
Copy link

any luck in getting this fixed?

@jockeisorby
Copy link

jockeisorby commented Oct 27, 2016

It seems like IE doesn't like the immediate focus on the search input when clicking the dropdown.
I made the following quick fix to make the search menu work in IE for my project:
Removed the entire div class="scrolling menu" from the example code, it appears that it works better when the items in the dropdown is higher up in the hierarchy.
See the label filter example on the semantics page.
But that only solved it about 50% of the time, I also hacked the dropdown.js file and commented out the following part:
if(module.has.menuSearch()) {
module.focusSearch();
}.

@jlukic jlukic modified the milestones: 2.2.6, 2.2.7, 2.2.8 Oct 28, 2016
@moondawg69
Copy link

Any luck on a fix with this, got a huge admin UI going out and this is the only issue LOL ;)

@jockeisorby
Copy link

I solved it by styling the dropdown as a link which shows the search menu
onclick.

Den 8 nov. 2016 1:47 em skrev "Jez Timms" notifications@github.com:

Any luck on a fix with this, got a huge admin UI going out and this is the
only issue LOL ;)


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#4237 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARfsj7--qaVKWzdkdeMa0V7PnVmHf3ZXks5q8G9UgaJpZM4JHGb7
.

@moondawg69
Copy link

That may not work being a large Angular app, I'll have to pass it onto the Angular devs. Thank you anyway.

@jlukic
Copy link
Member

jlukic commented Mar 13, 2017

Triggering a menu item in IE in this scenario seems to retrigger focus on ui dropdown which because it has a tabindex will trigger focus causing it to re-open.

jlukic added a commit that referenced this issue Mar 13, 2017
@jlukic
Copy link
Member

jlukic commented Mar 13, 2017

I've fixed this in next, bizarrely focus is triggered in IE11 on menu when clicking an item, even though tabindex=-1 on menu.

Then when the menu is hidden my best guess is the display: none; from transition: hidden causes focus on dropdown to occur again. TBH this is some IE11 weirdness that hopefully we wont have to deal with in a few years.

@jlukic jlukic closed this as completed Mar 13, 2017
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

6 participants