Skip to content

Outlook-safe rendering for visual email builder output#2978

Open
MrJoky wants to merge 4 commits intoknadh:masterfrom
MrJoky:outlook-type-safe
Open

Outlook-safe rendering for visual email builder output#2978
MrJoky wants to merge 4 commits intoknadh:masterfrom
MrJoky:outlook-type-safe

Conversation

@MrJoky
Copy link
Copy Markdown

@MrJoky MrJoky commented Mar 30, 2026

This PR improves Outlook compatibility for emails created with the visual email builder.

It adds Outlook-oriented post-processing to the visual builder HTML output so that buttons, images, text blocks, and
spacers render more reliably in Outlook. It also fixes format switching in the campaign editor so that switching
between visual and raw/html does not discard the visual builder source for unchanged content.

References: #2576

Not sure if this issue addresses the same problem #1231

Timo Dahlenburg added 2 commits March 30, 2026 14:58
Keep the last visual builder source/body snapshot when switching between visual and raw formats so unchanged visual content can round-trip without losing button styling or other builder-specific structure.
Move the visual email post-processing into a dedicated outlook module, add Outlook-safe wrappers for buttons, images, text blocks, and spacers, and add a local ESLint config for the email-builder TypeScript subproject so its TS/TSX files are parsed with the correct parser.
@knadh
Copy link
Copy Markdown
Owner

knadh commented Mar 31, 2026

Thanks @MrJoky. This is a big PR, which is going to be very difficult for me to test and verify. Would you be able to share screenshots+HTML of e-mails post-processed via this rendering in Outlook and non-Outlook clients?

@MrJoky
Copy link
Copy Markdown
Author

MrJoky commented Mar 31, 2026

@knadh Different clients.

@knadh knadh self-assigned this Mar 31, 2026
@MrJoky
Copy link
Copy Markdown
Author

MrJoky commented Apr 13, 2026

@knadh how far are you with the review?

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.

2 participants