Page MenuHomePhabricator
Differential D15855

Produce a more defensible HTML mail rendering of Remarkup blockquotes
ClosedPublic
Actions

Authored by epriestley on May 5 2016, 7:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 22, 6:15 AM
Unknown Object (File)
Mon, Nov 18, 10:19 AM
Unknown Object (File)
Thu, Nov 14, 4:31 AM
Unknown Object (File)
Sun, Nov 10, 6:11 AM
Unknown Object (File)
Wed, Nov 6, 12:37 AM
Unknown Object (File)
Tue, Nov 5, 10:39 AM
Unknown Object (File)
Oct 22 2024, 7:10 PM
Unknown Object (File)
Oct 15 2024, 6:30 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T7031: Blockquote formatting in HTML mail could be improved
T10694: Improve behavior of context for inline comments in mail
Commits
rPHUd4e36c61b1ac: Produce a more defensible HTML mail rendering of Remarkup blockquotes
Summary

Ref T10694. Fixes T7031. We don't currently put any inline styles on <blockquote> tags produced by prefixing blocks with > in Remarkup.

Add some styles.

Test Plan

Generated some reasonable mail with complex blockquote elements:

Screen Shot 2016-05-05 at 12.26.34 PM.png (1×1 px, 249 KB)

Diff Detail

Repository
rPHU libphutil
Branch
blockquote1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12070
Build 15206: Run Core Tests
Build 15205: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 38195.May 5 2016, 7:28 PM
epriestley retitled this revision from to Produce a more defensible HTML mail rendering of Remarkup blockquotes.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added tasks: T10694: Improve behavior of context for inline comments in mail, T7031: Blockquote formatting in HTML mail could be improved.
chad accepted this revision.May 5 2016, 7:29 PM
chad edited edge metadata.
This revision is now accepted and ready to land.May 5 2016, 7:29 PM
Closed by commit rPHUd4e36c61b1ac: Produce a more defensible HTML mail rendering of Remarkup blockquotes (authored by epriestley, committed by epriestley). · Explain WhyMay 5 2016, 7:34 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
markup/
engine/
remarkup/
blockrule/
16 lines

Diff 38195

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupQuotesBlockRule.php

Loading...
Phabricator · Built by Phacility