1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-22 23:02:42 +01:00

Extend Config to full-width

Summary:
Ref T13362. Some applications moved to fixed-width a while ago but I was generally unsatisfied with where they ended up and have been pushing them back to full-width.

Push Config back to full-width. Some of the subpages end up a little weird, but this provides more space to work with to make some improvements, like makign `maniphest.statuses` more legible in the UI>

Test Plan: Grepped for `setFixed(`, updated each page in `/config/`. Browsed each controller, saw workable full-width UIs.

Maniphest Tasks: T13362

Differential Revision: https://secure.phabricator.com/D20925
This commit is contained in:
epriestley 2019-11-25 14:34:32 -08:00
parent 1667acfa5d
commit 33c534f9b7
18 changed files with 39 additions and 57 deletions

View file

@ -64,13 +64,12 @@ final class PhabricatorConfigAllController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($view);
->setFixed(true)
->setMainColumn($view);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -18,9 +18,7 @@ final class PhabricatorConfigApplicationController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($apps_list);
->setFixed(true)
->setMainColumn($apps_list);
$crumbs = $this->buildApplicationCrumbs() $crumbs = $this->buildApplicationCrumbs()
->addTextCrumb($title) ->addTextCrumb($title)
@ -29,6 +27,7 @@ final class PhabricatorConfigApplicationController
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -33,13 +33,12 @@ final class PhabricatorConfigCacheController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($page);
->setFixed(true)
->setMainColumn($page);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -26,13 +26,12 @@ final class PhabricatorConfigClusterDatabasesController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($status);
->setFixed(true)
->setMainColumn($status);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -28,13 +28,12 @@ final class PhabricatorConfigClusterNotificationsController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($status);
->setFixed(true)
->setMainColumn($status);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -32,9 +32,8 @@ final class PhabricatorConfigClusterRepositoriesController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter(
->setFixed(true) array(
->setMainColumn(array(
$repo_status, $repo_status,
$repo_errors, $repo_errors,
)); ));
@ -42,6 +41,7 @@ final class PhabricatorConfigClusterRepositoriesController
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -26,13 +26,12 @@ final class PhabricatorConfigClusterSearchController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($search_status);
->setFixed(true)
->setMainColumn($search_status);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -7,8 +7,6 @@ abstract class PhabricatorConfigController extends PhabricatorController {
} }
public function buildSideNavView($filter = null, $for_app = false) { public function buildSideNavView($filter = null, $for_app = false) {
$guide_href = new PhutilURI('/guides/'); $guide_href = new PhutilURI('/guides/');
$nav = new AphrontSideNavFilterView(); $nav = new AphrontSideNavFilterView();
$nav->setBaseURI(new PhutilURI($this->getApplicationURI())); $nav->setBaseURI(new PhutilURI($this->getApplicationURI()));

View file

@ -167,13 +167,12 @@ final class PhabricatorConfigDatabaseIssueController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($view);
->setFixed(true)
->setMainColumn($view);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -142,13 +142,12 @@ final class PhabricatorConfigDatabaseStatusController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($body);
->setFixed(true)
->setMainColumn($body);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -237,9 +237,8 @@ final class PhabricatorConfigEditController
$view = id(new PHUITwoColumnView()) $view = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter(
->setFixed(true) array(
->setMainColumn(array(
$error_view, $error_view,
$form_box, $form_box,
$status_items, $status_items,
@ -250,6 +249,7 @@ final class PhabricatorConfigEditController
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($view); ->appendChild($view);
} }

View file

@ -36,13 +36,12 @@ final class PhabricatorConfigGroupController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($view);
->setFixed(true)
->setMainColumn($view);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -36,13 +36,12 @@ final class PhabricatorConfigHistoryController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($timeline);
->setFixed(true)
->setMainColumn($timeline);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -59,13 +59,12 @@ final class PhabricatorConfigIssueListController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($issue_list);
->setFixed(true)
->setMainColumn($issue_list);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -47,13 +47,12 @@ final class PhabricatorConfigIssueViewController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($content);
->setFixed(true)
->setMainColumn($content);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -22,13 +22,12 @@ final class PhabricatorConfigListController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($core_list);
->setFixed(true)
->setMainColumn($core_list);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -28,13 +28,12 @@ final class PhabricatorConfigModuleController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($view);
->setFixed(true)
->setMainColumn($view);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }

View file

@ -23,15 +23,13 @@ final class PhabricatorConfigVersionController
$content = id(new PHUITwoColumnView()) $content = id(new PHUITwoColumnView())
->setHeader($header) ->setHeader($header)
->setNavigation($nav) ->setFooter($view);
->setFixed(true)
->setMainColumn($view);
return $this->newPage() return $this->newPage()
->setTitle($title) ->setTitle($title)
->setCrumbs($crumbs) ->setCrumbs($crumbs)
->setNavigation($nav)
->appendChild($content); ->appendChild($content);
} }
public function renderModuleStatus($viewer) { public function renderModuleStatus($viewer) {