Page MenuHomePhabricator

Add inline styles for `{key ...}` in HTML mail
ClosedPublic

Authored by epriestley on Sep 8 2016, 11:39 PM.
Tags
None
Referenced Files
F18855141: D16523.id.diff
Sat, Nov 1, 3:30 AM
F18851993: D16523.diff
Fri, Oct 31, 5:59 AM
F18819863: D16523.diff
Wed, Oct 22, 10:24 AM
F18636976: D16523.diff
Sep 17 2025, 5:34 AM
F18108237: D16523.id.diff
Aug 11 2025, 5:40 AM
F18104021: D16523.diff
Aug 10 2025, 9:17 AM
F17907988: D16523.id39763.diff
Jul 29 2025, 1:30 PM
F17829727: D16523.diff
Jul 26 2025, 10:18 AM
Subscribers
None

Details

Summary

Fixes T11607.

Test Plan
  • Made a comment using {key ...}.
  • Used bin/mail show-outbound --id X --dump-html > test.html to review HTML:

Screen Shot 2016-09-08 at 4.38.06 PM.png (1×1 px, 109 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Add inline styles for `{key ...}` in HTML mail.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 9 2016, 12:08 AM

some day we will be able to write all our CSS in PHP so we don't have to do this :3

This revision was automatically updated to reflect the committed changes.