When reviewing, always show "Accept" checkboxes for packages/projects, even if there's only one checkbox
ClosedPublic
Actions

Authored by epriestley on Apr 11 2017, 12:10 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mar 31 2026, 11:27 AM

	Unknown Object (File)
	Mar 5 2026, 8:47 PM

	Unknown Object (File)
	Mar 5 2026, 12:14 PM

	Unknown Object (File)
	Jan 31 2026, 7:33 AM

	Unknown Object (File)
	Jan 30 2026, 3:11 PM

	Unknown Object (File)
	Jan 13 2026, 5:14 AM

	Unknown Object (File)
	Jan 8 2026, 3:34 PM

	Unknown Object (File)
	Dec 3 2025, 2:51 PM

View All Files

Subscribers

None

Details

Reviewers

chad
lvital

Maniphest Tasks

T12533: Reducing "Accept" action to not render a checkbox when only one package or project would be accepted is perhaps more confusing than not

Commits

rP26d6096e0a5b: When reviewing, always show "Accept" checkboxes for packages/projects, even if…

Summary

Fixes T12533.

Test Plan

Screen Shot 2017-04-10 at 5.09.11 PM.png (328×774 px, 34 KB)

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 11 2017, 12:10 AM

Harbormaster completed remote builds in B16384: Diff 42454.Apr 11 2017, 12:11 AM

lvital accepted this revision.Apr 11 2017, 12:22 AM

This revision is now accepted and ready to land.Apr 11 2017, 12:22 AM

Closed by commit rP26d6096e0a5b: When reviewing, always show "Accept" checkboxes for packages/projects, even if… (authored by epriestley, committed by epriestley). · Explain WhyApr 11 2017, 12:28 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

xaction/

DifferentialRevisionActionTransaction.php

8 lines

Diff 42455

View Options

src/applications/differential/xaction/DifferentialRevisionActionTransaction.php

When reviewing, always show "Accept" checkboxes for packages/projects, even if there's only one checkboxClosedPublicActions