Page MenuHomePhabricator

Replace weird, redundant list of branches in Diffusion "Manage" UI with a link to the main branch list
ClosedPublic

Authored by epriestley on Jun 17 2019, 5:52 PM.

Details

Summary

Fixes T13312. Currently, ManageBranches has a list of branches on the same page. This has a few minor issues:

  • Pager is at the top (see T13312), which is weird.
  • "Default" icon is mystery meat.
  • Table is kind of pointless/redundant in general?

Previously, this table had more information about technical status of each branch (autoclose/track/publish) but most of these details have been simplified/eliminated, and the main "Branches" view now has more information than it did before.

Get rid of this and just link to the main view.

Test Plan

Viewed "Branches" in UI, saw a link to the main view instead of a weird table.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Jun 17 2019, 5:52 PM
epriestley requested review of this revision.Jun 17 2019, 5:53 PM
amckinley accepted this revision.Jun 18 2019, 9:19 PM
This revision is now accepted and ready to land.Jun 18 2019, 9:19 PM