1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-18 19:40:55 +01:00

Use ObjectBoxView for Config pages

Summary: Doing to start to try to remove all the 'purple' PHUIHeaders around Phabricator and see what's left after.

Test Plan:
View each page

{F171007}

Reviewers: epriestley

Reviewed By: epriestley

Differential Revision: https://secure.phabricator.com/D9750
This commit is contained in:
Chad Little 2014-06-27 08:28:33 -07:00
parent 99b839948c
commit 7ec09d3924
6 changed files with 25 additions and 19 deletions

View file

@ -7,7 +7,7 @@
return array(
'names' =>
array(
'core.pkg.css' => '4b1115ac',
'core.pkg.css' => '2151ff22',
'core.pkg.js' => '834b4eda',
'darkconsole.pkg.js' => 'df001cab',
'differential.pkg.css' => '4a93db37',
@ -129,7 +129,7 @@ return array(
'rsrc/css/phui/phui-fontkit.css' => '8246e56e',
'rsrc/css/phui/phui-form-view.css' => 'ebac1b1d',
'rsrc/css/phui/phui-form.css' => 'b78ec020',
'rsrc/css/phui/phui-header-view.css' => 'a2071a67',
'rsrc/css/phui/phui-header-view.css' => '39594ac0',
'rsrc/css/phui/phui-icon.css' => 'd8526aa1',
'rsrc/css/phui/phui-image-mask.css' => '5a8b09c8',
'rsrc/css/phui/phui-info-panel.css' => '27ea50a1',
@ -775,7 +775,7 @@ return array(
'phui-fontkit-css' => '8246e56e',
'phui-form-css' => 'b78ec020',
'phui-form-view-css' => 'ebac1b1d',
'phui-header-view-css' => 'a2071a67',
'phui-header-view-css' => '39594ac0',
'phui-icon-view-css' => 'd8526aa1',
'phui-image-mask-css' => '5a8b09c8',
'phui-info-panel-css' => '27ea50a1',

View file

@ -39,7 +39,6 @@ final class PhabricatorConfigAllController
);
}
$table = id(new AphrontTableView($rows))
->setDeviceReadyTable(true)
->setColumnClasses(
array(
'',
@ -58,9 +57,9 @@ final class PhabricatorConfigAllController
->buildApplicationCrumbs()
->addTextCrumb($title);
$panel = new AphrontPanelView();
$panel = new PHUIObjectBoxView();
$panel->setHeaderText(pht('Current Settings'));
$panel->appendChild($table);
$panel->setNoBackground();
$versions = $this->loadVersions();
@ -71,7 +70,7 @@ final class PhabricatorConfigAllController
}
$object_box = id(new PHUIObjectBoxView())
->setHeaderText(pht('Current Settings'))
->setHeaderText(pht('Current Version'))
->addPropertyList($version_property_list);
$phabricator_root = dirname(phutil_get_library_root('phabricator'));

View file

@ -20,12 +20,12 @@ final class PhabricatorConfigGroupController
}
$title = pht('%s Configuration', $options->getName());
$header = id(new PHUIHeaderView())
->setHeader($title);
$list = $this->buildOptionList($options->getOptions());
$box = id(new PHUIObjectBoxView())
->setHeaderText($title)
->appendChild($list);
$crumbs = $this
->buildApplicationCrumbs()
->addTextCrumb(pht('Config'), $this->getApplicationURI())
@ -34,8 +34,7 @@ final class PhabricatorConfigGroupController
return $this->buildApplicationPage(
array(
$crumbs,
$header,
$list,
$box,
),
array(
'title' => $title,

View file

@ -16,14 +16,15 @@ final class PhabricatorConfigIssueListController
$list = $this->buildIssueList($issues);
$list->setNoDataString(pht('There are no open setup issues.'));
$list->setStackable(true);
$header = id(new PHUIHeaderView())
->setHeader(pht('Open Phabricator Setup Issues'));
$box = id(new PHUIObjectBoxView())
->setHeaderText(pht('Open Phabricator Setup Issues'))
->appendChild($list);
$nav->appendChild(
array(
$header,
$list,
$box,
));
$title = pht('Setup Issues');

View file

@ -15,7 +15,14 @@ final class PhabricatorConfigListController
$title = pht('Phabricator Configuration');
$nav->appendChild($list);
$box = id(new PHUIObjectBoxView())
->setHeaderText($title)
->appendChild($list);
$nav->appendChild(
array(
$box
));
$crumbs = $this
->buildApplicationCrumbs()

View file

@ -6,7 +6,7 @@
background-color: #e0e3ec;
border-width: 1px 0;
border-style: solid;
border-color: #c0c5d1;
border-color: {$hovergrey};
overflow: hidden;
}