Page MenuHomePhabricator
Differential D21243

Render proper "Show Context" links in DocumentEngine diffs, not just bullets
ClosedPublic
Actions

Authored by epriestley on May 12 2020, 11:00 PM.
Tags
None
Referenced Files
F19099509: D21243.id.diff
Thu, Dec 4, 8:09 PM
F19089470: D21243.diff
Wed, Dec 3, 7:25 AM
F19034521: D21243.diff
Tue, Nov 25, 2:22 PM
F18866935: D21243.id.diff
Nov 3 2025, 8:23 PM
F18799335: D21243.id50590.diff
Oct 17 2025, 1:16 PM
F18671049: D21243.id50591.diff
Sep 25 2025, 2:32 AM
F18627649: D21243.diff
Sep 16 2025, 4:27 AM
F18235918: D21243.id.diff
Aug 20 2025, 7:53 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13513: Make schema changes to inlines to prepare for new inline features
Commits
rPdf139f044b9b: Render proper "Show Context" links in DocumentEngine diffs, not just bullets
Summary

Ref T13513. Currently, viewing a Jupyter document, hidden context just gets a plain "* * *" facade with no way to expand it.

Support click-to-expand, like source changes.

Test Plan
  • Clicked to expand various Jupyter diffs.
  • Clicked to expand normal source changes.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.May 12 2020, 11:00 PM
Harbormaster completed remote builds in B24407: Diff 50590.May 12 2020, 11:01 PM
epriestley requested review of this revision.May 12 2020, 11:01 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 12 2020, 11:09 PM
Closed by commit rPdf139f044b9b: Render proper "Show Context" links in DocumentEngine diffs, not just bullets (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPdf139f044b9b: Render proper "Show Context" links in DocumentEngine diffs, not just bullets.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
parser/
16 lines
render/
41 lines
56 lines
files/
diff/
76 lines
infrastructure/
internationalization/
translation/
45 lines

Diff 50591

View Options

src/applications/differential/parser/DifferentialChangesetParser.php

Loading...
View Options

src/applications/differential/render/DifferentialChangesetHTMLRenderer.php

Loading...
View Options

src/applications/differential/render/DifferentialChangesetTwoUpRenderer.php

Loading...
View Options

src/applications/files/diff/PhabricatorDocumentEngineBlocks.php

Loading...
View Options

src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php

Loading...
Phabricator · Built by Phacility