Bug #40809
supprimer le margin au wcs-tracking-code-input form
0%
Description
Suite à #40649 où la cellule tracking-code-input s'est uniformisé avec les autres cellule en gagnant une div pour entourer son contenu, le margin appliqué sur la balise form n'est plus nécessaire car elle s'ajoute au padding de la div.
Fichiers
Historique
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
- Fichier 0001-scss-remove-margin-to-wcs-tracking-code-input-form-4.patch 0001-scss-remove-margin-to-wcs-tracking-code-input-form-4.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a environ 4 ans
Avant #40649, le commit https://git.entrouvert.org/publik-base-theme.git/commit/?id=07aceb8b7aed306d1ae5f5fbc770c3be1844970e
faisait en sorte que le contenu de la cellule ne soit pas collé à la bordure (et là, du coup le contenu est trop éloigné).
Je ne suis pas sûr de moi, mais peut-être vaut-il mieux "reverter" ce commit ?
$ git revert 07aceb8b7aed306d1ae5f5fbc770c3be1844970e
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
Oui sur le fond ça revient exactement au même. Mon patch correspond au revert de ce commit.
Sur la forme, revert ou nouveau commit, je ne sais pas.
Mis à jour par Frédéric Péters il y a environ 4 ans
En gros tu proposes une manière alternative de supprimer le code ? Je dirais que non, c'est mieux d'avoir un commit avec un commentaire explicite, lié à un ticket.
Mis à jour par Nicolas Roche il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
ok
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit d147099e43cb12a4d9164e6ab175af8478459d31 (HEAD -> master, origin/master, origin/HEAD) Author: Thomas JUND <tjund@entrouvert.com> Date: Wed Mar 18 12:02:45 2020 +0100 scss: remove margin to wcs-tracking-code-input form (#40649)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée