Page MenuHomePhabricator

Introduce a quoted reply block rule for Remarkup
ClosedPublic

Authored by epriestley on May 2 2014, 5:11 PM.
Tags
None
Referenced Files
F13236651: D8953.id21244.diff
Tue, May 21, 10:20 AM
F13229656: D8953.id.diff
Mon, May 20, 6:14 PM
F13202921: D8953.diff
Tue, May 14, 11:18 PM
F13191190: D8953.id21309.diff
Sat, May 11, 5:52 PM
F13191189: D8953.id21244.diff
Sat, May 11, 5:52 PM
F13187797: D8953.diff
Sat, May 11, 4:45 AM
Unknown Object (File)
Tue, May 7, 6:59 AM
Unknown Object (File)
Fri, May 3, 6:37 AM

Details

Summary

Ref T4119. The tricky part here is that it's desirable to preserve other block-level formatting elements (like lists, other quotes, etc) inside the block.

To permit that, allow block-level elements to reparse their contents into more block-level elements, and essentially build a parse tree.

The new ReplyBlockRule strips off the ">>!" header and the ">" prefixes, then renders the rest normally, then slaps a blockquote around everything.

I haven't tweaked the styles too much, @chad might have thoughts on that. There should be enough tags/classes to get things rendering pretty flexibly.

I'm probably going to make the normal quote style work like this too (so block-level elements inside it are preserved). A big thing this will help with is inline replies into these blocks. This might imply keeping both quote styles relatively similar.

The selection of ">>!" is arbitrary, but I want some unique marker to indicate the meta/header text. This seemed as good as any.

Test Plan

Screen_Shot_2014-05-02_at_10.08.15_AM.png (843×628 px, 52 KB)

Diff Detail

Repository
rPHU libphutil
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Introduce a quoted reply block rule for Remarkup.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: btrahan, chad.
epriestley added a subscriber: chad.
btrahan edited edge metadata.
This revision is now accepted and ready to land.May 2 2014, 7:08 PM
epriestley updated this revision to Diff 21309.

Closed by commit rPHUb88b8ba78580 (authored by @epriestley).