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
F19507520: D15855.diff
Fri, Jan 9, 9:12 PM
F19385896: D15855.id38195.diff
Sat, Dec 27, 2:46 AM
F19316842: D15855.diff
Thu, Dec 25, 5:11 AM
F19092575: D15855.id38195.diff
Dec 3 2025, 5:50 PM
F19092334: D15855.diff
Dec 3 2025, 5:12 PM
F18903214: D15855.diff
Nov 8 2025, 4:26 AM
F18852322: D15855.id38195.diff
Oct 31 2025, 7:59 AM
F18763731: D15855.diff
Oct 7 2025, 1:56 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