mirror of
https://github.com/healthchecks/healthchecks.git
synced 2026-04-25 15:05:49 +03:00
[PR #488] [MERGED] Added formatting on ping.body in emails #973
Labels
No labels
bug
bug
bug
feature
good-first-issue
new integration
pull-request
question
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/healthchecks#973
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/healthchecks/healthchecks/pull/488
Author: @cocide
Created: 3/5/2021
Status: ✅ Merged
Merged: 3/8/2021
Merged by: @cuu508
Base:
master← Head:patch-2📝 Commits (1)
e0e3b6fAdded formatting on ping.body in emails📊 Changes
1 file changed (+1 additions, -1 deletions)
View changed files
📝
templates/emails/alert-body-html.html(+1 -1)📄 Description
Not all email clients are formatting the
ping.bodycontents uniformly. Even using different applications from the same email provider results in a different display of theping.bodycontents. There are two basic issues:<pre>tags. Using fixed-width font is listed in the definition on https://www.w3schools.com/tags/tag_pre.asp9fd9c8e4efbut is not part of the definition of the<pre>tag. I'd like to add this to the emails to make Healthchecks more uniform between the website and the email notification.Gmail Webmail:
Gmail Android App:
ProtonMail Webmail:
ProtonMail Android:
The testing I performed is not extensive, but it does show how multiple clients are displaying the contents differently. To make the display of the
ping.bodymore uniform I'd like to add a bit of formatting information to the<pre>tag.Gmail Webmail Screenshot:


Gmail Android Screenshot:
ProtonMail Webmail Screenshot:


ProtonMail Android Screenshot:
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.