Summary email colors change recently?

Continuing the discussion from Summary Email feedback:

Sometime in the past few weeks, the background behind the logo in the summary emails changed, which makes things look odd in our case.

Ours used to be this:

<table style=3D"background-color:#059a91;width:100%;">
  <tr>
    <td align=3D"center" style=3D"text-align: center;padding: 20px 0; font-=
family:Helvetica,Arial,sans-serif;">

  <a href=3D"https://www.myforum.com" style=3D"text-decoration: none; font-=
weight: bold; color: #059a91;; color:#B5FAF2;font-size:22px;text-decoration=
:none;">

but now it’s this:

<table style=3D"background-color:#ffffff;width:100%;">
  <tr>
    <td align=3D"center" style=3D"text-align: center;padding: 20px 0; font-=
family:Helvetica,Arial,sans-serif;">

  <a href=3D"https://www.myforum.com" style=3D"text-decoration: none; font-=
weight: bold; color: #059a91;; color:#333333;font-size:22px;text-decoration=
:none;">

I haven’t been able to track down where the change occurred, unfortunately, and I’m not sure whether it’s possible to change any existing settings to get things looking the way they used to…

Aside: it may be helpful to include in the Admin logs when software was updated and the before/after commit hashes

Was this change intentional?

Is there a way I can change things to look the way they used to?

3 Me gusta

Does anyone know the answer to either of these questions?

Wondering if we need to just change our digest to use a logo that works on a white background…

Supongo que es un Contribute > Bug relacionado con el tema que se indica a continuación:

En ambos errores, se obtiene el valor del color a partir del código ColorScheme.hex_for_name('header_background').

4 Me gusta

It wasn’t intentional, and was probably a side effect of the new theme support.

4 Me gusta

I fixed this bug today.

https://github.com/discourse/discourse/commit/389a0e1a1cf4069703a226d842fd9524a54042bd

7 Me gusta