Page MenuHomePhabricator

Add addLinkSection to MailBody to properly format URIs
ClosedPublic

Authored by chad on Oct 30 2014, 5:33 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 28, 9:27 AM
Unknown Object (File)
Fri, Mar 22, 2:08 PM
Unknown Object (File)
Fri, Mar 22, 1:44 AM
Unknown Object (File)
Sun, Mar 10, 10:56 PM
Unknown Object (File)
Feb 11 2024, 3:43 PM
Unknown Object (File)
Feb 3 2024, 1:13 AM
Unknown Object (File)
Jan 31 2024, 3:15 AM
Unknown Object (File)
Jan 12 2024, 4:48 PM
Tokens
"Like" token, awarded by hashar.

Details

Summary

Fixes T6343. Grepped for all callsites and added addLinkSection where needed.

Test Plan

Tested Differential, Maniphest, Conpherence, Ponder and Macro. Inspect HTML mail for anchor tags. Inspect text mails for non-disruption.

Diff Detail

Repository
rP Phabricator
Branch
mail-html
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 2912
Build 2916: [Placeholder Plan] Wait for 30 Seconds

Unit TestsFailed

TimeTest
0 mstestMercurialParser
0 mstestBodyRender
0 mstestBodyRenderNoHerald
0 mstestBodyRenderNoReply
0 mstestGitParser
View Full Test Results (1 Failed · 9 Passed)

Event Timeline

chad retitled this revision from to Add addLinkSection to MailBody to properly format URIs.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
src/applications/metamta/view/PhabricatorMetaMTAMailBody.php
89

should I wrap this in another div?

btrahan edited edge metadata.

Nice!

src/applications/metamta/view/PhabricatorMetaMTAMailBody.php
89

idk... want to give something like this a whirl?

https://litmus.com/email-testing

theoretically each and every html email client may render it differently. I also feel fine just letting users report rendering issues as they come up.

107–109

this site will need updating too

This revision is now accepted and ready to land.Oct 30 2014, 5:50 PM

Looks good, although it won't fix inline links (in comments and such)

Tried Litmus, things looked good. It might be worth signing up.

This revision was automatically updated to reflect the committed changes.