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

	F15436476: D19300.diff
	Tue, Mar 25, 2:30 PM

	F15435236: D19300.diff
	Tue, Mar 25, 7:28 AM

	F15431314: D19300.id46201.diff
	Mon, Mar 24, 11:44 AM

	F15427082: D19300.id.diff
	Sun, Mar 23, 12:39 PM

	F15342835: D19300.id46188.diff
	Sun, Mar 9, 9:30 PM

	Unknown Object (File)
	Feb 20 2025, 1:39 AM

	Unknown Object (File)
	Feb 2 2025, 2:48 PM

	Unknown Object (File)
	Jan 28 2025, 3:38 AM

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 5 2018, 2:33 PM

Harbormaster completed remote builds in B20010: Diff 46188.Apr 5 2018, 2:34 PM

epriestley requested review of this revision.Apr 5 2018, 2:34 PM

mydeveloperday accepted this revision.Apr 5 2018, 5:06 PM

This revision is now accepted and ready to land.Apr 5 2018, 5:06 PM

Closed by commit rP7d4e25614d00: Remove the ability to disable blame in Diffusion (authored by epriestley). · Explain WhyApr 9 2018, 11:45 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

controller/

DiffusionBrowseController.php

126 lines

Diff 46201

View Options

src/applications/diffusion/controller/DiffusionBrowseController.php

Remove the ability to disable blame in DiffusionClosedPublicActions