In "Move Tasks to Column...", show only visible columns
ClosedPublic
Actions

Authored by epriestley on Oct 2 2017, 6:38 PM.

Tags

None

Referenced Files

	F19043380: D18668.id44821.diff
	Nov 26 2025, 9:51 PM

	F18960403: D18668.id.diff
	Nov 13 2025, 11:31 AM

	F18809567: D18668.diff
	Oct 19 2025, 3:09 PM

	F18777721: D18668.diff
	Oct 11 2025, 4:46 AM

	F18742211: D18668.diff
	Oct 2 2025, 6:07 PM

	F18589004: D18668.diff
	Sep 12 2025, 2:08 AM

	F18546866: D18668.diff
	Sep 8 2025, 4:49 AM

	F18260196: D18668.diff
	Aug 22 2025, 5:13 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Commits

rP4390a274c1d8: (stable) In "Move Tasks to Column...", show only visible columns
rPf9110b87abf3: In "Move Tasks to Column...", show only visible columns

Summary

See PHI94. I considered this initially but wasn't sure about it. However, PHI94 brings up the good point that we already use a similar rule in Maniphest.

For consistency, only show visible columns here too.

Test Plan

Used "Move tasks to column..." on a board with visible and hidden columns, only saw visbile columns offered in the dropdown.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Oct 2 2017, 6:38 PM

Harbormaster completed remote builds in B18603: Diff 44821.Oct 2 2017, 6:39 PM

amckinley accepted this revision.Oct 2 2017, 6:40 PM

This revision is now accepted and ready to land.Oct 2 2017, 6:40 PM

Closed by commit rPf9110b87abf3: In "Move Tasks to Column...", show only visible columns (authored by epriestley). · Explain WhyOct 2 2017, 6:41 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

project/

controller/

PhabricatorProjectBoardViewController.php

6 lines

Diff 44822

View Options

src/applications/project/controller/PhabricatorProjectBoardViewController.php

In "Move Tasks to Column...", show only visible columnsClosedPublicActions