search

davidlhoumaud / plxMyShop

plugin ECommerce pour le CMS PluXML
GNU General Public License v2.0
1 stars 4 forks source link

Ajout d'un template static #17

Closed ppmt closed 8 years ago

ppmt commented 8 years ago

Ca permet d'afficher tous les produits classe par categories

pour utiliser: copier ce fichier dans votre theme en faisant attention a ne pas utiliser de majuscule dans le nom du fichier. Le template devient alors disponible pour etre utilise dans vos pages statiques

mathieu269 commented 8 years ago

ah oui bien vu le format du nom de fichier sans majuscules. j'ai changé les noms des 2 exemples.

par contre le template que tu proposes ressemble à "static-boutique-tous-les-produits", tu penses que ça peut apporter quelque chose de rajouter le tien en plus ?

ppmt commented 8 years ago

hum!!! pour être honnête je n'avais même pas vérifier ce ficher. J'ai juste essayer celui qui liste les catégories et j'ai assume que celui listerai les produits dans une liste.

Donc en effet ils sont identiques. Sauf que le tiens a plus de code. A quoi sert le code javascript ?

Le tiens est peut-être plus complet que le mien car il va aussi afficher l'image de la catégories si elle a en une.

Donc au final c'est toi qui voit :) je ne serais pas vexer si tu décides de ne pas faire le merge.

mathieu269 commented 8 years ago

bonne question pour le code javascript, je crois que ça sert dans le panier mais c'est aussi possible qu'il ne sert plus à rien maintenant que l'ajout dans le panier n'utilise plus ajax. je ne pense que ça soit utile de vérifier cela maintenant, on étudiera ça plus en détail si quelqu'un l'utilise et trouve un problème

en ce qui concerne l'exemple de page statique "static-boutique-tous-les-produits", je crois que je l'ai fait en m'inspirant de la page d'accueil de ton site donc il y a déjà un peu de ton esprit dans les exemples de template alors que tu le ne savais même pas jusqu'à aujourd'hui :wavy_dash: (je n'ai trouvé que ça comme icône en cherchant "transmission de pensé") :sunglasses:

ppmt commented 8 years ago

un autre cas de "Les grands Esprits se rencontrent"

Donc je pense que tu peux annuler ce merge. Je vais utiliser le tiens et le simplifier en enlevant le javascript pour commencer. Je te tiendrais au courant.