Page MenuHomePhabricator

"Hide" columns instead of "Deleting" them
ClosedPublic

Authored by epriestley on Jun 25 2014, 3:56 AM.

Details

Summary

Fixes T5342. Fixes T5161. Previously, we were a bit strict about deleting columns because you could orphan tasks. Let users recover these columns more easily so they can't shoot themselves in the foot.

  • Change "Delete" language to "Hide".
  • Add a button to let you see hidden columns.
  • Remove restriction that you can only delete empty columns.

The new button is a little funky, but maybe it merges into the "Add Column" button and that becomes a dropdown with board actions? The rest of this feels OK to me.

Test Plan

See screenshot.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to "Hide" columns instead of "Deleting" them.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

Yeah I think we're going to want a generic action dropdown so this doesn't become buttontown, population 20.

Yeah, let me get a diff out for that.

chad edited edge metadata.

I'd probably lean towards enable/disable over show/hide as it seems more clear that you won't be able to do things to it for certain. But I don't feel super strongly, up to you.

This revision is now accepted and ready to land.Jun 25 2014, 7:07 PM

The columns do still work, you just have to show them. There are a couple of other UI things, though:

  • It would be nice to visually indicate that hidden columns are being shown somehow, but my 2 seconds of poking at it didn't produce anything that felt very good.
  • The red header for hidden columns isn't great/obvious.
  • "Hide Hidden Columns" isn't great.

I think I'm going to leave it as-is for now, but probably tweak things soon and/or if users are confused.

epriestley updated this revision to Diff 23354.

Closed by commit rPa823d143b404 (authored by @epriestley).