Page Menu
Home
Phabricator
Search
Configure Global Search
Log In
Files
F18860764
D11533.id27748.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Award Token
Flag For Later
Size
11 KB
Referenced Files
None
Subscribers
None
D11533.id27748.diff
View Options
diff --git a/src/applications/auth/controller/config/PhabricatorAuthListController.php b/src/applications/auth/controller/config/PhabricatorAuthListController.php
--- a/src/applications/auth/controller/config/PhabricatorAuthListController.php
+++ b/src/applications/auth/controller/config/PhabricatorAuthListController.php
@@ -90,6 +90,7 @@
pht('Add Authentication Provider'))));
$crumbs = $this->buildApplicationCrumbs();
+ $crumbs->setBorder(true);
$crumbs->addTextCrumb(pht('Auth Providers'));
$config_name = 'auth.email-domains';
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
@@ -33,6 +33,7 @@
$title = pht('Settings History');
$crumbs = $this->buildApplicationCrumbs();
+ $crumbs->setBorder(true);
$crumbs->addTextCrumb('Config', $this->getApplicationURI());
$crumbs->addTextCrumb($title, '/config/history/');
diff --git a/src/applications/conpherence/controller/ConpherenceController.php b/src/applications/conpherence/controller/ConpherenceController.php
--- a/src/applications/conpherence/controller/ConpherenceController.php
+++ b/src/applications/conpherence/controller/ConpherenceController.php
@@ -32,6 +32,7 @@
protected function buildApplicationCrumbs() {
$crumbs = parent::buildApplicationCrumbs();
+ $crumbs->setBorder(true);
$crumbs
->addAction(
diff --git a/src/applications/differential/controller/DifferentialRevisionViewController.php b/src/applications/differential/controller/DifferentialRevisionViewController.php
--- a/src/applications/differential/controller/DifferentialRevisionViewController.php
+++ b/src/applications/differential/controller/DifferentialRevisionViewController.php
@@ -442,15 +442,9 @@
->setRequestURI($request->getRequestURI()));
}
-
$object_id = 'D'.$revision->getID();
- $top_anchor = id(new PhabricatorAnchorView())
- ->setAnchorName('top')
- ->setNavigationMarker(true);
-
$content = array(
- $top_anchor,
$revision_detail_box,
$page_pane,
);
@@ -467,7 +461,6 @@
false);
$nav = id(new DifferentialChangesetFileTreeSideNavBuilder())
- ->setAnchorName('top')
->setTitle('D'.$revision->getID())
->setBaseURI(new PhutilURI('/D'.$revision->getID()))
->setCollapsed((bool)$collapsed)
diff --git a/src/applications/diffusion/controller/DiffusionCommitController.php b/src/applications/diffusion/controller/DiffusionCommitController.php
--- a/src/applications/diffusion/controller/DiffusionCommitController.php
+++ b/src/applications/diffusion/controller/DiffusionCommitController.php
@@ -69,11 +69,6 @@
));
}
-
- $top_anchor = id(new PhabricatorAnchorView())
- ->setAnchorName('top')
- ->setNavigationMarker(true);
-
$audit_requests = $commit->getAudits();
$this->auditAuthorityPHIDs =
PhabricatorAuditCommentEditor::loadAuditPHIDsForUser($user);
@@ -93,7 +88,6 @@
"didn't affect the tracked subdirectory ('%s'), so no ".
"information is available.", $subpath));
$content[] = $error_panel;
- $content[] = $top_anchor;
} else {
$engine = PhabricatorMarkupEngine::newDifferentialMarkupEngine();
$engine->setConfig('viewer', $user);
@@ -151,7 +145,6 @@
'class' => 'diffusion-commit-message phabricator-remarkup',
),
$message));
- $content[] = $top_anchor;
$object_box = id(new PHUIObjectBoxView())
->setHeader($headsup_view)
@@ -381,7 +374,6 @@
if ($changesets && $show_filetree) {
$nav = id(new DifferentialChangesetFileTreeSideNavBuilder())
- ->setAnchorName('top')
->setTitle($short_name)
->setBaseURI(new PhutilURI('/'.$commit_id))
->build($changesets)
diff --git a/src/applications/herald/controller/HeraldRuleEditHistoryController.php b/src/applications/herald/controller/HeraldRuleEditHistoryController.php
--- a/src/applications/herald/controller/HeraldRuleEditHistoryController.php
+++ b/src/applications/herald/controller/HeraldRuleEditHistoryController.php
@@ -30,10 +30,9 @@
->setHandles($handles)
->setUser($this->getRequest()->getUser());
- $panel = new AphrontPanelView();
- $panel->setHeader(pht('Edit History'));
+ $panel = new PHUIObjectBoxView();
+ $panel->setHeaderText(pht('Edit History'));
$panel->appendChild($list_view);
- $panel->setNoBackground();
$crumbs = $this
->buildApplicationCrumbs($can_create = false)
diff --git a/src/applications/legalpad/controller/LegalpadDocumentSignController.php b/src/applications/legalpad/controller/LegalpadDocumentSignController.php
--- a/src/applications/legalpad/controller/LegalpadDocumentSignController.php
+++ b/src/applications/legalpad/controller/LegalpadDocumentSignController.php
@@ -276,6 +276,7 @@
}
$crumbs = $this->buildApplicationCrumbs();
+ $crumbs->setBorder(true);
$crumbs->addTextCrumb($document->getMonogram());
return $this->buildApplicationPage(
diff --git a/src/applications/people/controller/PhabricatorPeopleLdapController.php b/src/applications/people/controller/PhabricatorPeopleLdapController.php
--- a/src/applications/people/controller/PhabricatorPeopleLdapController.php
+++ b/src/applications/people/controller/PhabricatorPeopleLdapController.php
@@ -32,11 +32,9 @@
id(new AphrontFormSubmitControl())
->setValue(pht('Search')));
- $panel = id(new AphrontPanelView())
- ->setHeader(pht('Import LDAP Users'))
- ->setNoBackground()
- ->setWidth(AphrontPanelView::WIDTH_FORM)
- ->appendChild($form);
+ $panel = id(new PHUIObjectBoxView())
+ ->setHeaderText(pht('Import LDAP Users'))
+ ->setForm($form);
$crumbs = $this->buildApplicationCrumbs();
$crumbs->addTextCrumb(
diff --git a/src/applications/phame/controller/post/PhamePostListController.php b/src/applications/phame/controller/post/PhamePostListController.php
--- a/src/applications/phame/controller/post/PhamePostListController.php
+++ b/src/applications/phame/controller/post/PhamePostListController.php
@@ -75,6 +75,7 @@
->appendChild($post_list);
$crumbs = $this->buildApplicationCrumbs();
+ $crumbs->setBorder(true);
$crumbs->addTextCrumb($title, $this->getApplicationURI());
$nav->appendChild(
diff --git a/src/applications/phriction/controller/PhrictionDocumentController.php b/src/applications/phriction/controller/PhrictionDocumentController.php
--- a/src/applications/phriction/controller/PhrictionDocumentController.php
+++ b/src/applications/phriction/controller/PhrictionDocumentController.php
@@ -185,6 +185,7 @@
$actions = $this->buildActionView($user, $document);
$crumbs = $this->buildApplicationCrumbs();
+ $crumbs->setBorder(true);
$crumb_views = $this->renderBreadcrumbs($slug);
foreach ($crumb_views as $view) {
$crumbs->addCrumb($view);
diff --git a/src/applications/search/controller/PhabricatorApplicationSearchController.php b/src/applications/search/controller/PhabricatorApplicationSearchController.php
--- a/src/applications/search/controller/PhabricatorApplicationSearchController.php
+++ b/src/applications/search/controller/PhabricatorApplicationSearchController.php
@@ -261,6 +261,7 @@
$crumbs = $parent
->buildApplicationCrumbs()
+ ->setBorder(true)
->addTextCrumb($title);
$nav->setCrumbs($crumbs);
diff --git a/src/applications/tokens/controller/PhabricatorTokenGivenController.php b/src/applications/tokens/controller/PhabricatorTokenGivenController.php
--- a/src/applications/tokens/controller/PhabricatorTokenGivenController.php
+++ b/src/applications/tokens/controller/PhabricatorTokenGivenController.php
@@ -39,6 +39,7 @@
}
$list = new PHUIObjectItemListView();
+ $list->setStackable(true);
foreach ($tokens_given as $token_given) {
$handle = $handles[$token_given->getObjectPHID()];
$token = idx($tokens, $token_given->getTokenPHID());
@@ -57,17 +58,20 @@
$list->addItem($item);
}
- $list->setPager($pager);
-
$title = pht('Tokens Given');
+ $box = id(new PHUIObjectBoxView())
+ ->setHeaderText($title)
+ ->appendChild($list);
+
$nav = $this->buildSideNav();
$nav->setCrumbs(
$this->buildApplicationCrumbs()
->addTextCrumb($title));
$nav->selectFilter('given/');
- $nav->appendChild($list);
+ $nav->appendChild($box);
+ $nav->appendChild($pager);
return $this->buildApplicationPage(
$nav,
diff --git a/src/applications/tokens/controller/PhabricatorTokenLeaderController.php b/src/applications/tokens/controller/PhabricatorTokenLeaderController.php
--- a/src/applications/tokens/controller/PhabricatorTokenLeaderController.php
+++ b/src/applications/tokens/controller/PhabricatorTokenLeaderController.php
@@ -30,6 +30,7 @@
}
$list = new PHUIObjectItemListView();
+ $list->setStackable(true);
foreach ($phids as $object) {
$count = idx($counts, $object, 0);
$item = id(new PHUIObjectItemView());
@@ -43,13 +44,17 @@
$title = pht('Token Leader Board');
+ $box = id(new PHUIObjectBoxView())
+ ->setHeaderText($title)
+ ->appendChild($list);
+
$nav = $this->buildSideNav();
$nav->setCrumbs(
$this->buildApplicationCrumbs()
->addTextCrumb($title));
$nav->selectFilter('leaders/');
- $nav->appendChild($list);
+ $nav->appendChild($box);
$nav->appendChild($pager);
return $this->buildApplicationPage(
diff --git a/src/view/phui/PHUICrumbsView.php b/src/view/phui/PHUICrumbsView.php
--- a/src/view/phui/PHUICrumbsView.php
+++ b/src/view/phui/PHUICrumbsView.php
@@ -4,6 +4,7 @@
private $crumbs = array();
private $actions = array();
+ private $border;
protected function canAppendChild() {
return false;
@@ -35,6 +36,11 @@
return $this;
}
+ public function setBorder($border) {
+ $this->border = $border;
+ return $this;
+ }
+
public function render() {
require_celerity_resource('phui-crumbs-view-css');
@@ -99,6 +105,9 @@
$classes = array();
$classes[] = 'phui-crumbs-view';
+ if ($this->border) {
+ $classes[] = 'phui-crumbs-border';
+ }
return phutil_tag(
'div',
diff --git a/webroot/rsrc/css/phui/phui-crumbs-view.css b/webroot/rsrc/css/phui/phui-crumbs-view.css
--- a/webroot/rsrc/css/phui/phui-crumbs-view.css
+++ b/webroot/rsrc/css/phui/phui-crumbs-view.css
@@ -10,7 +10,6 @@
Remove this once that gets sorted out. */
position: relative;
-webkit-font-smoothing: antialiased;
- border-bottom: 1px solid rgba(205, 207, 211, 1);
}
.phui-crumbs-view,
@@ -99,3 +98,15 @@
.phui-crumb-divider {
margin: 2px 8px;
}
+
+.phui-crumbs-view.phui-crumbs-border {
+ border-bottom: 1px solid rgba(205, 207, 211, 1);
+}
+
+.phui-crumbs-view + .phui-object-box {
+ margin-top: 0;
+}
+
+body .phui-crumbs-view + .phui-object-item-list-view {
+ padding-top: 0;
+}
File Metadata
Details
Attached
Mime Type
text/plain
Expires
Nov 3 2025, 12:39 PM (10 w, 3 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
12962540
Default Alt Text
D11533.id27748.diff (11 KB)
Attached To
Mode
D11533: Make border conditional in crumbs
Attached
Detach File
Event Timeline
Log In to Comment