Page MenuHomePhabricator

Use a single newline between HTML mail sections
ClosedPublic

Authored by epriestley on Aug 25 2014, 2:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 15, 11:24 PM
Unknown Object (File)
Sat, Jun 14, 5:36 PM
Unknown Object (File)
May 19 2025, 8:24 AM
Unknown Object (File)
May 10 2025, 4:22 AM
Unknown Object (File)
Apr 9 2025, 12:28 PM
Unknown Object (File)
Apr 8 2025, 10:52 PM
Unknown Object (File)
Apr 8 2025, 3:28 PM
Unknown Object (File)
Apr 7 2025, 4:09 AM
Subscribers

Details

Summary

Ref T992. This makes HTML mail layout more consistent with text mail layout and fixes my greatest annoyance with it.

Test Plan

Used bin/mail list-outbound --id <id> --dump-html to view mail in Safari, saw it have a normal amount of whitespace between sections.

Diff Detail

Repository
rP Phabricator
Branch
twobr
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 2339
Build 2343: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Use a single newline between HTML mail sections.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, btrahan, talshiri.
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 25 2014, 3:47 PM
epriestley updated this revision to Diff 24906.

Closed by commit rP7e655da977c0 (authored by @epriestley).