Repository navigation
fix: padding in Container/Section failing on Klavyio and Outlook - #2257
Conversation
🦋 Changeset detectedLatest commit: 8d3f673 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@bernaferrari is attempting to deploy a commit to the resend Team on Vercel. A member of the Team first needs to authorize it. |
bernaferrari
changed the title
padding + table failing on Klavyio and Outlook
commit: |
61d045a to
cee3eec
Compare
addf436 to
ade5f69
Compare
3ff8597 to
3c4ea0a
Compare
gabrielmfern
changed the title
padding + table failing on Klavyio and Outlook
7da803f to
e9af285
Compare
|
Going to close this due to inactivity, and since it was also working in Outlook, the email client we promise support for. |
|
it is only inactivity because no one reviewed |
you're right, I forgot to actually send the comment that I wrote a while back. feel free to continue with it, sorry |
gabrielmfern
changed the title
|
can you also update the test snapshots? |
|
It should work now. |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
0 issues found across 2 files (changes from recent commits).
Auto-approved: This PR moves padding styles from the outer table to an inner td element in Container and Section components to improve compatibility with Klavyio and Outlook, a straightforward and low-risk change that only affects email rendering with a simple loop splitting style properties.
Re-trigger cubic
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Auto-approved: This PR moves padding styles from the outer table element to the inner td element in Container and Section components to improve email client compatibility, with corresponding snapshot and test updates, and the change is isolated, low-risk, and addresses a known rendering issue.
Re-trigger cubic
|
Hey @bernaferrari thank you for the contribution! we are merging this one now. |
This branch was successfully deployed
3 participants
Closes #2256