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
F14073250: D19274.diff
Wed, Nov 20, 11:35 PM
Unknown Object (File)
Sat, Nov 16, 6:22 PM
Unknown Object (File)
Sat, Nov 9, 3:06 PM
Unknown Object (File)
Fri, Nov 1, 12:04 AM
Unknown Object (File)
Wed, Oct 23, 11:01 AM
Unknown Object (File)
Tue, Oct 22, 10:52 PM
Unknown Object (File)
Oct 20 2024, 12:48 AM
Unknown Object (File)
Sep 16 2024, 3:07 AM
View All 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 46137

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