Summary email colors change recently?

Contribuer Bug
1

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 « J'aime »
2

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…

3

Je suppose que c’est un Contribute > Bug lié au sujet ci-dessous

Dans les deux bogues, ceux qui obtiennent la valeur de la couleur à partir du code ColorScheme.hex_for_name('header_background')

4 « J'aime »
4

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

4 « J'aime »
6

I fixed this bug today.

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

7 « J'aime »