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

fix: #13560 || DropDown clear or close icon on click dropDown option show #13564

Conversation

ashikjs
Copy link
Contributor

@ashikjs ashikjs commented Aug 29, 2023

Addressing issue || #13560

After solution video for details and test result

Video for details

Code explain

Previously we was checking ONLY

DomHandler.hasClass(event.target, 'p-dropdown-clear-icon')

this case event don't get all the time same element, some time it's also get path from svg element or others .

So I also check closest class has '.p-dropdown-clear-icon'

@vercel
Copy link

vercel bot commented Aug 29, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
primeng-ssr-test ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 29, 2023 11:35am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
primeng ⬜️ Ignored (Inspect) Visit Preview Aug 29, 2023 11:35am

@ashikjs ashikjs force-pushed the issues-13560/dropdown-clear-selection-icon-open-options-modal branch 2 times, most recently from 4917b55 to 76ccfea Compare August 29, 2023 11:24
@ashikjs ashikjs marked this pull request as draft August 29, 2023 11:25
@ashikjs ashikjs force-pushed the issues-13560/dropdown-clear-selection-icon-open-options-modal branch from 76ccfea to 927e25d Compare August 29, 2023 11:27
@ashikjs ashikjs marked this pull request as ready for review August 29, 2023 11:29
@ashikjs
Copy link
Contributor Author

ashikjs commented Aug 29, 2023

@SoyDiego and @gucal hi, can you please check this PR. Thanks advance

@ashikjs ashikjs changed the title fix: #13560 || DropDown clear or close icon on click dropDown options… fix: #13560 || DropDown clear or close icon on click dropDown option Aug 29, 2023
@ashikjs ashikjs changed the title fix: #13560 || DropDown clear or close icon on click dropDown option fix: #13560 || DropDown clear or close icon on click dropDown option show Aug 29, 2023
@vercel vercel bot temporarily deployed to Preview – primeng-ssr-test August 29, 2023 11:35 Inactive
@gucal gucal self-requested a review August 29, 2023 12:00
@gucal gucal added the Status: Pending Review Issue or pull request is being reviewed by Core Team label Aug 29, 2023
@mertsincan mertsincan merged commit e8859c8 into primefaces:master Aug 29, 2023
@mertsincan
Copy link
Member

Thanks a lot for your contribution!

Best Regards,

@gucal gucal removed the Status: Pending Review Issue or pull request is being reviewed by Core Team label Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DropDown clear or close icon on click dropDown options open randomly
3 participants