Resolver estilos finales en notificaciones por correo electrónico

[thoka (Thomas Kalka)]

Al menos el cliente de correo electrónico web de posteo.de renderiza los colores con el primer valor dado en una definición de estilo CSS, lo que da como resultado botones ilegibles en las notificaciones:

en lugar de

Actualmente, el estilo correspondiente es

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;

Si se cambia para incluir solo las definiciones más recientes

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;

se obtiene el comportamiento deseado.

Dado que no hay ningún beneficio en enviar estilos sobrescritos posteriormente, los problemas de renderizado con visores de correo electrónico defectuosos podrían resolverse enviando solo las definiciones CSS finales.

[thoka (Thomas Kalka)]

Hay una implementación en

feature: Resolve final styles in email notifications

[zogstrip]

Si te animas a hacer una pull request, la fusionaré felizmente

No resuelve este problema, pero tener estilos duplicados es un desperdicio y propenso a errores de todos modos.

Haría algunos pequeños ajustes a la función deduplicate_style, principalmente para asegurarme de que tanto la key como los values estén presentes.

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]

Gracias, acabo de fusionarlo