search

MaVoix / groupes-locaux

Un formulaire permettant aux groupes locaux #MAVOIX de formaliser leur engagement à faire campagne.
The Unlicense
4 stars 1 forks source link

Les pages de groupe avec "objectif atteint" ne montrent pas le montant "promis" #49

Closed ofa- closed 6 years ago

ofa- commented 7 years ago

cf https://collectifs-locaux.mavoix.info/groupe/page.html?id=19

image

ofa- commented 7 years ago

a priori, c'est l'image générée qui "tronque" le texte : https://collecte.mavoix.info/smallpic/dons/19/tirelire.png

oceanBigOne commented 7 years ago

Problème réglé en hotfix (j'ai mis à jour de Readme, si tout est bon il n'y a plus qu'à Finish le hotfix)

En fait le problème venait de la taille de la jauge sur l'image. Dans mes calculs je lui donnais la même largeur que l'image alors qu'on a une marge à droite et à gauche. Le texte promesse se retrouvait à déborder sur la droite quand la jauge était pleine pour les même raison.

Comme pour le hotFix précédent j'ai du tester avec des valeurs en dur dans le script, c’est donc pas mal de le tester avant de finish si vous pouvez

oceanBigOne commented 7 years ago

@ColinMaudry je crois que j'ai toujours pas compris le fonctionnement des Pull Request, ... je veux bien un éclaircissement sur ce point à l'occaz ;)

ColinMaudry commented 7 years ago

Merci de ne pas fermer l'issue tant que bug n'est pas résolu en production. Rien ne nous assure que les mesures prises vont bel et bien le corriger :)