Page MenuHomePhabricator

Render object monograms and embedding references more gracefully when building a Remarkup table of contents
ClosedPublic

Authored by epriestley on Mar 16 2018, 10:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 24, 12:11 AM
Unknown Object (File)
Thu, Sep 12, 12:40 PM
Unknown Object (File)
Thu, Sep 12, 12:39 PM
Unknown Object (File)
Thu, Sep 12, 12:38 PM
Unknown Object (File)
Thu, Sep 12, 12:38 PM
Unknown Object (File)
Thu, Sep 12, 5:05 AM
Unknown Object (File)
Tue, Sep 10, 5:01 AM
Unknown Object (File)
Fri, Sep 6, 2:51 AM
Subscribers
None

Details

Summary

Fixes T8845. Ref T13102. See PHI467. Currently, object monograms like L1 which appear in Remarkup headers render incorrectly (with an internal placeholder "x") in the table of contents:

Screen Shot 2018-03-16 at 3.31.55 PM.png (347×372 px, 21 KB)

Instead, render them down to just, e.g., L1 in plain text.

For {P123} I just rendered it to {P123} since it's not really clear to me what users intend. This could be adjusted if there's some reasonable thing that someone is trying to do with this.

Test Plan

Wrote a Phriction document with several object references (like L1 and {P123}) in headers. After patch, saw "x"-free, sensible-looking header names in the table of contents.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Mar 16 2018, 10:44 PM
This revision was automatically updated to reflect the committed changes.