Paths

Table of Contentst

Differential D19259

Minor document rendering fixes: dropdown for synchronous files, URI normalization for default renderers
ClosedPublic
Actions

Authored by epriestley on Mar 26 2018, 2:20 PM.

Tags

None

Referenced Files

	F18821170: D19259.diff
	Wed, Oct 22, 7:56 PM

	F18754947: D19259.id46105.diff
	Sun, Oct 5, 2:31 AM

	F18745718: D19259.id46111.diff
	Fri, Oct 3, 8:57 AM

	F18719809: D19259.diff
	Mon, Sep 29, 5:40 PM

	F18585722: D19259.diff
	Sep 11 2025, 4:12 PM

	F18563515: D19259.id46111.diff
	Sep 9 2025, 7:37 AM

	F18509476: D19259.id.diff
	Sep 5 2025, 3:25 AM

	F18502484: D19259.diff
	Sep 4 2025, 10:28 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T13105: Plans: Rich presentation and diff rendering pipelines for various file types

Commits

rPb7d3101e7cbf: Minor document rendering fixes: dropdown for synchronous files, URI…

Summary

Depends on D19258. Ref T13105.

When the default renderer is an Ajax renderer, don't replace the URI. For example, when viewing a Jupyter notebook, the URI should remain /F123, not instantly change to /view/123/jupyter/.
Fix an issue where non-ajax renderers could fail to display the dropdown menu properly.

Test Plan

Viewed a Jupyter notebook, stayed on the same URI.
Changed rendering, got different URIs.
Viewed a JSON file and toggled renderers via dropdown.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 26 2018, 2:20 PM

Harbormaster completed remote builds in B19925: Diff 46105.Mar 26 2018, 2:21 PM

epriestley requested review of this revision.Mar 26 2018, 2:21 PM

epriestley added a child revision: D19260: Render directly embedded image data represented as a string in Jupyter notebooks.Mar 26 2018, 2:23 PM

mydeveloperday accepted this revision.Mar 26 2018, 2:24 PM

This revision is now accepted and ready to land.Mar 26 2018, 2:24 PM

Closed by commit rPb7d3101e7cbf: Minor document rendering fixes: dropdown for synchronous files, URI… (authored by epriestley). · Explain WhyMar 28 2018, 10:07 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

14 lines

webroot/

rsrc/

js/

application/

files/

behavior-document-engine.js

7 lines

Diff 46111

resources/celerity/map.php

Loading...

webroot/rsrc/js/application/files/behavior-document-engine.js

Loading...