Differential D19306

Disable default syntax highlighting for large files in DocumentEngine
ClosedPublic
Actions

Authored by epriestley on Apr 8 2018, 3:13 PM.

Tags

None

Referenced Files

	F15187598: D19306.diff
	Fri, Feb 21, 10:17 AM

	F15186888: D19306.id46196.diff
	Fri, Feb 21, 4:38 AM

	Unknown Object (File)
	Thu, Feb 20, 10:40 AM

	Unknown Object (File)
	Thu, Feb 20, 1:36 AM

	Unknown Object (File)
	Tue, Feb 18, 1:28 AM

	Unknown Object (File)
	Sun, Feb 16, 12:26 PM

	Unknown Object (File)
	Tue, Jan 28, 3:39 AM

	Unknown Object (File)
	Fri, Jan 24, 9:15 AM

View All Files

Subscribers

None

Details

Reviewers

mydeveloperday

Maniphest Tasks

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

Commits

rP0363febeb2c6: Disable default syntax highlighting for large files in DocumentEngine

Summary

Ref T13105. See also T7895. When users render very large files as source via DocumentEngine, skip highlighting.

Test Plan

Fiddled with the limit, viewed files, saw highlighting degrade.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 8 2018, 3:13 PM

Harbormaster completed remote builds in B20020: Diff 46196.Apr 8 2018, 3:14 PM

epriestley requested review of this revision.Apr 8 2018, 3:14 PM

mydeveloperday accepted this revision.Apr 9 2018, 10:07 AM

This revision is now accepted and ready to land.Apr 9 2018, 10:07 AM

Closed by commit rP0363febeb2c6: Disable default syntax highlighting for large files in DocumentEngine (authored by epriestley). · Explain WhyApr 9 2018, 11:47 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

files/

document/

PhabricatorSourceDocumentEngine.php

21 lines

Diff 46205

View Options

src/applications/files/document/PhabricatorSourceDocumentEngine.php

Disable default syntax highlighting for large files in DocumentEngineClosedPublicActions