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] clearable is not workin with on: 'hover' option #6594

Closed
mthoretton opened this issue Sep 21, 2018 · 5 comments
Closed

[Dropdown] clearable is not workin with on: 'hover' option #6594

mthoretton opened this issue Sep 21, 2018 · 5 comments

Comments

@mthoretton
Copy link

Steps

$('.ui.dropdown').dropdown({transition: 'drop', on: 'hover', clearable: true,});

Expected Result

I'd expect I can click the x to clear the selection.

Actual Result

The x cannot be clicked.

Version

2.4.0

Testcase

https://jsfiddle.net/aeou6q5m/

@mthoretton
Copy link
Author

mthoretton commented Sep 21, 2018

Also, even if I didn't manage to reproduce in the jsfiddle, the dropdown items are blurry when I open it (only with clearable). They become normal when I move the pointer over the list.

screen shot 2018-09-21 at 15 08 19
=> when I move the pointer over it gets back to normal
screen shot 2018-09-21 at 15 08 33

@jlukic
Copy link
Member

jlukic commented Sep 24, 2018

Although from a UX perspective I don't think its a great idea to use hover with a clickable clear icon, it is still a bug. I'll fix.

@mthoretton
Copy link
Author

Thanks for looking at it! And yes, even if it's fixed I'll probably keep "non-hover" dropdowns with clickable

y0hami pushed a commit to fomantic/Fomantic-UI that referenced this issue Oct 15, 2018
attach 'click' event to clear icon independent of the on option

closes #171 / Semantic-Org/Semantic-UI#6617 / Semantic-Org/Semantic-UI#6594
y0hami pushed a commit to fomantic/Fomantic-UI that referenced this issue Oct 15, 2018
attach 'click' event to clear icon independent of the on option

closes #171 / Semantic-Org/Semantic-UI#6617 / Semantic-Org/Semantic-UI#6594
@jlukic
Copy link
Member

jlukic commented Oct 21, 2018

This was fixed in 2.4.1

@jlukic jlukic modified the milestones: 2.4.2, 2.4.1 Oct 21, 2018
@jlukic jlukic closed this as completed Oct 21, 2018
@lubber-de

This comment was marked as spam.

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

3 participants