Page MenuHomePhabricator
Differential D15847

When rendering Remarkup codeblocks, allow calling engines to provide a style map
ClosedPublic
Actions

Authored by epriestley on May 5 2016, 1:39 AM.
Tags
None
Referenced Files
F19509878: D15847.id38175.diff
Sun, Jan 11, 2:21 AM
F19500959: D15847.id38184.diff
Thu, Jan 8, 2:52 PM
F19386088: D15847.id38177.diff
Sat, Dec 27, 2:54 AM
F19385894: D15847.id38175.diff
Sat, Dec 27, 2:46 AM
F19005795: D15847.diff
Nov 21 2025, 4:51 PM
F18807763: D15847.id38175.diff
Oct 19 2025, 3:26 AM
F18798360: D15847.diff
Oct 17 2025, 6:22 AM
F18767692: D15847.id38177.diff
Oct 7 2025, 11:51 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T9790: HTML email doesn't syntax highlight code blocks, inline diffs, or diff context above inline comments
Commits
rPHUe499b37f5dd3: When rendering Remarkup codeblocks, allow calling engines to provide a style map
Summary

Ref T9790.

  • Make the HTML version of code blocks look more like the web version.
  • If the caller passes a style map, reformat the body to use style="..." instead of class="..." for HTML mail mode.
Test Plan

Faked a style map and got some inline styles (red stuff) -- see also the "style" in the inspector:

Screen Shot 2016-05-04 at 6.37.26 PM.png (970×754 px, 228 KB)

Diff Detail

Repository
rPHU libphutil
Branch
syntax3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12051
Build 15176: Run Core Tests
Build 15175: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 38175.May 5 2016, 1:39 AM
epriestley retitled this revision from to When rendering Remarkup codeblocks, allow calling engines to provide a style map.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T9790: HTML email doesn't syntax highlight code blocks, inline diffs, or diff context above inline comments.
chad accepted this revision.May 5 2016, 1:47 AM
chad edited edge metadata.
chad added inline comments.
src/markup/engine/remarkup/blockrule/PhutilRemarkupCodeBlockRule.php
146

(y)

This revision is now accepted and ready to land.May 5 2016, 1:47 AM
epriestley updated this revision to Diff 38177.May 5 2016, 2:02 AM
epriestley edited edge metadata.
  • Synchronize more styles better: counterexample, headers, margins, font sizing.
Closed by commit rPHUe499b37f5dd3: When rendering Remarkup codeblocks, allow calling engines to provide a style map (authored by epriestley, committed by epriestley). · Explain WhyMay 5 2016, 9:51 AM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T10694: Improve behavior of context for inline comments in mail.May 5 2016, 6:55 PM
epriestley mentioned this in T9593: Counterexample header is incorrectly colored in email.May 6 2016, 12:41 AM

Revision Contents
Changeset List

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

Diff 38175

View Options

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

Loading...
Phabricator · Built by Phacility