search

popcorn-nantes / popcorn-machine

🍿La machine à Popcorn permet est un package NPM qui permet de créer un annuaire de dev freelances "Popcorn", tel que celui de Popcorn Nantes.
https://www.npmjs.com/package/popcorn-machine
MIT License
10 stars 8 forks source link

Images qui ne s'affichent pas quand on dev : la directive v-lazy pour charger les images n'est pas reconnue si on installe popcorn-machine avec npm link #4

Closed yann-yinn closed 5 years ago

yann-yinn commented 5 years ago

c'est le lazy loading des images. J'ai remis un attribut src pour le moment. Il y a écrit que la directive lazy n'est pas "reconnue" alors qu'a priori le plugin est bien chargé dans Vue ... L'impact sur le chargement de la page est trop grand pour que ça ne soit pas bloquant

yann-yinn commented 5 years ago

réessayer une fois https://github.com/popcorn-nantes/popcorn-machine/pull/11 et https://github.com/popcorn-nantes/popcorn-machine/pull/12 mergées, pour une raison que j'ignore, ça semblait refonctionner avec ces PR ...

yann-yinn commented 5 years ago

J'ai remis le v-lazy . Le problème semble n'arriver que dans le cas où npm link est utilisé pour popcorn-machine. Une installation normale de popcorn-nantes via npm i ne pose pas ce souci... Ca dépasse mes connaissances actuelles; est ce que ça vient de la configuration de webpack avec les symlinks, du plugin de vue ? ...

fabienjuif commented 5 years ago

J'ai pas trop d'idée là dessus et j'ai la flemme de creuser vu le cas à la limite.

On clos @yann-yinn ?

yann-yinn commented 5 years ago

J'avoue que j'irai pas creuser non plus, pas la moindre idée du niveau ou ça se joue (gestion des symlinks par la config webpack de Vue?). j'ai créer une issue ici au cas où, car théoriquement c'est censé marcher avec Vue de manière générale : https://github.com/hilongjw/vue-lazyload/issues/340.

Je propose de laisser ouvert pour le moment même si c'est chiant, comme ça ça reste visible par les contributeurs éventuels qui rejoignent le projet, qu'en penses tu ?

fabienjuif commented 5 years ago

Deal

yann-yinn commented 5 years ago

C'est un peu relou pour le dev. Peut être un hack webpack pour remplacer les balises v-lazy par des balises :src pour la compilation lors du dev, histoire d'avoir les images quand même quand on dev ?

yann-yinn commented 5 years ago

Bon ça c'est remis à marcher tout seul sans que je comprenne pourquoi, je clos cette désagréable issue.

fabienjuif commented 5 years ago

xD ça marche