Skip to content

feat: Improve the output of release emails#9218

Merged
isoos merged 5 commits intomasterfrom
fix_email_format
Feb 17, 2026
Merged

feat: Improve the output of release emails#9218
isoos merged 5 commits intomasterfrom
fix_email_format

Conversation

@kevmoo
Copy link
Member

@kevmoo kevmoo commented Feb 13, 2026

Format the changelog excerpt like code

Format the changelog excerpt like code
@kevmoo
Copy link
Member Author

kevmoo commented Feb 13, 2026

I really want this to be better

image

From https://pub.dev/packages/gma_mediation_bidmachine/changelog

@kevmoo kevmoo requested a review from isoos February 13, 2026 17:49
@isoos isoos requested a review from sigurdm February 16, 2026 14:09
@sigurdm
Copy link
Contributor

sigurdm commented Feb 16, 2026

This will render something like this I think:
image
I guess ideally we actually rendered the markdown from the changelog to email-valid html . But I think that is rather hard. So this is preferable to status quo.

String _changeLogDiv(String excerpt) =>
'''
Excerpt of the changelog:<br/>
<div style="background-color: #f6f8fa; border-radius: 6px; padding: 16px; overflow: auto; border: 1px solid #d0d7de; margin: 20px 0;">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It hurts me to see sizes in px. I wonder if we can use font-relative units, such that the default font size of the email client has effect. And if that is worth it.

Excerpt of the changelog:<br/><div style="background-color: #f6f8fa; border-radius: .5rem; padding: 2rem; overflow: auto; border: 1px solid #d0d7de; margin: 3rem 0;">
  <pre style="margin: 0; font-family: 'Courier New', Courier, monospace; font-size: 1rem; line-height: 1.5; color: #24292f;">
Image

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated.

@kevmoo
Copy link
Member Author

kevmoo commented Feb 16, 2026

Can we land this? Who can review?

@isoos isoos merged commit 0efad2b into master Feb 17, 2026
28 checks passed
@isoos isoos deleted the fix_email_format branch February 17, 2026 08:48
@sigurdm
Copy link
Contributor

sigurdm commented Feb 18, 2026

How the mail looks now:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments