Résoudre les styles finaux dans les notifications par email

[thoka (Thomas Kalka)]

Au moins, le client de messagerie Web de posteo.de affiche les couleurs avec la première valeur donnée dans une définition de style CSS, ce qui donne des boutons illisibles dans les notifications :

au lieu de

Actuellement, le style correspondant est

font-weight: normal;; text-decoration: none; font-weight: bold; color: #006699;; background-color: #2F70AC; color: #FFFFFF; border-top: 4px solid #2F70AC; border-right: 6px solid #2F70AC; border-bottom: 4px solid #2F70AC; border-left: 6px solid #2F70AC; display: inline-block; font-weight: bold;

S’il est modifié pour inclure uniquement les dernières définitions

font-weight: normal; text-decoration: none; font-weight: bold; background-color: #2F70AC; color: #FFFFFF; border-top: 4px solid #2F70AC; border-right: 6px solid #2F70AC; border-bottom: 4px solid #2F70AC; border-left: 6px solid #2F70AC; display: inline-block; font-weight: bold;

on obtient le comportement souhaité.

Comme il n’y a aucun avantage à envoyer des styles ultérieurement écrasés, les problèmes d’affichage avec des visualiseurs d’e-mails bogués pourraient être résolus en n’envoyant que les définitions CSS finales.

[thoka (Thomas Kalka)]

Il y a une implémentation dans

feature: Resolve final styles in email notifications

[zogstrip]

Si vous avez envie de faire une pull request, je serai heureux de la fusionner

Cela ne résout pas ce problème mais avoir des styles dupliqués est inutile et sujet aux erreurs de toute façon.

Je ferais quelques légères modifications à la fonction deduplicate_style, principalement pour m’assurer que la key et les values sont présentes.

def deduplicate_style(style)
  styles = {}

  style
    .split(";")
    .select(&:present?)
    .map { _1.split(":", 2).map(&:strip) }
    .each { |k, v| styles[k] = v if k.present? && v.present? }

  styles.map { |k, v| "#{k}:#{v}" }.join(";")
end

[thoka (Thomas Kalka)]

FIX: deduplicate css in mails

[zogstrip]

Merci, je viens de le fusionner