Page MenuHomePhabricator
Files F18634595

D18519.id44481.diff
No OneTemporary
Actions

D18519.id44481.diff
View Options

diff --git a/resources/celerity/map.php b/resources/celerity/map.php
--- a/resources/celerity/map.php
+++ b/resources/celerity/map.php
@@ -9,7 +9,7 @@
'names' => array(
'conpherence.pkg.css' => 'e68cf1fa',
'conpherence.pkg.js' => 'b5b51108',
- 'core.pkg.css' => '4ac857bf',
+ 'core.pkg.css' => '1d883ad7',
'core.pkg.js' => '6c085267',
'darkconsole.pkg.js' => '1f9a31bc',
'differential.pkg.css' => '45951e9e',
@@ -45,10 +45,9 @@
'rsrc/css/application/base/standard-page-view.css' => '34ee718b',
'rsrc/css/application/chatlog/chatlog.css' => 'd295b020',
'rsrc/css/application/conduit/conduit-api.css' => '7bc725c4',
- 'rsrc/css/application/config/config-options.css' => 'd55ed093',
- 'rsrc/css/application/config/config-page.css' => 'c1d5121b',
+ 'rsrc/css/application/config/config-options.css' => '4615667b',
'rsrc/css/application/config/config-template.css' => '8f18fa41',
- 'rsrc/css/application/config/setup-issue.css' => 'f794cfc3',
+ 'rsrc/css/application/config/setup-issue.css' => '7dae7f18',
'rsrc/css/application/config/unhandled-exception.css' => '4c96257a',
'rsrc/css/application/conpherence/color.css' => 'abb4c358',
'rsrc/css/application/conpherence/durable-column.css' => '89ea6bef',
@@ -132,7 +131,7 @@
'rsrc/css/phui/calendar/phui-calendar-list.css' => '576be600',
'rsrc/css/phui/calendar/phui-calendar-month.css' => '21154caf',
'rsrc/css/phui/calendar/phui-calendar.css' => 'f1ddf11c',
- 'rsrc/css/phui/object-item/phui-oi-big-ui.css' => '19f9369b',
+ 'rsrc/css/phui/object-item/phui-oi-big-ui.css' => '628f59de',
'rsrc/css/phui/object-item/phui-oi-color.css' => 'cd2b9b77',
'rsrc/css/phui/object-item/phui-oi-drag-ui.css' => '08f4ccc3',
'rsrc/css/phui/object-item/phui-oi-flush-ui.css' => '9d9685d6',
@@ -143,7 +142,7 @@
'rsrc/css/phui/phui-badge.css' => '22c0cf4f',
'rsrc/css/phui/phui-basic-nav-view.css' => '98c11ab3',
'rsrc/css/phui/phui-big-info-view.css' => 'acc3492c',
- 'rsrc/css/phui/phui-box.css' => '745e881d',
+ 'rsrc/css/phui/phui-box.css' => '4165eb0d',
'rsrc/css/phui/phui-chart.css' => '6bf6f78e',
'rsrc/css/phui/phui-cms.css' => '504b4b23',
'rsrc/css/phui/phui-comment-form.css' => 'ac68149f',
@@ -158,12 +157,12 @@
'rsrc/css/phui/phui-form-view.css' => 'ae9f8d16',
'rsrc/css/phui/phui-form.css' => '7aaa04e3',
'rsrc/css/phui/phui-head-thing.css' => 'fd311e5f',
- 'rsrc/css/phui/phui-header-view.css' => '808b82c7',
+ 'rsrc/css/phui/phui-header-view.css' => '859c5150',
'rsrc/css/phui/phui-hovercard.css' => 'f0592bcf',
'rsrc/css/phui/phui-icon-set-selector.css' => '87db8fee',
'rsrc/css/phui/phui-icon.css' => '5c4a5de6',
'rsrc/css/phui/phui-image-mask.css' => 'a8498f9c',
- 'rsrc/css/phui/phui-info-view.css' => 'e1b4ec37',
+ 'rsrc/css/phui/phui-info-view.css' => '04e071d7',
'rsrc/css/phui/phui-invisible-character-view.css' => '6993d9f0',
'rsrc/css/phui/phui-left-right.css' => '75227a4d',
'rsrc/css/phui/phui-lightbox.css' => '0a035e40',
@@ -551,8 +550,7 @@
'auth-css' => '0877ed6e',
'bulk-job-css' => 'df9c1d4a',
'conduit-api-css' => '7bc725c4',
- 'config-options-css' => 'd55ed093',
- 'config-page-css' => 'c1d5121b',
+ 'config-options-css' => '4615667b',
'conpherence-color-css' => 'abb4c358',
'conpherence-durable-column-view' => '89ea6bef',
'conpherence-header-pane-css' => 'cb6f4e19',
@@ -822,7 +820,7 @@
'phui-badge-view-css' => '22c0cf4f',
'phui-basic-nav-view-css' => '98c11ab3',
'phui-big-info-view-css' => 'acc3492c',
- 'phui-box-css' => '745e881d',
+ 'phui-box-css' => '4165eb0d',
'phui-button-bar-css' => 'f1ff5494',
'phui-button-css' => '1863cc6e',
'phui-button-simple-css' => '8e1baf68',
@@ -845,20 +843,20 @@
'phui-form-css' => '7aaa04e3',
'phui-form-view-css' => 'ae9f8d16',
'phui-head-thing-view-css' => 'fd311e5f',
- 'phui-header-view-css' => '808b82c7',
+ 'phui-header-view-css' => '859c5150',
'phui-hovercard' => '1bd28176',
'phui-hovercard-view-css' => 'f0592bcf',
'phui-icon-set-selector-css' => '87db8fee',
'phui-icon-view-css' => '5c4a5de6',
'phui-image-mask-css' => 'a8498f9c',
- 'phui-info-view-css' => 'e1b4ec37',
+ 'phui-info-view-css' => '04e071d7',
'phui-inline-comment-view-css' => '65ae3bc2',
'phui-invisible-character-view-css' => '6993d9f0',
'phui-left-right-css' => '75227a4d',
'phui-lightbox-css' => '0a035e40',
'phui-list-view-css' => '38f8c9bd',
'phui-object-box-css' => '9cff003c',
- 'phui-oi-big-ui-css' => '19f9369b',
+ 'phui-oi-big-ui-css' => '628f59de',
'phui-oi-color-css' => 'cd2b9b77',
'phui-oi-drag-ui-css' => '08f4ccc3',
'phui-oi-flush-ui-css' => '9d9685d6',
@@ -896,7 +894,7 @@
'releeph-preview-branch' => 'b7a6f4a5',
'releeph-request-differential-create-dialog' => '8d8b92cd',
'releeph-request-typeahead-css' => '667a48ae',
- 'setup-issue-css' => 'f794cfc3',
+ 'setup-issue-css' => '7dae7f18',
'sprite-login-css' => '396f3c3a',
'sprite-tokens-css' => '9cdfd599',
'syntax-default-css' => '9923583c',
@@ -997,9 +995,6 @@
'185bbd53' => array(
'javelin-install',
),
- '19f9369b' => array(
- 'phui-oi-list-view-css',
- ),
'1ad0a787' => array(
'javelin-install',
'javelin-reactor',
@@ -1366,6 +1361,9 @@
'javelin-magical-init',
'javelin-util',
),
+ '628f59de' => array(
+ 'phui-oi-list-view-css',
+ ),
'62dfea03' => array(
'javelin-install',
'javelin-util',
diff --git a/src/__phutil_library_map__.php b/src/__phutil_library_map__.php
--- a/src/__phutil_library_map__.php
+++ b/src/__phutil_library_map__.php
@@ -2445,7 +2445,6 @@
'PhabricatorConfigOption' => 'applications/config/option/PhabricatorConfigOption.php',
'PhabricatorConfigOptionType' => 'applications/config/custom/PhabricatorConfigOptionType.php',
'PhabricatorConfigPHIDModule' => 'applications/config/module/PhabricatorConfigPHIDModule.php',
- 'PhabricatorConfigPageView' => 'applications/config/view/PhabricatorConfigPageView.php',
'PhabricatorConfigProxySource' => 'infrastructure/env/PhabricatorConfigProxySource.php',
'PhabricatorConfigPurgeCacheController' => 'applications/config/controller/PhabricatorConfigPurgeCacheController.php',
'PhabricatorConfigRegexOptionType' => 'applications/config/custom/PhabricatorConfigRegexOptionType.php',
@@ -7735,7 +7734,6 @@
),
'PhabricatorConfigOptionType' => 'Phobject',
'PhabricatorConfigPHIDModule' => 'PhabricatorConfigModule',
- 'PhabricatorConfigPageView' => 'AphrontTagView',
'PhabricatorConfigProxySource' => 'PhabricatorConfigSource',
'PhabricatorConfigPurgeCacheController' => 'PhabricatorConfigController',
'PhabricatorConfigRegexOptionType' => 'PhabricatorConfigJSONOptionType',
diff --git a/src/applications/config/constants/PhabricatorConfigGroupConstants.php b/src/applications/config/constants/PhabricatorConfigGroupConstants.php
--- a/src/applications/config/constants/PhabricatorConfigGroupConstants.php
+++ b/src/applications/config/constants/PhabricatorConfigGroupConstants.php
@@ -28,8 +28,17 @@
public static function getGroupURI($group) {
$map = array(
self::GROUP_CORE => '/',
- self::GROUP_APPLICATION => pht('application/'),
- self::GROUP_DEVELOPER => pht('developer/'),
+ self::GROUP_APPLICATION => 'application/',
+ self::GROUP_DEVELOPER => 'developer/',
+ );
+ return idx($map, $group, '#');
+ }
+
+ public static function getGroupFullURI($group) {
+ $map = array(
+ self::GROUP_CORE => '/config/',
+ self::GROUP_APPLICATION => '/config/application/',
+ self::GROUP_DEVELOPER => '/config/developer/',
);
return idx($map, $group, '#');
}
diff --git a/src/applications/config/controller/PhabricatorConfigAllController.php b/src/applications/config/controller/PhabricatorConfigAllController.php
--- a/src/applications/config/controller/PhabricatorConfigAllController.php
+++ b/src/applications/config/controller/PhabricatorConfigAllController.php
@@ -49,29 +49,24 @@
));
$title = pht('Current Settings');
-
- $crumbs = $this
- ->buildApplicationCrumbs()
- ->addTextCrumb($title)
- ->setBorder(true);
-
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true);
+ $header = $this->buildHeaderView($title);
$nav = $this->buildSideNavView();
$nav->selectFilter('all/');
- $content = id(new PhabricatorConfigPageView())
+ $view = $this->buildConfigBoxView(
+ pht('All Settings'),
+ $table);
+
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($table);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($view);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
diff --git a/src/applications/config/controller/PhabricatorConfigApplicationController.php b/src/applications/config/controller/PhabricatorConfigApplicationController.php
--- a/src/applications/config/controller/PhabricatorConfigApplicationController.php
+++ b/src/applications/config/controller/PhabricatorConfigApplicationController.php
@@ -11,28 +11,20 @@
$groups = PhabricatorApplicationConfigOptions::loadAll();
$apps_list = $this->buildConfigOptionsList($groups, 'apps');
+ $apps_list = $this->buildConfigBoxView(pht('Applications'), $apps_list);
$title = pht('Application Settings');
+ $header = $this->buildHeaderView($title);
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true);
-
- $crumbs = $this
- ->buildApplicationCrumbs()
- ->addTextCrumb(pht('Applications'))
- ->setBorder(true);
-
- $content = id(new PhabricatorConfigPageView())
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($apps_list);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($apps_list);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
private function buildConfigOptionsList(array $groups, $type) {
diff --git a/src/applications/config/controller/PhabricatorConfigCacheController.php b/src/applications/config/controller/PhabricatorConfigCacheController.php
--- a/src/applications/config/controller/PhabricatorConfigCacheController.php
+++ b/src/applications/config/controller/PhabricatorConfigCacheController.php
@@ -9,16 +9,15 @@
$nav = $this->buildSideNavView();
$nav->selectFilter('cache/');
- $title = pht('Cache Status');
-
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true);
+ $purge_button = id(new PHUIButtonView())
+ ->setText(pht('Purge Caches'))
+ ->setHref('/config/cache/purge/')
+ ->setTag('a')
+ ->setWorkflow(true)
+ ->setIcon('fa-exclamation-triangle');
- $crumbs = $this
- ->buildApplicationCrumbs()
- ->addTextCrumb(pht('Cache Status'))
- ->setBorder(true);
+ $title = pht('Cache Status');
+ $header = $this->buildHeaderView($title, $purge_button);
$code_box = $this->renderCodeBox();
$data_box = $this->renderDataBox();
@@ -28,40 +27,22 @@
$data_box,
);
- $content = id(new PhabricatorConfigPageView())
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($page);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($page);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
private function renderCodeBox() {
$cache = PhabricatorOpcodeCacheSpec::getActiveCacheSpec();
-
$properties = id(new PHUIPropertyListView());
-
$this->renderCommonProperties($properties, $cache);
-
- $purge_button = id(new PHUIButtonView())
- ->setText(pht('Purge Caches'))
- ->setHref('/config/cache/purge/')
- ->setTag('a')
- ->setWorkflow(true)
- ->setIcon('fa-exclamation-triangle');
-
- $header = id(new PHUIHeaderView())
- ->setHeader(pht('Opcode Cache'))
- ->addActionLink($purge_button);
-
- return id(new PHUIObjectBoxView())
- ->setHeader($header)
- ->setBackground(PHUIObjectBoxView::BLUE_PROPERTY)
- ->addPropertyList($properties);
+ return $this->buildConfigBoxView(pht('Opcode Cache'), $properties);
}
private function renderDataBox() {
@@ -109,11 +90,9 @@
));
}
- return id(new PHUIObjectBoxView())
- ->setHeaderText(pht('Data Cache'))
- ->addPropertyList($properties)
- ->setBackground(PHUIObjectBoxView::BLUE_PROPERTY)
- ->setTable($table);
+ $properties = $this->buildConfigBoxView(pht('Data Cache'), $properties);
+ $table = $this->buildConfigBoxView(pht('Cache Storage'), $table);
+ return array($properties, $table);
}
private function renderCommonProperties(
diff --git a/src/applications/config/controller/PhabricatorConfigClusterDatabasesController.php b/src/applications/config/controller/PhabricatorConfigClusterDatabasesController.php
--- a/src/applications/config/controller/PhabricatorConfigClusterDatabasesController.php
+++ b/src/applications/config/controller/PhabricatorConfigClusterDatabasesController.php
@@ -9,34 +9,26 @@
$title = pht('Cluster Database Status');
$doc_href = PhabricatorEnv::getDoclink('Cluster: Databases');
+ $button = id(new PHUIButtonView())
+ ->setIcon('fa-book')
+ ->setHref($doc_href)
+ ->setTag('a')
+ ->setText(pht('Documentation'));
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true)
- ->addActionLink(
- id(new PHUIButtonView())
- ->setIcon('fa-book')
- ->setHref($doc_href)
- ->setTag('a')
- ->setText(pht('Documentation')));
-
- $crumbs = $this
- ->buildApplicationCrumbs()
- ->addTextCrumb($title)
- ->setBorder(true);
+ $header = $this->buildHeaderView($title, $button);
$database_status = $this->buildClusterDatabaseStatus();
+ $status = $this->buildConfigBoxView(pht('Status'), $database_status);
- $content = id(new PhabricatorConfigPageView())
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($database_status);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($status);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
private function buildClusterDatabaseStatus() {
diff --git a/src/applications/config/controller/PhabricatorConfigClusterNotificationsController.php b/src/applications/config/controller/PhabricatorConfigClusterNotificationsController.php
--- a/src/applications/config/controller/PhabricatorConfigClusterNotificationsController.php
+++ b/src/applications/config/controller/PhabricatorConfigClusterNotificationsController.php
@@ -9,34 +9,28 @@
$title = pht('Cluster Notifications');
$doc_href = PhabricatorEnv::getDoclink('Cluster: Notifications');
+ $button = id(new PHUIButtonView())
+ ->setIcon('fa-book')
+ ->setHref($doc_href)
+ ->setTag('a')
+ ->setText(pht('Documentation'));
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true)
- ->addActionLink(
- id(new PHUIButtonView())
- ->setIcon('fa-book')
- ->setHref($doc_href)
- ->setTag('a')
- ->setText(pht('Documentation')));
-
- $crumbs = $this
- ->buildApplicationCrumbs()
- ->addTextCrumb($title)
- ->setBorder(true);
+ $header = $this->buildHeaderView($title, $button);
$notification_status = $this->buildClusterNotificationStatus();
+ $status = $this->buildConfigBoxView(
+ pht('Notifications Status'),
+ $notification_status);
- $content = id(new PhabricatorConfigPageView())
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($notification_status);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($status);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
private function buildClusterNotificationStatus() {
diff --git a/src/applications/config/controller/PhabricatorConfigClusterRepositoriesController.php b/src/applications/config/controller/PhabricatorConfigClusterRepositoriesController.php
--- a/src/applications/config/controller/PhabricatorConfigClusterRepositoriesController.php
+++ b/src/applications/config/controller/PhabricatorConfigClusterRepositoriesController.php
@@ -10,39 +10,35 @@
$title = pht('Cluster Repository Status');
$doc_href = PhabricatorEnv::getDoclink('Cluster: Repositories');
+ $button = id(new PHUIButtonView())
+ ->setIcon('fa-book')
+ ->setHref($doc_href)
+ ->setTag('a')
+ ->setText(pht('Documentation'));
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true)
- ->addActionLink(
- id(new PHUIButtonView())
- ->setIcon('fa-book')
- ->setHref($doc_href)
- ->setTag('a')
- ->setText(pht('Documentation')));
-
- $crumbs = $this
- ->buildApplicationCrumbs()
- ->addTextCrumb(pht('Repository Servers'))
- ->setBorder(true);
+ $header = $this->buildHeaderView($title, $button);
$repository_status = $this->buildClusterRepositoryStatus();
+ $repo_status = $this->buildConfigBoxView(
+ pht('Repository Status'), $repository_status);
+
$repository_errors = $this->buildClusterRepositoryErrors();
+ $repo_errors = $this->buildConfigBoxView(
+ pht('Repository Errors'), $repository_errors);
+
- $content = id(new PhabricatorConfigPageView())
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent(
- array(
- $repository_status,
- $repository_errors,
- ));
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn(array(
+ $repo_status,
+ $repo_errors,
+ ));
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
private function buildClusterRepositoryStatus() {
diff --git a/src/applications/config/controller/PhabricatorConfigClusterSearchController.php b/src/applications/config/controller/PhabricatorConfigClusterSearchController.php
--- a/src/applications/config/controller/PhabricatorConfigClusterSearchController.php
+++ b/src/applications/config/controller/PhabricatorConfigClusterSearchController.php
@@ -10,33 +10,25 @@
$title = pht('Cluster Search');
$doc_href = PhabricatorEnv::getDoclink('Cluster: Search');
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true)
- ->addActionLink(
- id(new PHUIButtonView())
- ->setIcon('fa-book')
- ->setHref($doc_href)
- ->setTag('a')
- ->setText(pht('Documentation')));
-
- $crumbs = $this
- ->buildApplicationCrumbs($nav)
- ->addTextCrumb($title)
- ->setBorder(true);
+ $button = id(new PHUIButtonView())
+ ->setIcon('fa-book')
+ ->setHref($doc_href)
+ ->setTag('a')
+ ->setText(pht('Documentation'));
+
+ $header = $this->buildHeaderView($title, $button);
$search_status = $this->buildClusterSearchStatus();
- $content = id(new PhabricatorConfigPageView())
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($search_status);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($search_status);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
private function buildClusterSearchStatus() {
@@ -105,15 +97,16 @@
->setNoDataString(pht('No search servers are configured.'))
->setHeaders($head);
- $view = id(new PHUIObjectBoxView())
- ->setHeaderText($service->getDisplayName())
- ->setBackground(PHUIObjectBoxView::BLUE_PROPERTY)
- ->setTable($table);
+ $view = $this->buildConfigBoxView(pht('Search Servers'), $table);
- if ($stats_view) {
- $view->addPropertyList($stats_view);
+ $stats = null;
+ if ($stats_view->hasAnyProperties()) {
+ $stats = $this->buildConfigBoxView(
+ pht('%s Stats', $service->getDisplayName()),
+ $stats_view);
}
- return $view;
+
+ return array($stats, $view);
}
private function renderIndexStats($stats) {
diff --git a/src/applications/config/controller/PhabricatorConfigController.php b/src/applications/config/controller/PhabricatorConfigController.php
--- a/src/applications/config/controller/PhabricatorConfigController.php
+++ b/src/applications/config/controller/PhabricatorConfigController.php
@@ -11,46 +11,45 @@
$guide_href = new PhutilURI('/guides/');
$nav = new AphrontSideNavFilterView();
$nav->setBaseURI(new PhutilURI($this->getApplicationURI()));
- $nav->addLabel(pht('Configuration'));
$nav->addFilter('/',
- pht('Core Settings'), null, 'fa-gear');
+ pht('Core Settings'));
$nav->addFilter('application/',
- pht('Application Settings'), null, 'fa-globe');
- $nav->addFilter('history/',
- pht('Settings History'), null, 'fa-history');
- $nav->addFilter('version/',
- pht('Version Information'), null, 'fa-download');
+ pht('Application Settings'));
$nav->addFilter('all/',
- pht('All Settings'), null, 'fa-list-ul');
+ pht('All Settings'));
+ $nav->addFilter('version/',
+ pht('Version Information'));
+ $nav->addFilter('history/',
+ pht('Settings History'));
$nav->addLabel(pht('Setup'));
$nav->addFilter('issue/',
- pht('Setup Issues'), null, 'fa-warning');
+ pht('Setup Issues'));
$nav->addFilter(null,
- pht('Installation Guide'), $guide_href, 'fa-book');
+ pht('Installation Guide'), $guide_href);
$nav->addLabel(pht('Database'));
$nav->addFilter('database/',
- pht('Database Status'), null, 'fa-heartbeat');
+ pht('Database Status'));
$nav->addFilter('dbissue/',
- pht('Database Issues'), null, 'fa-exclamation-circle');
+ pht('Database Issues'));
$nav->addLabel(pht('Cache'));
$nav->addFilter('cache/',
- pht('Cache Status'), null, 'fa-home');
+ pht('Cache Status'));
$nav->addLabel(pht('Cluster'));
$nav->addFilter('cluster/databases/',
- pht('Database Servers'), null, 'fa-database');
+ pht('Database Servers'));
$nav->addFilter('cluster/notifications/',
- pht('Notification Servers'), null, 'fa-bell-o');
+ pht('Notification Servers'));
$nav->addFilter('cluster/repositories/',
- pht('Repository Servers'), null, 'fa-code');
+ pht('Repository Servers'));
$nav->addFilter('cluster/search/',
- pht('Search Servers'), null, 'fa-search');
+ pht('Search Servers'));
$nav->addLabel(pht('Modules'));
$modules = PhabricatorConfigModule::getAllModules();
foreach ($modules as $key => $module) {
$nav->addFilter('module/'.$key.'/',
- $module->getModuleName(), null, 'fa-puzzle-piece');
+ $module->getModuleName());
}
return $nav;
@@ -60,4 +59,37 @@
return $this->buildSideNavView(null, true)->getMenu();
}
+ public function buildHeaderView($text, $action = null) {
+ $viewer = $this->getViewer();
+
+ $file = PhabricatorFile::loadBuiltin($viewer, 'projects/v3/manage.png');
+ $image = $file->getBestURI($file);
+ $header = id(new PHUIHeaderView())
+ ->setHeader($text)
+ ->setProfileHeader(true)
+ ->setImage($image);
+
+ if ($action) {
+ $header->addActionLink($action);
+ }
+
+ return $header;
+ }
+
+ public function buildConfigBoxView($title, $content, $action = null) {
+ $header = id(new PHUIHeaderView())
+ ->setHeader($title);
+
+ if ($action) {
+ $header->addActionItem($action);
+ }
+
+ $view = id(new PHUIObjectBoxView())
+ ->setHeader($header)
+ ->appendChild($content)
+ ->setBackground(PHUIObjectBoxView::WHITE_CONFIG);
+
+ return $view;
+ }
+
}
diff --git a/src/applications/config/controller/PhabricatorConfigDatabaseIssueController.php b/src/applications/config/controller/PhabricatorConfigDatabaseIssueController.php
--- a/src/applications/config/controller/PhabricatorConfigDatabaseIssueController.php
+++ b/src/applications/config/controller/PhabricatorConfigDatabaseIssueController.php
@@ -12,10 +12,6 @@
$expect = $query->loadExpectedSchemata();
$comp_servers = $query->buildComparisonSchemata($expect, $actual);
- $crumbs = $this->buildApplicationCrumbs();
- $crumbs->addTextCrumb(pht('Database Issues'));
- $crumbs->setBorder(true);
-
// Collect all open issues.
$issues = array();
foreach ($comp_servers as $ref_name => $comp) {
@@ -158,24 +154,22 @@
}
$title = pht('Database Issues');
-
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true);
+ $header = $this->buildHeaderView($title);
$nav = $this->buildSideNavView();
$nav->selectFilter('dbissue/');
- $content = id(new PhabricatorConfigPageView())
+ $view = $this->buildConfigBoxView(pht('Issues'), $table);
+
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($table);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($view);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
}
diff --git a/src/applications/config/controller/PhabricatorConfigDatabaseStatusController.php b/src/applications/config/controller/PhabricatorConfigDatabaseStatusController.php
--- a/src/applications/config/controller/PhabricatorConfigDatabaseStatusController.php
+++ b/src/applications/config/controller/PhabricatorConfigDatabaseStatusController.php
@@ -118,6 +118,7 @@
);
}
+ // TODO: Make these work in header
$crumbs = $this->buildApplicationCrumbs();
$crumbs->setBorder(true);
@@ -132,27 +133,23 @@
}
$doc_link = PhabricatorEnv::getDoclink('Managing Storage Adjustments');
+ $button = id(new PHUIButtonView())
+ ->setTag('a')
+ ->setIcon('fa-book')
+ ->setHref($doc_link)
+ ->setText(pht('Documentation'));
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true)
- ->addActionLink(
- id(new PHUIButtonView())
- ->setTag('a')
- ->setIcon('fa-book')
- ->setHref($doc_link)
- ->setText(pht('Learn More')));
-
- $content = id(new PhabricatorConfigPageView())
+ $header = $this->buildHeaderView($title, $button);
+
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($body);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($body);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
@@ -221,11 +218,12 @@
));
$title = pht('Database Status');
-
$properties = $this->buildProperties(
array(
),
$comp->getIssues());
+ $properties = $this->buildConfigBoxView(pht('Properties'), $properties);
+ $table = $this->buildConfigBoxView(pht('Database'), $table);
return $this->buildResponse($title, array($properties, $table));
}
@@ -280,7 +278,7 @@
null,
));
- $title = pht('Database: %s', $database_name);
+ $title = $database_name;
$actual_database = $actual->getDatabase($database_name);
if ($actual_database) {
@@ -325,6 +323,9 @@
),
$database->getIssues());
+ $properties = $this->buildConfigBoxView(pht('Properties'), $properties);
+ $table = $this->buildConfigBoxView(pht('Database'), $table);
+
return $this->buildResponse($title, array($properties, $table));
}
@@ -503,7 +504,7 @@
null,
));
- $title = pht('Database: %s.%s', $database_name, $table_name);
+ $title = pht('%s.%s', $database_name, $table_name);
if ($actual_table) {
$actual_collation = $actual_table->getCollation();
@@ -534,8 +535,14 @@
),
$table->getIssues());
+ $box_header = pht('%s.%s', $database_name, $table_name);
+
+ $properties = $this->buildConfigBoxView(pht('Properties'), $properties);
+ $table = $this->buildConfigBoxView(pht('Database'), $table_view);
+ $keys = $this->buildConfigBoxView(pht('Keys'), $keys_view);
+
return $this->buildResponse(
- $title, array($properties, $table_view, $keys_view));
+ $title, array($properties, $table, $keys));
}
private function renderColumn(
@@ -613,7 +620,7 @@
$title = pht(
- 'Database Status: %s.%s.%s',
+ '%s.%s.%s',
$database_name,
$table_name,
$column_name);
@@ -671,6 +678,8 @@
),
$column->getIssues());
+ $properties = $this->buildConfigBoxView(pht('Properties'), $properties);
+
return $this->buildResponse($title, $properties);
}
@@ -734,7 +743,7 @@
}
$title = pht(
- 'Database Status: %s.%s (%s)',
+ '%s.%s (%s)',
$database_name,
$table_name,
$key_name);
@@ -764,6 +773,8 @@
),
$key->getIssues());
+ $properties = $this->buildConfigBoxView(pht('Properties'), $properties);
+
return $this->buildResponse($title, $properties);
}
diff --git a/src/applications/config/controller/PhabricatorConfigEditController.php b/src/applications/config/controller/PhabricatorConfigEditController.php
--- a/src/applications/config/controller/PhabricatorConfigEditController.php
+++ b/src/applications/config/controller/PhabricatorConfigEditController.php
@@ -7,7 +7,6 @@
$viewer = $request->getViewer();
$key = $request->getURIData('key');
-
$options = PhabricatorApplicationConfigOptions::loadAllOptions();
if (empty($options[$key])) {
$ancient = PhabricatorExtraConfigSetupCheck::getAncientConfig();
@@ -104,44 +103,45 @@
$error_view = null;
if ($errors) {
$error_view = id(new PHUIInfoView())
+ ->setSeverity(PHUIInfoView::SEVERITY_ERROR)
->setErrors($errors);
}
+ $doc_href = PhabricatorEnv::getDoclink(
+ 'Configuration Guide: Locked and Hidden Configuration');
+
+ $doc_link = phutil_tag(
+ 'a',
+ array(
+ 'href' => $doc_href,
+ 'target' => '_blank',
+ ),
+ pht('Learn more about locked and hidden options.'));
+
$status_items = array();
+ $tag = null;
if ($option->getHidden()) {
+ $tag = id(new PHUITagView())
+ ->setName(pht('Hidden'))
+ ->setColor(PHUITagView::COLOR_GREY)
+ ->setBorder(PHUITagView::BORDER_NONE)
+ ->setType(PHUITagView::TYPE_SHADE);
+
$message = pht(
'This configuration is hidden and can not be edited or viewed from '.
'the web interface.');
-
- $status_items[] = id(new PHUIStatusItemView())
- ->setIcon('fa-eye-slash red')
- ->setTarget(phutil_tag('strong', array(), pht('Configuration Hidden')))
- ->setNote($message);
+ $status_items[] = id(new PHUIInfoView())
+ ->appendChild(array($message, ' ', $doc_link));
} else if ($option->getLocked()) {
- $message = $option->getLockedMessage();
+ $tag = id(new PHUITagView())
+ ->setName(pht('Locked'))
+ ->setColor(PHUITagView::COLOR_RED)
+ ->setBorder(PHUITagView::BORDER_NONE)
+ ->setType(PHUITagView::TYPE_SHADE);
- $status_items[] = id(new PHUIStatusItemView())
- ->setIcon('fa-lock red')
- ->setTarget(phutil_tag('strong', array(), pht('Configuration Locked')))
- ->setNote($message);
- }
-
- if ($status_items) {
- $doc_href = PhabricatorEnv::getDoclink(
- 'Configuration Guide: Locked and Hidden Configuration');
-
- $doc_link = phutil_tag(
- 'a',
- array(
- 'href' => $doc_href,
- 'target' => '_blank',
- ),
- pht('Configuration Guide: Locked and Hidden Configuration'));
-
- $status_items[] = id(new PHUIStatusItemView())
- ->setIcon('fa-book')
- ->setTarget(phutil_tag('strong', array(), pht('Learn More')))
- ->setNote($doc_link);
+ $message = $option->getLockedMessage();
+ $status_items[] = id(new PHUIInfoView())
+ ->appendChild(array($message, ' ', $doc_link));
}
if ($option->getHidden() || $option->getLocked()) {
@@ -168,18 +168,6 @@
->setUser($viewer)
->addHiddenInput('issue', $request->getStr('issue'));
- if ($status_items) {
- $status_view = id(new PHUIStatusListView());
-
- foreach ($status_items as $status_item) {
- $status_view->addItem($status_item);
- }
-
- $form->appendControl(
- id(new AphrontFormMarkupControl())
- ->setValue($status_view));
- }
-
$description = $option->getDescription();
if (strlen($description)) {
$description_view = new PHUIRemarkupView($viewer, $description);
@@ -213,60 +201,60 @@
->setValue(pht('Save Config Entry')));
}
+ $current_config = null;
if (!$option->getHidden()) {
- $form->appendChild(
- id(new AphrontFormMarkupControl())
- ->setLabel(pht('Current Configuration'))
- ->setValue($this->renderDefaults($option, $config_entry)));
+ $current_config = $this->renderDefaults($option, $config_entry);
+ $current_config = $this->buildConfigBoxView(
+ pht('Current Configuration'),
+ $current_config);
}
$examples = $this->renderExamples($option);
if ($examples) {
- $form->appendChild(
- id(new AphrontFormMarkupControl())
- ->setLabel(pht('Examples'))
- ->setValue($examples));
+ $examples = $this->buildConfigBoxView(
+ pht('Examples'),
+ $examples);
}
- $title = pht('Edit Option: %s', $key);
- $header_icon = 'fa-pencil';
- $short = pht('Edit');
-
- $form_box = id(new PHUIObjectBoxView())
- ->setHeaderText(pht('Config Option'))
- ->setBackground(PHUIObjectBoxView::BLUE_PROPERTY)
- ->setForm($form);
-
- if ($error_view) {
- $form_box->setInfoView($error_view);
- }
-
- $crumbs = $this->buildApplicationCrumbs();
- $crumbs->addTextCrumb(pht('Config'), $this->getApplicationURI());
+ $title = $key;
+ $box_header = array();
if ($group) {
- $crumbs->addTextCrumb($group->getName(), $group_uri);
+ $box_header[] = phutil_tag(
+ 'a',
+ array(
+ 'href' => $group_uri,
+ ),
+ $group->getName());
+ $box_header[] = " \xC2\xBB ";
}
-
- $crumbs->addTextCrumb($key, '/config/edit/'.$key);
- $crumbs->setBorder(true);
+ $box_header[] = $key;
+ $form_box = $this->buildConfigBoxView($box_header, $form, $tag);
$timeline = $this->buildTransactionTimeline(
$config_entry,
new PhabricatorConfigTransactionQuery());
$timeline->setShouldTerminate(true);
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setHeaderIcon($header_icon);
+ $nav = $this->buildSideNavView();
+ $nav->selectFilter($group_uri);
+
+ $header = $this->buildHeaderView($title);
$view = id(new PHUITwoColumnView())
->setHeader($header)
- ->setFooter($form_box);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn(array(
+ $error_view,
+ $form_box,
+ $status_items,
+ $examples,
+ $current_config,
+ ));
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
->appendChild($view);
}
@@ -426,7 +414,7 @@
}
}
- $table[] = phutil_tag('tr', array(), array(
+ $table[] = phutil_tag('tr', array('class' => 'column-labels'), array(
phutil_tag('th', array(), $description),
phutil_tag('td', array(), $value),
));
@@ -438,6 +426,8 @@
'table',
array(
'class' => 'config-option-table',
+ 'cellspacing' => '0',
+ 'cellpadding' => '0',
),
$table);
}
@@ -497,6 +487,8 @@
'table',
array(
'class' => 'config-option-table',
+ 'cellspacing' => '0',
+ 'cellpadding' => '0',
),
$table);
}
diff --git a/src/applications/config/controller/PhabricatorConfigGroupController.php b/src/applications/config/controller/PhabricatorConfigGroupController.php
--- a/src/applications/config/controller/PhabricatorConfigGroupController.php
+++ b/src/applications/config/controller/PhabricatorConfigGroupController.php
@@ -13,7 +13,7 @@
return new Aphront404Response();
}
- $group_uri = PhabricatorConfigGroupConstants::getGroupURI(
+ $group_uri = PhabricatorConfigGroupConstants::getGroupFullURI(
$options->getGroup());
$group_name = PhabricatorConfigGroupConstants::getGroupShortName(
$options->getGroup());
@@ -22,28 +22,22 @@
$nav->selectFilter($group_uri);
$title = pht('%s Configuration', $options->getName());
+ $header = $this->buildHeaderView($title);
$list = $this->buildOptionList($options->getOptions());
+ $group_url = phutil_tag('a', array('href' => $group_uri), $group_name);
- $crumbs = $this
- ->buildApplicationCrumbs()
- ->addTextCrumb($group_name, $this->getApplicationURI($group_uri))
- ->addTextCrumb($options->getName())
- ->setBorder(true);
+ $box_header = pht("%s \xC2\xBB %s", $group_url, $options->getName());
+ $view = $this->buildConfigBoxView($box_header, $list);
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true);
-
- $content = id(new PhabricatorConfigPageView())
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($list);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($view);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
private function buildOptionList(array $options) {
@@ -77,13 +71,11 @@
->setHref('/config/edit/'.$option->getKey().'/')
->addAttribute($summary);
- $label = pht('Current Value:');
$color = null;
$db_value = idx($db_values, $option->getKey());
if ($db_value && !$db_value->getIsDeleted()) {
$item->setEffect('visited');
$color = 'violet';
- $label = pht('Customized Value:');
}
if ($option->getHidden()) {
@@ -91,6 +83,8 @@
$item->setDisabled(true);
} else if ($option->getLocked()) {
$item->setStatusIcon('fa-lock '.$color, pht('Locked'));
+ } else if ($color) {
+ $item->setStatusIcon('fa-pencil '.$color, pht('Editable'));
} else {
$item->setStatusIcon('fa-pencil-square-o '.$color, pht('Editable'));
}
@@ -102,14 +96,13 @@
$current_value = phutil_tag(
'div',
array(
- 'class' => 'config-options-current-value',
+ 'class' => 'config-options-current-value '.$color,
),
array(
- phutil_tag('span', array(), $label),
- ' '.$current_value,
+ $current_value,
));
- $item->appendChild($current_value);
+ $item->setSideColumn($current_value);
}
$list->addItem($item);
diff --git a/src/applications/config/controller/PhabricatorConfigHistoryController.php b/src/applications/config/controller/PhabricatorConfigHistoryController.php
--- a/src/applications/config/controller/PhabricatorConfigHistoryController.php
+++ b/src/applications/config/controller/PhabricatorConfigHistoryController.php
@@ -29,28 +29,20 @@
$object->willRenderTimeline($timeline, $this->getRequest());
$title = pht('Settings History');
-
- $crumbs = $this->buildApplicationCrumbs();
- $crumbs->addTextCrumb($title);
- $crumbs->setBorder(true);
+ $header = $this->buildHeaderView($title);
$nav = $this->buildSideNavView();
$nav->selectFilter('history/');
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true);
-
- $content = id(new PhabricatorConfigPageView())
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($timeline);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($timeline);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
}
diff --git a/src/applications/config/controller/PhabricatorConfigIssueListController.php b/src/applications/config/controller/PhabricatorConfigIssueListController.php
--- a/src/applications/config/controller/PhabricatorConfigIssueListController.php
+++ b/src/applications/config/controller/PhabricatorConfigIssueListController.php
@@ -43,34 +43,29 @@
}
$title = pht('Setup Issues');
+ $header = $this->buildHeaderView($title);
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true);
-
- $crumbs = $this
- ->buildApplicationCrumbs()
- ->addTextCrumb(pht('Setup Issues'))
- ->setBorder(true);
-
- $page = array(
- $no_issues,
+ $issue_list = array(
$important,
$php,
$mysql,
$other,
);
- $content = id(new PhabricatorConfigPageView())
+ $issue_list = $this->buildConfigBoxView(pht('Issues'), $issue_list);
+
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($page);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn(array(
+ $no_issues,
+ $issue_list,
+ ));
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
private function buildIssueList(array $issues, $group, $fonticon) {
diff --git a/src/applications/config/controller/PhabricatorConfigIssueViewController.php b/src/applications/config/controller/PhabricatorConfigIssueViewController.php
--- a/src/applications/config/controller/PhabricatorConfigIssueViewController.php
+++ b/src/applications/config/controller/PhabricatorConfigIssueViewController.php
@@ -36,19 +36,17 @@
$title = $issue->getShortName();
}
- $crumbs = $this
- ->buildApplicationCrumbs()
- ->setBorder(true)
- ->addTextCrumb(pht('Setup Issues'), $this->getApplicationURI('issue/'))
- ->addTextCrumb($title, $request->getRequestURI())
- ->setBorder(true);
+ $header = $this->buildHeaderView($title);
+
+ $content = id(new PHUITwoColumnView())
+ ->setHeader($header)
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($content);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
private function renderIssue(PhabricatorSetupIssue $issue) {
diff --git a/src/applications/config/controller/PhabricatorConfigListController.php b/src/applications/config/controller/PhabricatorConfigListController.php
--- a/src/applications/config/controller/PhabricatorConfigListController.php
+++ b/src/applications/config/controller/PhabricatorConfigListController.php
@@ -11,28 +11,20 @@
$groups = PhabricatorApplicationConfigOptions::loadAll();
$core_list = $this->buildConfigOptionsList($groups, 'core');
+ $core_list = $this->buildConfigBoxView(pht('Core'), $core_list);
$title = pht('Core Settings');
+ $header = $this->buildHeaderView($title);
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true);
-
- $crumbs = $this
- ->buildApplicationCrumbs()
- ->addTextCrumb(pht('Core'))
- ->setBorder(true);
-
- $content = id(new PhabricatorConfigPageView())
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($core_list);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($core_list);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
private function buildConfigOptionsList(array $groups, $type) {
diff --git a/src/applications/config/controller/PhabricatorConfigModuleController.php b/src/applications/config/controller/PhabricatorConfigModuleController.php
--- a/src/applications/config/controller/PhabricatorConfigModuleController.php
+++ b/src/applications/config/controller/PhabricatorConfigModuleController.php
@@ -16,27 +16,21 @@
$content = $module->renderModuleStatus($request);
$title = $module->getModuleName();
- $crumbs = $this->buildApplicationCrumbs();
- $crumbs->addTextCrumb($title);
- $crumbs->setBorder(true);
-
$nav = $this->buildSideNavView();
$nav->selectFilter('module/'.$key.'/');
+ $header = $this->buildHeaderView($title);
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true);
+ $view = $this->buildConfigBoxView($title, $content);
- $content = id(new PhabricatorConfigPageView())
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($content);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($view);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
}
diff --git a/src/applications/config/controller/PhabricatorConfigVersionController.php b/src/applications/config/controller/PhabricatorConfigVersionController.php
--- a/src/applications/config/controller/PhabricatorConfigVersionController.php
+++ b/src/applications/config/controller/PhabricatorConfigVersionController.php
@@ -7,31 +7,25 @@
$viewer = $request->getViewer();
$title = pht('Version Information');
-
- $crumbs = $this
- ->buildApplicationCrumbs()
- ->addTextCrumb($title)
- ->setBorder(true);
-
$versions = $this->renderModuleStatus($viewer);
$nav = $this->buildSideNavView();
$nav->selectFilter('version/');
+ $header = $this->buildHeaderView($title);
- $header = id(new PHUIHeaderView())
- ->setHeader($title)
- ->setProfileHeader(true);
+ $view = $this->buildConfigBoxView(
+ pht('Installed Versions'),
+ $versions);
- $content = id(new PhabricatorConfigPageView())
+ $content = id(new PHUITwoColumnView())
->setHeader($header)
- ->setContent($versions);
+ ->setNavigation($nav)
+ ->setFixed(true)
+ ->setMainColumn($view);
return $this->newPage()
->setTitle($title)
- ->setCrumbs($crumbs)
- ->setNavigation($nav)
- ->appendChild($content)
- ->addClass('white-background');
+ ->appendChild($content);
}
diff --git a/src/applications/config/view/PhabricatorConfigPageView.php b/src/applications/config/view/PhabricatorConfigPageView.php
deleted file mode 100644
--- a/src/applications/config/view/PhabricatorConfigPageView.php
+++ /dev/null
@@ -1,60 +0,0 @@
-<?php
-
-final class PhabricatorConfigPageView extends AphrontTagView {
-
- private $header;
- private $content;
- private $footer;
-
- public function setHeader(PHUIHeaderView $header) {
- $this->header = $header;
- return $this;
- }
-
- public function setContent($content) {
- $this->content = $content;
- return $this;
- }
-
- public function setFooter($footer) {
- $this->footer = $footer;
- return $this;
- }
-
- protected function getTagName() {
- return 'div';
- }
-
- protected function getTagAttributes() {
- require_celerity_resource('config-page-css');
-
- $classes = array();
- $classes[] = 'config-page';
-
- return array(
- 'class' => implode(' ', $classes),
- );
- }
-
- protected function getTagContent() {
-
- $header = null;
- if ($this->header) {
- $header = phutil_tag_div('config-page-header', $this->header);
- }
-
- $content = null;
- if ($this->content) {
- $content = phutil_tag_div('config-page-content', $this->content);
- }
-
- $footer = null;
- if ($this->footer) {
- $footer = phutil_tag_div('config-page-footer', $this->footer);
- }
-
- return array($header, $content, $footer);
-
- }
-
-}
diff --git a/src/applications/config/view/PhabricatorSetupIssueView.php b/src/applications/config/view/PhabricatorSetupIssueView.php
--- a/src/applications/config/view/PhabricatorSetupIssueView.php
+++ b/src/applications/config/view/PhabricatorSetupIssueView.php
@@ -148,7 +148,6 @@
array(
'href' => '/config/issue/'.$issue->getIssueKey().'/',
'class' => 'button button-grey',
- 'style' => 'float: right',
),
pht('Reload Page'));
}
@@ -194,14 +193,24 @@
array(
'class' => 'setup-issue-head',
),
- array($name, $status));
+ $name);
+
+ $body = phutil_tag(
+ 'div',
+ array(
+ 'class' => 'setup-issue-body',
+ ),
+ array(
+ $status,
+ $description,
+ ));
$tail = phutil_tag(
'div',
array(
'class' => 'setup-issue-tail',
),
- array($actions));
+ $actions);
$issue = phutil_tag(
'div',
@@ -210,7 +219,7 @@
),
array(
$head,
- $description,
+ $body,
$tail,
));
diff --git a/src/view/phui/PHUIInfoView.php b/src/view/phui/PHUIInfoView.php
--- a/src/view/phui/PHUIInfoView.php
+++ b/src/view/phui/PHUIInfoView.php
@@ -59,6 +59,7 @@
} else {
$icon = id(new PHUIIconView())
->setIcon($icon);
+ $this->icon = $icon;
}
return $this;
diff --git a/src/view/phui/PHUIObjectBoxView.php b/src/view/phui/PHUIObjectBoxView.php
--- a/src/view/phui/PHUIObjectBoxView.php
+++ b/src/view/phui/PHUIObjectBoxView.php
@@ -35,6 +35,7 @@
const BLUE = 'phui-box-blue';
const BLUE_PROPERTY = 'phui-box-blue-property';
+ const WHITE_CONFIG = 'phui-box-white-config';
const GREY = 'phui-box-grey';
public function addPropertyList(PHUIPropertyListView $property_list) {
diff --git a/webroot/rsrc/css/application/config/config-options.css b/webroot/rsrc/css/application/config/config-options.css
--- a/webroot/rsrc/css/application/config/config-options.css
+++ b/webroot/rsrc/css/application/config/config-options.css
@@ -5,20 +5,20 @@
.config-option-table {
width: 100%;
border-collapse: collapse;
- border: 1px solid {$thinblueborder};
+ border: none;
background: {$page.content};
}
.config-option-table th,
.config-option-table td {
- padding: 4px 12px;
- border: 1px solid {$lightgreyborder};
+ padding: 8px 12px;
+ border-bottom: 1px solid {$thinblueborder};
}
.config-option-table th {
background: {$lightgreybackground};
color: {$bluetext};
- text-align: right;
+ text-align: left;
white-space: nowrap;
}
@@ -37,17 +37,19 @@
.config-option-table .column-labels th {
font-weight: bold;
color: {$bluetext};
- text-align: center;
- background: {$greybackground};
+ background: {$lightgreybackground};
+ border-right: 1px solid {$thinblueborder};
}
.config-options-current-value {
- padding: 0 8px 6px;
- white-space: pre-wrap;
+ white-space: nowrap;
+ width: 200px;
+ text-overflow: ellipsis;
+ overflow: hidden;
}
-.config-options-current-value span {
- color: {$greytext};
+.config-options-current-value.violet {
+ color: {$violet};
}
.config-options-effective-value,
diff --git a/webroot/rsrc/css/application/config/config-page.css b/webroot/rsrc/css/application/config/config-page.css
deleted file mode 100644
--- a/webroot/rsrc/css/application/config/config-page.css
+++ /dev/null
@@ -1,77 +0,0 @@
-/**
- * @provides config-page-css
- */
-
-.config-page-header {
- margin: 28px 24px 0;
- padding-bottom: 28px;
- border-bottom: 1px solid {$thinblueborder};
-}
-
-.device-phone .config-page-header {
- margin: 4px 12px 0;
- padding-bottom: 4px;
-}
-
-.config-page-header .phui-profile-header {
- padding: 0;
-}
-
-.device-phone .config-page-header .phui-profile-header {
- padding-left: 4px;
- padding-right: 4px;
-}
-
-.config-page-header .phui-profile-header.phui-header-shell .phui-header-header {
- font-size: 20px;
-}
-
-.device-phone .config-page-header .phui-profile-header.phui-header-shell
- .phui-header-header {
- font-size: 16px;
-}
-
-.config-page-content {
- margin: 0 24px;
-}
-
-.device-phone .config-page-content {
- margin: 0 4px;
-}
-
-.device-desktop .config-page-content .phui-oi-list-view {
- padding-left: 0;
- padding-right: 0;
-}
-
-.device-desktop .config-page-content .phui-document-fluid .phui-document-view {
- padding: 16px 0;
- margin: 0;
-}
-
-.config-page-content .aphront-table-view {
- border: none;
-}
-
-.config-page-property {
- padding-top: 4px;
- border-bottom: 1px solid {$thinblueborder};
-}
-
-.config-page-content .aphront-table-notice {
- padding: 0;
-}
-
-.config-page-content .aphront-table-notice .phui-info-view {
- margin-left: 0;
- margin-right: 0;
-}
-
-.config-page-content .aphront-table-wrap + .aphront-table-wrap {
- margin-top: 20px;
- border-top: 1px solid {$thinblueborder};
-}
-
-.config-page-content .phui-box.phui-box-blue-property {
- margin-top: 16px;
-}
diff --git a/webroot/rsrc/css/application/config/setup-issue.css b/webroot/rsrc/css/application/config/setup-issue.css
--- a/webroot/rsrc/css/application/config/setup-issue.css
+++ b/webroot/rsrc/css/application/config/setup-issue.css
@@ -6,15 +6,15 @@
}
.setup-issue-shell {
- max-width: 760px;
+ max-width: 880px;
margin: 16px auto;
}
.setup-issue {
background: #fff;
- border: 1px solid #BFCFDA;
- padding: 8px;
- border-radius: 3px;
+ border: 1px solid #e4e5e6;
+ padding: 0;
+ border-radius: 5px;
}
.setup-issue p {
@@ -24,29 +24,30 @@
.setup-issue table {
width: 100%;
border-collapse: collapse;
- border: 1px solid #BFCFDA;
+ border: 1px solid #e2e2e2;
}
.setup-issue table th {
text-align: right;
width: 30%;
background: #F8F9FC;
- border: 1px solid #BFCFDA;
+ border: 1px solid #e2e2e2;
padding: 8px;
}
.setup-issue table td {
- border: 1px solid #BFCFDA;
+ border: 1px solid #e2e2e2;
padding: 8px;
word-break: break-word;
}
.setup-issue pre {
width: auto;
- border: 1px solid #BFCFDA;
- padding: 10px 2%;
+ border: 1px solid #e2e2e2;
+ padding: 12px;
background: #F8F9FC;
overflow-x: auto;
+ border-radius: 3px;
}
.setup-issue tt {
@@ -60,51 +61,51 @@
}
.setup-issue-instructions {
- font-size: 14px;
- padding: 12px 0;
+ font-size: 13px;
+ padding: 16px;
line-height: 20px;
- background: #fff;
- border-bottom: 1px solid #BFCFDA;
- margin: 0 12px;
+ background: rgba(71,87,120,0.08);
+ border-radius: 3px;
}
.setup-fatal .setup-issue-instructions {
- color: #c0392b;
+ color: #af1404;
background: #f4dddb;
- padding: 12px;
- margin: 0 0 12px;
- border-bottom: 1px solid #c0392b;
}
.setup-issue-name {
- color: #464C5C;
- padding: 4px 8px 12px;
- border-bottom: 1px solid #BFCFDA;
font-size: 15px;
font-weight: bold;
+ color: #6B748C;
+ border-bottom: 1px solid #e2e2e2;
+ overflow: hidden;
+ padding: 16px;
+ line-height: 24px;
}
-.setup-issue-tail {
- margin-top: 12px;
+.setup-issue-body {
+ padding: 16px 16px 0 16px;
}
.setup-issue-status {
- margin: 12px 4px 0;
+ margin: 0 0 16px 0;
padding: 12px;
background: #FDF5D4;
- color: #bc7837;
- border: 1px solid #bc7837;
+ color: #ab8206;
border-radius: 3px;
}
.setup-issue-actions {
- padding: 8px 12px;
- border-top: 1px solid #dfdfdf;
- background-color: #f7f7f7;
- border-bottom-right-radius: 3px;
- border-bottom-left-radius: 3px;
+ padding: 12px;
+ background-color: #f5f5f5;
+ border-bottom-right-radius: 5px;
+ border-bottom-left-radius: 5px;
overflow: hidden;
- margin: 0 -8px -8px -8px;
+ text-align: right;
+}
+
+.setup-issue-actions .button {
+ margin-left: 8px;
}
.setup-issue-next {
@@ -112,14 +113,17 @@
border: 1px solid #BFCFDA;
background: #daeaf3;
text-align: center;
- font-size: 16px;
margin: 12px 0;
color: #2980b9;
border-radius: 3px;
}
.setup-issue-config {
- padding: 8px 12px;
+ padding: 12px 0;
+}
+
+.setup-issue-config + .setup-issue-config {
+ padding-top: 0;
}
.setup-issue ul {
@@ -134,3 +138,12 @@
text-align: right;
color: #74777D;
}
+
+.phui-two-column-view .setup-issue-background {
+ padding: 0;
+}
+
+.phui-two-column-view .setup-issue-shell {
+ width: auto;
+ margin: 0;
+}
diff --git a/webroot/rsrc/css/phui/object-item/phui-oi-big-ui.css b/webroot/rsrc/css/phui/object-item/phui-oi-big-ui.css
--- a/webroot/rsrc/css/phui/object-item/phui-oi-big-ui.css
+++ b/webroot/rsrc/css/phui/object-item/phui-oi-big-ui.css
@@ -31,7 +31,7 @@
}
.device-desktop .phui-oi-list-big .phui-oi {
- margin-bottom: 8px;
+ margin-bottom: 4px;
}
.phui-oi-list-big .phui-oi-col0 {
@@ -46,3 +46,16 @@
.phui-oi-list-big .phui-oi-visited a.phui-oi-link {
color: {$violet};
}
+
+.phui-box-white-config .phui-oi-list-big.phui-oi-list-view {
+ padding: 8px 8px 4px;
+}
+
+.phui-box-white-config .phui-oi-frame {
+ padding: 4px 8px 0;
+}
+
+.device-desktop .phui-box-white-config .phui-oi:hover .phui-oi-frame {
+ background-color: {$hoverblue};
+ border-radius: 3px;
+}
diff --git a/webroot/rsrc/css/phui/phui-box.css b/webroot/rsrc/css/phui/phui-box.css
--- a/webroot/rsrc/css/phui/phui-box.css
+++ b/webroot/rsrc/css/phui/phui-box.css
@@ -115,3 +115,24 @@
.phui-header-shell {
background: {$page.content};
}
+
+/* Config Boxes */
+
+.phui-box-white-config.phui-box-border {
+ border-color: #e2e2e2;
+ border-radius: 5px;
+}
+
+.phui-box-white-config.phui-object-box {
+ padding: 16px 0 0 0;
+}
+
+.phui-box-white-config .phui-header-shell {
+ border-bottom: 1px solid #e2e2e2;
+ overflow: hidden;
+ padding: 0 16px 16px;
+}
+
+.phui-box-white-config .phui-header-header {
+ color: {$bluetext};
+}
diff --git a/webroot/rsrc/css/phui/phui-header-view.css b/webroot/rsrc/css/phui/phui-header-view.css
--- a/webroot/rsrc/css/phui/phui-header-view.css
+++ b/webroot/rsrc/css/phui/phui-header-view.css
@@ -341,10 +341,6 @@
color: {$blacktext};
}
-.phui-profile-header .phui-header-col3 {
- vertical-align: top;
-}
-
.phui-header-view .phui-tag-indigo a {
color: {$sh-indigotext};
}
diff --git a/webroot/rsrc/css/phui/phui-info-view.css b/webroot/rsrc/css/phui/phui-info-view.css
--- a/webroot/rsrc/css/phui/phui-info-view.css
+++ b/webroot/rsrc/css/phui/phui-info-view.css
@@ -46,8 +46,11 @@
}
.phui-info-view-body tt {
- padding: 0 2px;
- background-color: rgba({$alphagrey},.1);
+ color: {$blacktext};
+ background: rgba({$alphablue},0.1);
+ padding: 1px 4px;
+ border-radius: 3px;
+ white-space: pre-wrap;
}
.phui-info-view-actions {

File Metadata

Mime Type
text/plain
Expires
Sep 17 2025, 11:04 PM (12 w, 11 h ago)
Storage Engine
amazon-s3
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
phabricator/secure/zg/z3/lgaalnxyrsmruwnz
Default Alt Text
D18519.id44481.diff (59 KB)

Event Timeline

Phabricator · Built by Phacility