Differential D17326 Diff 41682 src/applications/dashboard/engine/PhabricatorDashboardRenderingEngine.php
Changeset View
Changeset View
Standalone View
Standalone View
src/applications/dashboard/engine/PhabricatorDashboardRenderingEngine.php
Show First 20 Lines • Show All 75 Lines • ▼ Show 20 Lines | foreach ($panel_grid_locations as $column => $panel_column_locations) { | ||||
$result->addColumn( | $result->addColumn( | ||||
$column_result, | $column_result, | ||||
$column_class, | $column_class, | ||||
$sigil = 'dashboard-column', | $sigil = 'dashboard-column', | ||||
$metadata = array('columnID' => $column)); | $metadata = array('columnID' => $column)); | ||||
} | } | ||||
if ($this->arrangeMode) { | if ($this->arrangeMode) { | ||||
$footer = null; | |||||
Javelin::initBehavior( | Javelin::initBehavior( | ||||
'dashboard-move-panels', | 'dashboard-move-panels', | ||||
array( | array( | ||||
'dashboardID' => $dashboard_id, | 'dashboardID' => $dashboard_id, | ||||
'moveURI' => '/dashboard/movepanel/'.$dashboard->getID().'/', | 'moveURI' => '/dashboard/movepanel/'.$dashboard->getID().'/', | ||||
)); | )); | ||||
} else { | |||||
$name = $dashboard->getName(); | |||||
$icon = id(new PHUIIconView()) | |||||
->setIcon($dashboard->getIcon()) | |||||
->addClass('msr'); | |||||
$footer_left = phutil_tag( | |||||
'a', | |||||
array( | |||||
'class' => 'dashboard-footer-name', | |||||
'href' => '/dashboard/view/'.$dashboard->getID().'/', | |||||
), | |||||
array( | |||||
$icon, | |||||
$name, | |||||
)); | |||||
$footer = phutil_tag( | |||||
'div', | |||||
array( | |||||
'class' => 'dashboard-footer-view', | |||||
), | |||||
array( | |||||
$footer_left, | |||||
)); | |||||
epriestley: Yeah this thing is the confusing thing I think | |||||
Not Done Inline ActionsIt's footer-crumbs. Provides link to dashboard for editing. I may keep it I may not, but I wanted to start here, maybe I need to explicitly add more links to it? chad: It's footer-crumbs. Provides link to dashboard for editing. I may keep it I may not, but I… | |||||
} | } | ||||
$view = id(new PHUIBoxView()) | $view = id(new PHUIBoxView()) | ||||
->addClass('dashboard-view') | ->addClass('dashboard-view') | ||||
->appendChild($result); | ->appendChild($result) | ||||
->appendChild($footer); | |||||
return $view; | return $view; | ||||
} | } | ||||
private function renderAddPanelPlaceHolder($column) { | private function renderAddPanelPlaceHolder($column) { | ||||
$dashboard = $this->dashboard; | $dashboard = $this->dashboard; | ||||
$panels = $dashboard->getPanels(); | $panels = $dashboard->getPanels(); | ||||
Show All 15 Lines | private function renderAddPanelUI($column) { | ||||
$add_uri = id(new PhutilURI('/dashboard/addpanel/'.$dashboard_id.'/')) | $add_uri = id(new PhutilURI('/dashboard/addpanel/'.$dashboard_id.'/')) | ||||
->setQueryParam('column', $column); | ->setQueryParam('column', $column); | ||||
$create_button = id(new PHUIButtonView()) | $create_button = id(new PHUIButtonView()) | ||||
->setTag('a') | ->setTag('a') | ||||
->setHref($create_uri) | ->setHref($create_uri) | ||||
->setWorkflow(true) | ->setWorkflow(true) | ||||
->setColor(PHUIButtonView::GREY) | |||||
->setText(pht('Create Panel')) | ->setText(pht('Create Panel')) | ||||
->addClass(PHUI::MARGIN_MEDIUM); | ->addClass(PHUI::MARGIN_MEDIUM); | ||||
$add_button = id(new PHUIButtonView()) | $add_button = id(new PHUIButtonView()) | ||||
->setTag('a') | ->setTag('a') | ||||
->setHref($add_uri) | ->setHref($add_uri) | ||||
->setWorkflow(true) | ->setWorkflow(true) | ||||
->setColor(PHUIButtonView::GREY) | |||||
->setText(pht('Add Existing Panel')) | ->setText(pht('Add Existing Panel')) | ||||
->addClass(PHUI::MARGIN_MEDIUM); | ->addClass(PHUI::MARGIN_MEDIUM); | ||||
return phutil_tag( | return phutil_tag( | ||||
'div', | 'div', | ||||
array( | array( | ||||
'style' => 'text-align: center;', | 'style' => 'text-align: center;', | ||||
), | ), | ||||
array( | array( | ||||
$create_button, | $create_button, | ||||
$add_button, | $add_button, | ||||
)); | )); | ||||
} | } | ||||
} | } |
Yeah this thing is the confusing thing I think