Skip to content

Commit 87304e3

Browse files
author
Chad Little
committedMar 2, 2017
Remove dashboard footer
Summary: Doesn't seem popular, will rethink dashboard editing again in the future at some point. Test Plan: Review a dashboard, edit, install. Reviewers: epriestley Reviewed By: epriestley Subscribers: Korvin Differential Revision: https://secure.phabricator.com/D17450
1 parent 6f7bb8c commit 87304e3

File tree

3 files changed

+4
-44
lines changed

3 files changed

+4
-44
lines changed
 

‎resources/celerity/map.php

+3-3
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
'names' => array(
1010
'conpherence.pkg.css' => '6875302f',
1111
'conpherence.pkg.js' => '6249a1cf',
12-
'core.pkg.css' => 'c0340df0',
12+
'core.pkg.css' => '52a77c4d',
1313
'core.pkg.js' => '1fa7c0c5',
1414
'darkconsole.pkg.js' => 'e7393ebb',
1515
'differential.pkg.css' => '90b30783',
@@ -55,7 +55,7 @@
5555
'rsrc/css/application/contentsource/content-source-view.css' => '4b8b05d4',
5656
'rsrc/css/application/countdown/timer.css' => '16c52f5c',
5757
'rsrc/css/application/daemon/bulk-job.css' => 'df9c1d4a',
58-
'rsrc/css/application/dashboard/dashboard.css' => '0921c307',
58+
'rsrc/css/application/dashboard/dashboard.css' => 'fe5b1869',
5959
'rsrc/css/application/diff/inline-comment-summary.css' => '51efda3a',
6060
'rsrc/css/application/differential/add-comment.css' => 'c47f8c40',
6161
'rsrc/css/application/differential/changeset-view.css' => '41af6d25',
@@ -786,7 +786,7 @@
786786
'phabricator-content-source-view-css' => '4b8b05d4',
787787
'phabricator-core-css' => '9f4cb463',
788788
'phabricator-countdown-css' => '16c52f5c',
789-
'phabricator-dashboard-css' => '0921c307',
789+
'phabricator-dashboard-css' => 'fe5b1869',
790790
'phabricator-drag-and-drop-file-upload' => '58dea2fa',
791791
'phabricator-draggable-list' => 'bea6e7f4',
792792
'phabricator-fatal-config-template-css' => '8f18fa41',

‎src/applications/dashboard/engine/PhabricatorDashboardRenderingEngine.php

+1-27
Original file line numberDiff line numberDiff line change
@@ -81,43 +81,17 @@ public function renderDashboard() {
8181
}
8282

8383
if ($this->arrangeMode) {
84-
$footer = null;
8584
Javelin::initBehavior(
8685
'dashboard-move-panels',
8786
array(
8887
'dashboardID' => $dashboard_id,
8988
'moveURI' => '/dashboard/movepanel/'.$dashboard->getID().'/',
9089
));
91-
} else {
92-
$name = $dashboard->getName();
93-
$icon = id(new PHUIIconView())
94-
->setIcon($dashboard->getIcon())
95-
->addClass('msr');
96-
$footer_left = phutil_tag(
97-
'a',
98-
array(
99-
'class' => 'dashboard-footer-name',
100-
'href' => '/dashboard/view/'.$dashboard->getID().'/',
101-
),
102-
array(
103-
$icon,
104-
$name,
105-
));
106-
107-
$footer = phutil_tag(
108-
'div',
109-
array(
110-
'class' => 'dashboard-footer-view',
111-
),
112-
array(
113-
$footer_left,
114-
));
11590
}
11691

11792
$view = id(new PHUIBoxView())
11893
->addClass('dashboard-view')
119-
->appendChild($result)
120-
->appendChild($footer);
94+
->appendChild($result);
12195

12296
return $view;
12397
}

‎webroot/rsrc/css/application/dashboard/dashboard.css

-14
Original file line numberDiff line numberDiff line change
@@ -106,17 +106,3 @@
106106
.drag-frame .phui-object-box .phui-object-box {
107107
box-shadow: none;
108108
}
109-
110-
/*** Footer *******************************************************************/
111-
112-
.dashboard-footer-view {
113-
background-color: {$page.sidenav};
114-
padding: 8px 16px;
115-
border-radius: 3px;
116-
}
117-
118-
.dashboard-footer-name {
119-
color: {$darkbluetext};
120-
font-weight: bold;
121-
-webkit-font-smoothing: antialiased;
122-
}

0 commit comments

Comments
 (0)
Failed to load comments.