Page MenuHomePhabricator
Differential D19300

Remove the ability to disable blame in Diffusion
ClosedPublic
Actions

Authored by epriestley on Apr 5 2018, 2:33 PM.
Tags
None
Referenced Files
F15571827: D19300.id46188.diff
Mon, May 5, 10:47 AM
F15563723: D19300.id46201.diff
Wed, Apr 30, 10:06 AM
F15559957: D19300.id46188.diff
Tue, Apr 29, 12:09 PM
F15553846: D19300.id.diff
Mon, Apr 28, 4:08 AM
F15550865: D19300.diff
Sun, Apr 27, 2:11 PM
F15526527: D19300.diff
Mon, Apr 21, 9:17 PM
F15463907: D19300.id46188.diff
Apr 2 2025, 4:24 AM
F15462963: D19300.id46188.diff
Apr 1 2025, 7:20 PM
View All Files
Subscribers
None

Details

Reviewers
mydeveloperday
Maniphest Tasks
T13105: Plans: Rich presentation and diff rendering pipelines for various file types
Commits
rP7d4e25614d00: Remove the ability to disable blame in Diffusion
Summary

Ref T13105. Given that we now load blame with AJAX, it's not clear that there's any benefit to disabling it. This would also interact oddly with the document engine.

Test Plan

Viewed files in Diffusion, no longer saw blame-related options.

Diff Detail

Repository
rP Phabricator
Branch
dfile1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20010
Build 27132: Run Core Tests
Build 27131: arc lint + arc unit

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
controller/
126 lines

Diff 46188

View Options

src/applications/diffusion/controller/DiffusionBrowseController.php

Loading...
Phabricator · Built by Phacility