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] Fix IE11 submenu selection triggered menu opening #189

Merged
merged 1 commit into from
Oct 24, 2018

Conversation

lubber-de
Copy link
Member

@lubber-de lubber-de commented Oct 22, 2018

Description

If a submenu entry was selected from a dropdown, the Element got focus on IE11 and reopened the dropdown again. This was basically already fixed in case the dropdown had a searchmenu, but now it works anytime.

Testcase

https://jsbin.com/yofawovahi/edit?html,js,output

Screenshot (with applied fix)

ie11_submenu_focus_fix

Closes

Semantic-Org/Semantic-UI#6565
Semantic-Org/Semantic-UI#5965

@lubber-de lubber-de added type/bug Any issue which is a bug or PR which fixes a bug lang/javascript Anything involving JavaScript state/awaiting-reviews Pull requests which are waiting for reviews labels Oct 22, 2018
@lubber-de lubber-de changed the title [Dropdown ]Fix IE11 submenu selection triggered menu opening [Dropdown] Fix IE11 submenu selection triggered menu opening Oct 22, 2018
Copy link
Member

@y0hami y0hami left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@ColinFrick ColinFrick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@y0hami y0hami removed the state/awaiting-reviews Pull requests which are waiting for reviews label Oct 24, 2018
@y0hami y0hami merged commit 63bc9a8 into fomantic:beta Oct 24, 2018
@lubber-de lubber-de deleted the fix/6565/IE11_submenu_dropdown branch October 24, 2018 07:33
@lubber-de lubber-de added the tag/sui-issue Taken from an existing Issue/PR of SUI label Oct 30, 2018
@lubber-de lubber-de added this to the 2.6.4 milestone Nov 5, 2018
lubber-de added a commit to lubber-de/Fomantic-UI that referenced this pull request Nov 10, 2018
lubber-de added a commit to lubber-de/Fomantic-UI that referenced this pull request Nov 10, 2018
y0hami pushed a commit that referenced this pull request Nov 12, 2018
@y0hami y0hami mentioned this pull request Nov 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/javascript Anything involving JavaScript tag/sui-issue Taken from an existing Issue/PR of SUI type/bug Any issue which is a bug or PR which fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants