-
-
Notifications
You must be signed in to change notification settings - Fork 739
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] Fix VsButton throwing error on $nextTick #921
base: master
Are you sure you want to change the base?
[Bug] Fix VsButton throwing error on $nextTick #921
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FUNCIONA OK
@clopezpro Yo tengo el mismo problema. En mi caso el error aparece cuando presiono un botón dentro de un cuadro de diálogo, por ejemplo confirmación. En el sitio de la documentación está ocurriendo lo mismo https://lusaxweb.github.io/vuesax/components/dialog.html#alert Que podría hacer para resolverlo? Gracias |
Hello, same error here, when will this PR get merged ? thanks. |
This repository is unmaintained for a while. You can use this fork with some fixes (like this one mencioned): https://github.com/Brugarolas/vuesax Este repositorio lleva bastante tiempo sin mantenerse. Podéis usar este Fork que incluye algunas correcciones como la mencionada en esta issue: https://github.com/Brugarolas/vuesax |
@CarlosCondor Muchas gracias por tu aporte! |
@tepajc Si te sirve de algo, nosotros hemos optado por dejar de utilizar esta librería ya que no tiene ningún tipo de garantías y no es apta para producción, y estamos en proceso de eliminar las dependencias actuales a esta librería. Personalmente te recomendaría dejar de usarla si aún estas a tiempo, antes de acoplarte más a ella y que te sea más dificil. En su lugar puedes utilizar frameworks más sólidos como Boostrap, Vuetify o por el estilo. |
@CarlosCondor |
Soy nuevo, como puedo usar su Fork ? uso npm install vuesax3 |
Como podria usar https://github.com/Brugarolas/vuesax en mi aplicacion vue, usando npm? |
Lo podrías instalar con |
Buenas @clopezpro, el fork lo puedes instalar con Luego puedes seguir las instrucciones de uso de Vuesax (aún no las he actualizado en el fork) pero cambiando los Algo así, vaya (ejemplo modificado): import Vue from 'vue'
import Vuesax from '@brugarolas/vuesax'
import '@brugarolas/vuesax/dist/vuesax.css'
Vue.use(Vuesax) El fork contiene solo arreglos así que es compatible con la librería original, la lista de cambios la puedes encontrar aquí (de momento estamos usando la rama Si encuentras algún error más no dudes en ponerte en contacto conmigo y lo resolveré cuando pueda (ya tengo localizado alguno, pero no he tenido tiempo de arreglarlo todavía). Espero haberte servido de ayuda. ¡Un saludo! |
thanks |
Bug: Sometimes when we click a VsButton it throws the next error:
Console error: Error in nextTick: “TypeError: Cannot read property ‘clientWidth’ of undefine”
. That is caused because click function executes athis.$nextTick()
and VsButton may not exist in next tick (eg: because of transitions between routes in SPAs or when closing a VsDialog).It is reproducible in Vuesax documentation: https://lusaxweb.github.io/vuesax/components/dialog.html. Open and then close Dialog and you should see the error on the console.