Page MenuHomePhabricator
Differential D19309

Mostly make blame work with DocumentEngine
ClosedPublic
Actions

Authored by epriestley on Apr 8 2018, 7:53 PM.
Tags
None
Referenced Files
F15576143: D19309.id46200.diff
Tue, May 6, 8:27 AM
F15573236: D19309.id46200.diff
Mon, May 5, 8:04 PM
F15569804: D19309.diff
Sun, May 4, 4:19 AM
F15553790: D19309.id.diff
Mon, Apr 28, 3:47 AM
F15543665: D19309.diff
Sat, Apr 26, 12:14 AM
F15458136: D19309.id46207.diff
Mar 30 2025, 9:04 PM
F15449643: D19309.id.diff
Mar 28 2025, 11:26 AM
F15449306: D19309.id46200.diff
Mar 28 2025, 9:23 AM
View All Files
Subscribers
None

Details

Reviewers
mydeveloperday
Maniphest Tasks
T13105: Plans: Rich presentation and diff rendering pipelines for various file types
Commits
rP09c6d42b9599: Mostly make blame work with DocumentEngine
Summary

Ref T13105. This needs refinement but blame sort of works again, now.

Test Plan

Viewed files in Diffusion and Files; saw blame in Diffusion when viewing in source mode.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 8 2018, 7:53 PM
Harbormaster completed remote builds in B20024: Diff 46200.Apr 8 2018, 7:54 PM
epriestley requested review of this revision.Apr 8 2018, 7:54 PM
mydeveloperday accepted this revision.Apr 9 2018, 10:13 AM
This revision is now accepted and ready to land.Apr 9 2018, 10:13 AM
Closed by commit rP09c6d42b9599: Mostly make blame work with DocumentEngine (authored by epriestley). · Explain WhyApr 9 2018, 11:48 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
42 lines
1 line
src/
2 lines
applications/
diffusion/
application/
2 lines
controller/
254 lines
264 lines
document/
12 lines
files/
document/
4 lines
8 lines
14 lines
21 lines
render/
20 lines
repository/
storage/
2 lines
22 lines
view/
layout/
44 lines
webroot/
rsrc/
css/
application/
differential/
4 lines
diffusion/
layout/
43 lines
js/
application/
diffusion/
files/
84 lines

Diff 46207

View Options

resources/celerity/map.php

Loading...
View Options

resources/celerity/packages.php

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/diffusion/application/PhabricatorDiffusionApplication.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionBlameController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionBrowseController.php

Loading...
View Options

src/applications/diffusion/document/DiffusionDocumentRenderingEngine.php

Loading...
View Options

src/applications/files/document/PhabricatorDocumentEngine.php

Loading...
View Options

src/applications/files/document/PhabricatorDocumentRef.php

Loading...
View Options

src/applications/files/document/PhabricatorSourceDocumentEngine.php

Loading...
View Options

src/applications/files/document/PhabricatorTextDocumentEngine.php

Loading...
View Options

src/applications/files/document/render/PhabricatorDocumentRenderingEngine.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepository.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryCommit.php

Loading...
View Options

src/view/layout/PhabricatorSourceCodeView.php

Loading...
View Options

webroot/rsrc/css/application/differential/changeset-view.css

Loading...
View Options

webroot/rsrc/css/application/diffusion/diffusion-source.css

Loading...
View Options

webroot/rsrc/css/layout/phabricator-source-code-view.css

Loading...
View Options

webroot/rsrc/js/application/diffusion/behavior-load-blame.js

Loading...
View Options

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

Loading...
Phabricator · Built by Phacility