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
F14100494: D19234.diff
Tue, Nov 26, 2:12 PM
Unknown Object (File)
Sat, Nov 23, 5:43 PM
Unknown Object (File)
Sat, Nov 23, 5:10 AM
Unknown Object (File)
Sat, Nov 23, 4:49 AM
Unknown Object (File)
Sat, Nov 9, 3:25 PM
Unknown Object (File)
Sun, Nov 3, 11:45 PM
Unknown Object (File)
Fri, Nov 1, 2:31 AM
Unknown Object (File)
Oct 20 2024, 8:14 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
Branch
remarkup1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19872
Build 26931: Run Core Tests
Build 26930: arc lint + arc unit

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.