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: select is not defining popover which causes it to not show in production builds #24807

Closed
4 of 6 tasks
HipyCas opened this issue Feb 17, 2022 · 6 comments · Fixed by #24810
Closed
4 of 6 tasks

bug: select is not defining popover which causes it to not show in production builds #24807

HipyCas opened this issue Feb 17, 2022 · 6 comments · Fixed by #24810
Labels
package: core @ionic/core package type: bug a confirmed bug report

Comments

@HipyCas
Copy link

HipyCas commented Feb 17, 2022

Prerequisites

Ionic Framework Version

  • v4.x
  • v5.x
  • v6.x

Current Behavior

The ion-select does not work in android after updating the version of Ionic to 6.0.8. It works on web and displays the current value. Also, no error is risen when interacting with the popover, the item has the ripple effect which means it detects the tap, but the select does no trigger.

Expected Behavior

Simply work as the select should, showing the popover and letting me select

Steps to Reproduce

Create a Vue Ionic project with Ionic v6.0.8. Create a select element with popover style, which in my case renders elements dinamically with a v-for instruction, wrapped inside an ion-item with a label. You should see that it doesn't work on Android, though it does on web.

Code Reproduction URL

https://github.com/biocienciasgrx/ceebi/tree/v0.6.5-alpha (package.json shows a different version, as it is not updated in the repository

Ionic Info

Ionic:

Ionic CLI : 6.18.1 (/home/hipy/.nvm/versions/node/v16.4.1/lib/node_modules/@ionic/cli)
Ionic Framework : @ionic/vue 6.0.8

Capacitor:

Capacitor CLI : 3.3.2
@capacitor/android : 3.3.3
@capacitor/core : 3.3.2
@capacitor/ios : 3.3.3

Utility:

cordova-res (update available: 0.15.4) : 0.15.3
native-run : 1.5.0

System:

NodeJS : v16.4.1 (/home/hipy/.nvm/versions/node/v16.4.1/bin/node)
npm : 7.20.2
OS : Linux 5.15

Additional Information

No response

@ionitron-bot ionitron-bot bot added the triage label Feb 17, 2022
@liamdebeasi liamdebeasi self-assigned this Feb 17, 2022
@liamdebeasi liamdebeasi changed the title bug: bug: select is not defining popover which causes it to not show in production builds Feb 17, 2022
@liamdebeasi liamdebeasi added package: core @ionic/core package type: bug a confirmed bug report labels Feb 17, 2022
@ionitron-bot ionitron-bot bot removed the triage label Feb 17, 2022
@liamdebeasi
Copy link
Contributor

Thanks for the issue. Can you try the following dev build and let me know if it resolves the issue?

npm install @ionic/vue6.0.9-dev.1645109128.222168b @ionic/[email protected]

@wsamoht
Copy link

wsamoht commented Feb 17, 2022

I just came across this issue as well. I tried installing the dev version to test that you linked @liamdebeasi but npm says it is not found.

EDIT: Nevermind, just noticed @ was missing.

npm install @ionic/[email protected] @ionic/[email protected]

@wsamoht
Copy link

wsamoht commented Feb 17, 2022

I've confirmed that the dev version does fix this on Android (emulator) when built for prod.

@liamdebeasi
Copy link
Contributor

Thanks for testing! Also whoops, sorry about the typo 😅

@liamdebeasi
Copy link
Contributor

Thanks for the issue. This has been resolved via #24810, and a fix will be available in an upcoming release of Ionic Framework.

@ionitron-bot
Copy link

ionitron-bot bot commented Mar 20, 2022

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 Mar 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
package: core @ionic/core package type: bug a confirmed bug report
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants