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] dropdown with multiple selection search clearable class not showing cursor #3050

Open
ghabriel25 opened this issue May 13, 2024 · 2 comments
Assignees
Labels
tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build type/bug Any issue which is a bug or PR which fixes a bug
Milestone

Comments

@ghabriel25
Copy link

ghabriel25 commented May 13, 2024

Bug Report

Dropdown with .multiple.selection.search.clearable class not showing cursor after .error class added to .field class

Steps to reproduce

  1. Create a dropdown with class="ui floating search multiple selection clearable dropdown"
  2. Added .error class to .field class after validation failed from backend
  3. Optional for step 2: just add .error class after .field class within the browser's inspect

Expected result

Show cursor to .multiple.selection.search.clearable dropdown if .error class exists on .field class

Actual result

Cursor not showing

Testcase

https://jsfiddle.net/xuqk09h5/

Screenshot (if possible)

Untitled

Version

2.9.3

@ghabriel25 ghabriel25 added state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged type/bug Any issue which is a bug or PR which fixes a bug labels May 13, 2024
@lubber-de lubber-de self-assigned this May 13, 2024
@lubber-de lubber-de removed state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged labels May 13, 2024
@lubber-de lubber-de added this to the 2.9.4 milestone May 13, 2024
@ghabriel25
Copy link
Author

@lubber-de I think its because .multiple class, every dropdown without .multiple class is normal

@lubber-de
Copy link
Member

Fixed by #3051
See your adjusted jsfiddle here https://jsfiddle.net/lubber/omg7v1aj/41/

@lubber-de lubber-de added the state/has-pr An issue which has a related PR open label May 14, 2024
@lubber-de lubber-de added tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build and removed state/has-pr An issue which has a related PR open labels May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build type/bug Any issue which is a bug or PR which fixes a bug
Projects
None yet
Development

No branches or pull requests

2 participants