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

bug: checkbox not working inside Swiper with allow-touch-move="false" #27958

Closed
3 tasks done
BugProg opened this issue Aug 9, 2023 · 3 comments
Closed
3 tasks done

bug: checkbox not working inside Swiper with allow-touch-move="false" #27958

BugProg opened this issue Aug 9, 2023 · 3 comments
Labels

Comments

@BugProg
Copy link

BugProg commented Aug 9, 2023

Prerequisites

Ionic Framework Version

v7.x

Current Behavior

Sometimes the click is not taken into account.

signal-2023-08-09-144539.mp4

Expected Behavior

All the checkbox should be checked.

Steps to Reproduce

  1. Init Swiper in Angular and Ionic project.
  2. Add allow-touch-move="false" option.
  3. Add list of ion-checkbox.

Code Reproduction URL

https://github.com/BugProg/-ion-checkbox

Ionic Info

Ionic:

Ionic CLI : 7.1.1 (/usr/local/lib/node_modules/@ionic/cli)
Ionic Framework : @ionic/angular 7.2.2
@angular-devkit/build-angular : 16.1.8
@angular-devkit/schematics : 16.1.8
@angular/cli : 16.1.8
@ionic/angular-toolkit : 9.0.0

Capacitor:

Capacitor CLI : 5.2.2
@capacitor/android : 5.2.2
@capacitor/core : 5.2.2
@capacitor/ios : not installed

Utility:

cordova-res : 0.15.4
native-run : 1.7.2

System:

NodeJS : v18.16.1 (/usr/bin/node-18)
npm : 9.5.1
OS : Linux 6.4

Additional Information

No response

@ionitron-bot ionitron-bot bot added the triage label Aug 9, 2023
@liamdebeasi liamdebeasi self-assigned this Aug 9, 2023
@liamdebeasi
Copy link
Contributor

Thanks for the issue. I am going to close this as this is not a bug in Ionic Framework. The problem here is Swiper is calling event.preventDefault() which blocks the default browser behavior from occurring. When this happens on ion-checkbox this prevents the native checkbox inside of ion-checkbox from toggling. We do have plans to change this so ion-checkbox listens for a click event instead of the native checkbox's change event. This work is being tracked in #27169.

@liamdebeasi liamdebeasi closed this as not planned Won't fix, can't repro, duplicate, stale Aug 9, 2023
@liamdebeasi liamdebeasi removed their assignment Aug 9, 2023
@BugProg
Copy link
Author

BugProg commented Aug 9, 2023

Thank you for your answer. 😄
I did not see that.

@ionitron-bot
Copy link

ionitron-bot bot commented Sep 8, 2023

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Sep 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants