feat: ionCheck and ionUncheck for ion-checkbox#19097
Conversation
|
Thank you for the PR! The Here's a Codepen example: https://codepen.io/brandyscarney/pen/ExYKaaX?editors=1011 |
|
Hey @brandyscarney, |
|
@brandyscarney I agree, it makes sense to emit the checked and unchecked events. It's easy to do in this place, it's harder to handle the case of #19096 in business logic. |
|
@brandyscarney just wondering if there are any updates on this? |
|
Thanks for the PR! We appreciate the work you've put in. In the time that's passed, we've slated some changes for the upcoming Ionic 7 release that will solve the problems this PR is targeting. Namely, the Here is the RFC for this change in general: #25532 As such, I'm going to go ahead and close this PR out. Thank you! |
Pull request checklist
Please check if your PR fulfills the following requirements:
npm run build) was run locally and any changes were pushednpm run lint) has passed locally and any fixes were made for failuresPull request type
Please check the type of change your PR introduces:
What is the current behavior?
Issue Number: #19096
What is the new behavior?
Does this introduce a breaking change?
Other information
--