Page MenuHomePhabricator

Make "View as Document Type..." only show valid options
ClosedPublic

Authored by epriestley on May 12 2020, 8:33 PM.
Tags
None
Referenced Files
F19915153: D21241.id50585.diff
Fri, Apr 3, 5:12 PM
F19890650: D21241.id.diff
Thu, Mar 26, 3:33 AM
F19827134: D21241.diff
Mar 9 2026, 6:12 AM
F19826682: D21241.id50585.diff
Mar 8 2026, 11:56 PM
F19816174: D21241.id50584.diff
Mar 5 2026, 4:19 AM
F19804707: D21241.diff
Mar 2 2026, 9:31 AM
F19660994: D21241.id50588.diff
Feb 7 2026, 5:26 AM
F19535642: D21241.diff
Jan 21 2026, 7:37 AM
Subscribers
None

Details

Summary

Ref T13513. Currently, "View as Document Type..." lists every available engine.

This is hard to get completely right because we can't always rebuild the document ref accurately in the endpoint, but try harder to fake something reasonable.

Test Plan

Used "View as Document Type..." on Jupyter notebooks, was given "Jupyter" and "Source" as options.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

  • Also restrict the list for changes with no candidate engines.
This revision was not accepted when it landed; it landed in state Needs Review.May 12 2020, 9:25 PM
This revision was automatically updated to reflect the committed changes.