Page MenuHomePhabricator
Differential D19274

Support text encoding and syntax highlighting options in document rendering
ClosedPublic
Actions

Authored by epriestley on Mar 30 2018, 6:15 PM.
Tags
None
Referenced Files
F13121776: D19274.diff
Mon, Apr 29, 6:03 PM
Unknown Object (File)
Sat, Apr 27, 8:25 AM
Unknown Object (File)
Thu, Apr 25, 4:01 AM
Unknown Object (File)
Thu, Apr 25, 1:34 AM
Unknown Object (File)
Fri, Apr 19, 7:37 PM
Unknown Object (File)
Thu, Apr 11, 9:10 AM
Unknown Object (File)
Sat, Apr 6, 6:56 AM
Unknown Object (File)
Tue, Apr 2, 7:53 PM
View All 90 Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13105: Plans: Rich presentation and diff rendering pipelines for various file types
Commits
rP7189cb7ba816: Support text encoding and syntax highlighting options in document rendering
Summary

Depends on D19273. Ref T13105. Adds "Change Text Encoding..." and "Highlight As..." options when rendering documents, and makes an effort to automatically detect and handle text encoding.

Test Plan
  • Uploaded a Shift-JIS file, saw it auto-detect as Shift-JIS.
  • Converted files between encodings.
  • Highlighted various things as "Rainbow", etc.

Diff Detail

Repository
rP Phabricator
Branch
doce2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19956
Build 27054: Run Core Tests
Build 27053: arc lint + arc unit

Event Timeline

epriestley created this revision.Mar 30 2018, 6:15 PM
Harbormaster completed remote builds in B19956: Diff 46135.Mar 30 2018, 6:16 PM
epriestley requested review of this revision.Mar 30 2018, 6:16 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 30 2018, 6:28 PM
Closed by commit rP7189cb7ba816: Support text encoding and syntax highlighting options in document rendering (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
30 lines
src/
applications/
files/
controller/
10 lines
24 lines
document/
28 lines
17 lines
60 lines
webroot/
rsrc/
js/
application/
files/
99 lines
phuix/
4 lines

Diff 46135

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/files/controller/PhabricatorFileDocumentController.php

Loading...
View Options

src/applications/files/controller/PhabricatorFileViewController.php

Loading...
View Options

src/applications/files/document/PhabricatorDocumentEngine.php

Loading...
View Options

src/applications/files/document/PhabricatorSourceDocumentEngine.php

Loading...
View Options

src/applications/files/document/PhabricatorTextDocumentEngine.php

Loading...
View Options

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

Loading...
View Options

webroot/rsrc/js/phuix/PHUIXActionView.js

Loading...
Phabricator · Built by Phacility