Skip to content

Conversation

@betsyecastro
Copy link
Contributor

Changes email template background colors and footer style to match the application current styles.

image

@betsyecastro betsyecastro added the ⬆️ priority:high High priority issue label Apr 14, 2025
@betsyecastro betsyecastro requested a review from wunc April 14, 2025 19:40
@betsyecastro betsyecastro self-assigned this Apr 14, 2025
@wunc wunc marked this pull request as draft April 18, 2025 22:27
@wunc
Copy link
Collaborator

wunc commented Apr 18, 2025

@betsyecastro reverting this PR to draft. I think you're still working on it?

@betsyecastro
Copy link
Contributor Author

@wunc yes, I had not committed the changes that include the stevebauman/purify update because the image source of the template and .eml file and is already referencing the production image. Additionally, the green bar of the header already has a defined height. That said, I just pushed the current changes in case you want to take a look. I’ll continue looking into it as well in the meantime. Thanks!

@betsyecastro
Copy link
Contributor Author

The layout breaking issue seems to occur only in certain Windows environments. Fortunately, I was able to reproduce it on my old Windows laptop and made some style adjustments to resolve the problem. However, the SVG logo is still not rendering — this only happens in preview mode. An option to prevent this, could be implementing some logic to use an PNG image as a back up. We can discuss this further tomorrow.

Preview mode:
preview

Full message view:
image

@betsyecastro
Copy link
Contributor Author

AWS Workspace:
image

@betsyecastro betsyecastro marked this pull request as ready for review April 22, 2025 21:36
@betsyecastro betsyecastro added the ✨ enhancement New feature or request label Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ enhancement New feature or request ⬆️ priority:high High priority issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants