Skip to content

Commit d2f3e7f

Browse files
author
Chad Little
committedNov 17, 2016
Remove sidenav from Workboards
Summary: Visually, I think these are much cleaner (with colors), and provide more workspace. I also don't really use the sidenav here and if I did, it would be to go back to the project homepage. I think this is overall better. If navigation page to project home is difficult or hard to find, we can maybe make a better header / crumbs bar to reduce that. Test Plan: New project -> basic new board. Existing project -> color board. Desktop, Mobile, Fullscreen Reviewers: epriestley Reviewed By: epriestley Subscribers: Korvin Differential Revision: https://secure.phabricator.com/D16882
1 parent 42f3083 commit d2f3e7f

File tree

6 files changed

+14
-42
lines changed

6 files changed

+14
-42
lines changed
 

‎resources/celerity/map.php

+4-4
Original file line numberDiff line numberDiff line change
@@ -165,8 +165,8 @@
165165
'rsrc/css/phui/phui-tag-view.css' => '6bbd83e2',
166166
'rsrc/css/phui/phui-timeline-view.css' => 'bc523970',
167167
'rsrc/css/phui/phui-two-column-view.css' => 'bbe32c23',
168-
'rsrc/css/phui/workboards/phui-workboard-color.css' => 'ac6fe6a7',
169-
'rsrc/css/phui/workboards/phui-workboard.css' => 'e09eb53a',
168+
'rsrc/css/phui/workboards/phui-workboard-color.css' => '207828dd',
169+
'rsrc/css/phui/workboards/phui-workboard.css' => '60d09514',
170170
'rsrc/css/phui/workboards/phui-workcard.css' => '0c62d7c5',
171171
'rsrc/css/phui/workboards/phui-workpanel.css' => '92197373',
172172
'rsrc/css/sprite-login.css' => '6dbbbd97',
@@ -883,8 +883,8 @@
883883
'phui-theme-css' => '798c69b8',
884884
'phui-timeline-view-css' => 'bc523970',
885885
'phui-two-column-view-css' => 'bbe32c23',
886-
'phui-workboard-color-css' => 'ac6fe6a7',
887-
'phui-workboard-view-css' => 'e09eb53a',
886+
'phui-workboard-color-css' => '207828dd',
887+
'phui-workboard-view-css' => '60d09514',
888888
'phui-workcard-view-css' => '0c62d7c5',
889889
'phui-workpanel-view-css' => '92197373',
890890
'phuix-action-list-view' => 'b5c256b8',

‎src/applications/project/controller/PhabricatorProjectBoardViewController.php

-2
Original file line numberDiff line numberDiff line change
@@ -416,7 +416,6 @@ public function handleRequest(AphrontRequest $request) {
416416
->appendChild($board)
417417
->addClass('project-board-wrapper');
418418

419-
$nav = $this->getProfileMenu();
420419
$divider = id(new PHUIListItemView())
421420
->setType(PHUIListItemView::TYPE_DIVIDER);
422421
$fullscreen = $this->buildFullscreenMenu();
@@ -439,7 +438,6 @@ public function handleRequest(AphrontRequest $request) {
439438
))
440439
->setPageObjectPHIDs(array($project->getPHID()))
441440
->setShowFooter(false)
442-
->setNavigation($nav)
443441
->setCrumbs($crumbs)
444442
->addQuicksandConfig(
445443
array(

‎src/applications/project/controller/PhabricatorProjectController.php

+2-1
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,8 @@ protected function buildApplicationCrumbs() {
119119
foreach ($ancestors as $ancestor) {
120120
$crumbs->addTextCrumb(
121121
$ancestor->getName(),
122-
$ancestor->getURI());
122+
$ancestor->getProfileURI()
123+
);
123124
}
124125
}
125126

‎src/applications/project/storage/PhabricatorProject.php

+5
Original file line numberDiff line numberDiff line change
@@ -367,6 +367,11 @@ public function getURI() {
367367
return "/project/view/{$id}/";
368368
}
369369

370+
public function getProfileURI() {
371+
$id = $this->getID();
372+
return "/project/profile/{$id}/";
373+
}
374+
370375
public function save() {
371376
if (!$this->getMailKey()) {
372377
$this->setMailKey(Filesystem::readRandomCharacters(20));

‎webroot/rsrc/css/phui/workboards/phui-workboard-color.css

-16
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@
22
* @provides phui-workboard-color-css
33
*/
44

5-
.phui-workboard-color .phabricator-nav-content .phui-workboard-view-shadow {
6-
background-color: transparent;
7-
}
8-
95
.phui-workboard-color .phui-crumbs-view {
106
background-color: rgba({$alphagrey},.15);
117
border: none;
@@ -30,18 +26,6 @@
3026
background-color: rgba({$alphawhite},.6);
3127
}
3228

33-
body.phui-workboard-color .phui-profile-menu .phabricator-side-menu {
34-
background-color: rgba({$alphagrey},.3);
35-
}
36-
37-
body.phui-workboard-color .phabricator-side-menu .phui-profile-menu-footer-1 {
38-
background-color: transparent;
39-
}
40-
41-
.phui-workboard-color .phui-profile-menu .phabricator-side-menu {
42-
box-shadow: none;
43-
}
44-
4529
.phui-workboard-color-preview {
4630
width: 50px;
4731
height: 50px;

‎webroot/rsrc/css/phui/workboards/phui-workboard.css

+3-19
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
left: 0;
2020
right: 0;
2121
padding: 16px;
22-
background-color: #fff;
2322
}
2423

2524
.phui-workboard-view-shadow::-webkit-scrollbar {
@@ -32,15 +31,6 @@
3231
background: {$lightbluetext};
3332
}
3433

35-
.device-desktop .project-board-wrapper .phui-workboard-view-shadow {
36-
left: {$menu.profile.width};
37-
}
38-
39-
.device-desktop .phui-profile-menu-collapsed .project-board-wrapper
40-
.phui-workboard-view-shadow {
41-
left: {$menu.profile.width.collapsed};
42-
}
43-
4434
!print .project-board-wrapper .phui-workboard-view-shadow {
4535
position: static;
4636
}
@@ -75,21 +65,15 @@
7565
display: none;
7666
}
7767

78-
.device-desktop .phui-workboard-fullscreen .phui-profile-menu
79-
.phui-workboard-view-shadow {
80-
top: 35px;
81-
left: 0;
68+
.device-desktop .phui-workboard-fullscreen .phui-workboard-view-shadow {
69+
top: 35px;
70+
left: 0;
8271
}
8372

8473
.device-desktop .phui-workboard-fullscreen .phui-workpanel-body-content {
8574
max-height: calc(100vh - 120px);
8675
}
8776

88-
.device-desktop .phui-workboard-fullscreen .phui-profile-menu
89-
.phabricator-nav-local {
90-
display: none;
91-
}
92-
9377
.device .phui-workboard-expand-icon {
9478
display: none;
9579
}

0 commit comments

Comments
 (0)
Failed to load comments.