Differential D19302

Move Diffusion browse rendering to DocumentEngine, breaking almost all features
ClosedPublic
Actions

Authored by epriestley on Apr 5 2018, 6:40 PM.

Details

Reviewers

mydeveloperday

Maniphest Tasks

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

Commits

rP1fde4a945061: Move Diffusion browse rendering to DocumentEngine, breaking almost all features

Summary

Ref T13105. This breaks about 9,000 features but moves Diffusion to DocumentEngine for rendering. See T13105 for a more complete list of all the broken stuff.

But you can't bake a software without breaking all the features every time you make a change, right?

Test Plan

Viewed various files in Diffusion, used DocumentEngine features like highlighting and rendering engine selection.

Diff Detail

Repository

rP Phabricator

Branch

dfile3

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 20012
Build 27136: Run Core Tests
Build 27135: arc lint + arc unit

Event Timeline

epriestley created this revision.Apr 5 2018, 6:40 PM

Harbormaster completed remote builds in B20012: Diff 46190.Apr 5 2018, 6:41 PM

epriestley requested review of this revision.Apr 5 2018, 6:41 PM

When people realize what will be capable they won't worry about about the 9,000 features

If I understand correctly this commit moves that old "Browse in Diffusion" to use the DocumentRenderEngine, so it will use the source code with syntax highlighting that you added last week. I guess we lose blame (for now) as one of those 9000 applications, but gain the ability for different file types to select alternative renders.. sweet!.. "breaking eggs and making omelettes and all that!"

This revision is now accepted and ready to land.Apr 6 2018, 7:52 AM

mydeveloperday added inline comments.Apr 6 2018, 7:56 AM