Skip to content

ion-input etc not taking v-model #3

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

Closed
xewl opened this issue Aug 3, 2021 · 4 comments · Fixed by #4
Closed

ion-input etc not taking v-model #3

xewl opened this issue Aug 3, 2021 · 4 comments · Fixed by #4
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@xewl
Copy link

xewl commented Aug 3, 2021

Seems like the Vue support for Ionic wasn't pulled in?

<ion-input
  v-model="password"
  type="password"
  placeholder="••••••••••"
  required
></ion-input>

Won't update "password" in data.

I can see that there's the ionic.js plugin, but :S

@MexsonFernandes
Copy link
Owner

Thanks for the input @xewl. The support shouldn't be external as Vue is the base and Ionic is integrated. Let me reproduce your error on my end.

@MexsonFernandes MexsonFernandes self-assigned this Aug 4, 2021
@MexsonFernandes MexsonFernandes added bug Something isn't working enhancement New feature or request labels Aug 4, 2021
@xewl
Copy link
Author

xewl commented Aug 4, 2021

Vue has it's own support layer in ionic/vue to give v-model support to these components
But while trying to import from that I've hit some issues with ionic/vue-router giving warnings and the app not working.
I've also forwarded this as a question to compadré @daggerok at daggerok/ionic-nuxt-app#5

You two seem to have the most experience with this setup (Nuxt, Vue, Ionic).

@MexsonFernandes
Copy link
Owner

MexsonFernandes commented Sep 1, 2021

@xewl for now I have added an alternate fix which will handle v-model. I am waiting for Nuxt 3 beta to be rolled out so that @ionic/vue package can be utilized to experiment more with Nuxt + Ionic setup.

Feel free to reopen if issue persists.

@MexsonFernandes MexsonFernandes linked a pull request Sep 1, 2021 that will close this issue
@xewl
Copy link
Author

xewl commented Sep 1, 2021

Thanks for looking into it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants