Projet

Général

Profil

Bug #27853

Erreur d'interprétation si deux bouton d'action sont sur la même ligne

Ajouté par Laurent Séguin il y a plus de 5 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
08 novembre 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Si dans un courriel je met deux bouton d'action sur la même ligne, ils ne sont pas interprétés.

Ceci fonctionne

{% action_button "foo" label="bouton 1" delay=7 %}
{% action_button "bar" label="bouton 2" delay=7 %}

Ceci plante

{% action_button "foo" label="bouton 1" delay=7 %} {% action_button "bar" label="bouton 2" delay=7 %}


Fichiers

Révisions associées

Révision 18dedba9 (diff)
Ajouté par Thomas Noël il y a plus de 5 ans

allow multiple "action links" by line (#27853)

Historique

#1

Mis à jour par Thomas Noël il y a plus de 5 ans

C'est à cause de

    button_re = re.compile(r'---===BUTTON:(?P<token>[a-zA-Z0-9]*):(?P<label>.*)===---')

dans wcs/qommon/emails.py

#2

Mis à jour par Thomas Noël il y a plus de 5 ans

#3

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Solution proposée à Solution validée

Ack.

#4

Mis à jour par Thomas Noël il y a plus de 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 18dedba96627c3b438673f8b0b230a90a411de73
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Thu Nov 8 12:08:41 2018 +0100

    allow multiple "action links" by line (#27853)

#5

Mis à jour par Laurent Séguin il y a plus de 5 ans

Le patch résous le bug mais pas le comportement attendu.
Intuitivement, si on met les boutons sur la même ligne on attends que leur affichage soit sur la même ligne.

Les boutons sont aujourd'hui codé ainsi dans le mail :

<table>
 <tr>
   <td>
    <a /><span />
   </td>
 <tr>
</table>
<br>
<br>
<table>
 <tr>
   <td>
    <a /><span />
   </td>
 <tr>
</table>

Alors qu'on attend un :
<table>
 <tr>
  <td>
   <a /><span />
  </td>
  <td>
   <a /><span />
  </td>
 </tr>
</table>

#6

Mis à jour par Thomas Noël il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Fermé

Ah ben non effectivement les boutons seront toujours centrés, et designé seuls, via un table. Ca vient des design de https://tedgoas.github.io/Cerberus/

#7

Mis à jour par Thomas Noël il y a plus de 5 ans

et plus exactement https://litmus.com/blog/a-guide-to-bulletproof-buttons-in-email-design (ce sont les docs qu'on a suivies pour que "les mails en html s'affichent bien partout")

#8

Mis à jour par Laurent Séguin il y a plus de 5 ans

Ok, de toute façon c'est du fonctionnel, pas un bug.
On va tracer ceci dans #27887

Formats disponibles : Atom PDF