Summary email colors change recently?

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 Likes
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

I guess it is a bug related to the below topic

In both bugs those getting the color value from the code ColorScheme.hex_for_name('header_background')

4 Likes
4

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

4 Likes
6

I fixed this bug today.

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

7 Likes