-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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: popover select does not work in @ionic/[email protected]+ #26695
Comments
FYI, I did some testing with earlier versions and the bug appeared in 6.4. |
Thanks! Here is a dev build with a proposed fix if you are interested in testing: |
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. |
Prerequisites
Ionic Framework Version
Current Behavior
When I click on a select that is configured as a popover, there is a runtime error and the popover does not appear.
Error:
Expected Behavior
There should be no error.
Steps to Reproduce
Open the stackblitz and click on the popover. Then open the console and view the error.
Code Reproduction URL
https://stackblitz.com/edit/angular-ysm5ef
Ionic Info
Ionic:
Ionic CLI : 6.20.8 (@Ionic[email protected]/node_modules/@ionic/cli)
Ionic Framework : @ionic/vue 6.5.1 (@Ionic[email protected]/node_modules/@ionic/vue)
Capacitor:
Capacitor CLI : 4.6.2
@capacitor/android : 4.6.2 (@capacitor+android@4.6.2_@capacitor[email protected]/node_modules/@capacitor/android)
@capacitor/core : 4.6.2 (@capacitor[email protected]/node_modules/@capacitor/core)
@capacitor/ios : 4.6.2 (@capacitor+ios@4.6.2_@capacitor[email protected]/node_modules/@capacitor/ios)
Utility:
cordova-res : 0.15.4
native-run : not installed globally
System:
NodeJS : v19.5.0
npm : 9.3.1
OS : macOS Monterey
Additional Information
I'm seeing this in both Brave and Firefox, M1 MBP.
The cause:
popover.js#present
callsattachComponent
, passing the string 'ion-select-popover' as the component argument.attachComponent
callsattachViewToDom
, which tries to use that string as the key to a weak map, which doesn't work.The text was updated successfully, but these errors were encountered: