Page MenuHomePhabricator

D9005.id21392.diff
No OneTemporary

D9005.id21392.diff

diff --git a/resources/celerity/map.php b/resources/celerity/map.php
--- a/resources/celerity/map.php
+++ b/resources/celerity/map.php
@@ -7,7 +7,7 @@
return array(
'names' =>
array(
- 'core.pkg.css' => '989eee69',
+ 'core.pkg.css' => '719389b0',
'core.pkg.js' => 'b2ed04a2',
'darkconsole.pkg.js' => 'ca8671ce',
'differential.pkg.css' => '4b8686e3',
@@ -24,7 +24,7 @@
'rsrc/css/aphront/error-view.css' => '9f1d5518',
'rsrc/css/aphront/lightbox-attachment.css' => '7acac05d',
'rsrc/css/aphront/list-filter-view.css' => '2ae43867',
- 'rsrc/css/aphront/multi-column.css' => '667ca7b8',
+ 'rsrc/css/aphront/multi-column.css' => '0bbcd4dd',
'rsrc/css/aphront/notification.css' => 'ef2c9b34',
'rsrc/css/aphront/pager-view.css' => '2e3539af',
'rsrc/css/aphront/panel-view.css' => '5846dfa2',
@@ -132,12 +132,12 @@
'rsrc/css/phui/phui-fontkit.css' => 'de84aa4a',
'rsrc/css/phui/phui-form-view.css' => '867463b4',
'rsrc/css/phui/phui-form.css' => 'b78ec020',
- 'rsrc/css/phui/phui-header-view.css' => '5b79f0ef',
+ 'rsrc/css/phui/phui-header-view.css' => '5d245c9c',
'rsrc/css/phui/phui-icon.css' => '215fa314',
'rsrc/css/phui/phui-info-panel.css' => '27ea50a1',
'rsrc/css/phui/phui-list.css' => 'ef8035b6',
'rsrc/css/phui/phui-object-box.css' => 'ce92d8ec',
- 'rsrc/css/phui/phui-object-item-list-view.css' => '24444832',
+ 'rsrc/css/phui/phui-object-item-list-view.css' => '3bf8e1c9',
'rsrc/css/phui/phui-pinboard-view.css' => 'e7d3b05e',
'rsrc/css/phui/phui-property-list-view.css' => 'af4b381f',
'rsrc/css/phui/phui-remarkup-preview.css' => '19ad512b',
@@ -146,8 +146,8 @@
'rsrc/css/phui/phui-tag-view.css' => '295d81c4',
'rsrc/css/phui/phui-text.css' => '23e9b4b7',
'rsrc/css/phui/phui-timeline-view.css' => '15ff2a9f',
- 'rsrc/css/phui/phui-workboard-view.css' => '84f2c272',
- 'rsrc/css/phui/phui-workpanel-view.css' => 'b6ccc22d',
+ 'rsrc/css/phui/phui-workboard-view.css' => '5f9df642',
+ 'rsrc/css/phui/phui-workpanel-view.css' => '97b69459',
'rsrc/css/sprite-actions.css' => '969ad0e5',
'rsrc/css/sprite-apps-large.css' => '3e3ec4c3',
'rsrc/css/sprite-apps-xlarge.css' => 'db66c878',
@@ -495,7 +495,7 @@
'aphront-dialog-view-css' => 'c01d24b4',
'aphront-error-view-css' => '9f1d5518',
'aphront-list-filter-view-css' => '2ae43867',
- 'aphront-multi-column-view-css' => '667ca7b8',
+ 'aphront-multi-column-view-css' => '0bbcd4dd',
'aphront-pager-view-css' => '2e3539af',
'aphront-panel-view-css' => '5846dfa2',
'aphront-request-failure-view-css' => 'da14df31',
@@ -761,12 +761,12 @@
'phui-fontkit-css' => 'de84aa4a',
'phui-form-css' => 'b78ec020',
'phui-form-view-css' => '867463b4',
- 'phui-header-view-css' => '5b79f0ef',
+ 'phui-header-view-css' => '5d245c9c',
'phui-icon-view-css' => '215fa314',
'phui-info-panel-css' => '27ea50a1',
'phui-list-view-css' => 'ef8035b6',
'phui-object-box-css' => 'ce92d8ec',
- 'phui-object-item-list-view-css' => '24444832',
+ 'phui-object-item-list-view-css' => '3bf8e1c9',
'phui-pinboard-view-css' => 'e7d3b05e',
'phui-property-list-view-css' => 'af4b381f',
'phui-remarkup-preview-css' => '19ad512b',
@@ -775,8 +775,8 @@
'phui-tag-view-css' => '295d81c4',
'phui-text-css' => '23e9b4b7',
'phui-timeline-view-css' => '15ff2a9f',
- 'phui-workboard-view-css' => '84f2c272',
- 'phui-workpanel-view-css' => 'b6ccc22d',
+ 'phui-workboard-view-css' => '5f9df642',
+ 'phui-workpanel-view-css' => '97b69459',
'phuix-action-list-view' => 'b5c256b8',
'phuix-action-view' => '19a0b148',
'phuix-dropdown-menu' => 'bd4c8dca',
diff --git a/src/applications/project/controller/PhabricatorProjectBoardViewController.php b/src/applications/project/controller/PhabricatorProjectBoardViewController.php
--- a/src/applications/project/controller/PhabricatorProjectBoardViewController.php
+++ b/src/applications/project/controller/PhabricatorProjectBoardViewController.php
@@ -150,55 +150,37 @@
$board->addPanel($panel);
}
- $crumbs = $this->buildApplicationCrumbs();
- $crumbs->addTextCrumb(pht('Board'));
-
$can_edit = PhabricatorPolicyFilter::hasCapability(
$viewer,
$project,
PhabricatorPolicyCapability::CAN_EDIT);
- $actions = id(new PhabricatorActionListView())
- ->setUser($viewer)
- ->addAction(
- id(new PhabricatorActionView())
- ->setName(pht('Add Column'))
- ->setHref($this->getApplicationURI('board/'.$this->id.'/edit/'))
- ->setIcon('create')
- ->setDisabled(!$can_edit)
- ->setWorkflow(!$can_edit));
-
- $plist = id(new PHUIPropertyListView());
-
- // TODO: Need this to get actions to render.
- $plist->addProperty(
- pht('Project Boards'),
- phutil_tag(
- 'em',
- array(),
- pht(
- 'This feature is beta, but should mostly work.')));
- $plist->setActionList($actions);
+ $add_icon = id(new PHUIIconView())
+ ->setIconFont('fa-plus bluegrey');
+
+ $add_button = id(new PHUIButtonView())
+ ->setText(pht('Add Column'))
+ ->setIcon($add_icon)
+ ->setTag('a')
+ ->setHref($this->getApplicationURI('board/'.$this->id.'/edit/'))
+ ->setDisabled(!$can_edit)
+ ->setWorkflow(!$can_edit);
$header = id(new PHUIHeaderView())
->setHeader($project->getName())
->setUser($viewer)
+ ->setNoBackground(true)
->setImage($project->getProfileImageURI())
+ ->addActionLink($add_button)
->setPolicyObject($project);
- $box = id(new PHUIObjectBoxView())
- ->setHeader($header)
- ->addPropertyList($plist);
-
$board_box = id(new PHUIBoxView())
->appendChild($board)
- ->addClass('project-board-wrapper')
- ->addMargin(PHUI::MARGIN_LARGE);
+ ->addClass('project-board-wrapper');
return $this->buildApplicationPage(
array(
- $crumbs,
- $box,
+ $header,
$board_box,
),
array(
diff --git a/webroot/rsrc/css/aphront/multi-column.css b/webroot/rsrc/css/aphront/multi-column.css
--- a/webroot/rsrc/css/aphront/multi-column.css
+++ b/webroot/rsrc/css/aphront/multi-column.css
@@ -17,6 +17,11 @@
margin: 0;
}
+.device-desktop .phui-workboard-view-shadow .aphront-multi-column-inner
+ .aphront-multi-column-column-last {
+ margin-right: 8px;
+}
+
.device .aphront-multi-column-outer
div.aphront-multi-column-column-outer {
width: auto;
@@ -28,7 +33,7 @@
}
.aphront-multi-column-fixed .aphront-multi-column-column-outer {
- width: 280px;
+ width: 300px;
}
/* flexible, but with a minimum */
@@ -103,6 +108,10 @@
width: 100%;
}
+.device-desktop .aphront-multi-column-inner {
+ margin: 0 0 12px 12px;
+}
+
/* magic sauce */
.aphront-multi-column-fluid .aphront-multi-column-inner {
table-layout: fixed;
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
@@ -50,6 +50,11 @@
.phui-header-view .phui-header-action-links {
position: absolute;
+ top: 24px;
+ right: 12px;
+}
+
+.phui-object-box .phui-header-view .phui-header-action-links {
top: 5px;
right: 0;
}
diff --git a/webroot/rsrc/css/phui/phui-object-item-list-view.css b/webroot/rsrc/css/phui/phui-object-item-list-view.css
--- a/webroot/rsrc/css/phui/phui-object-item-list-view.css
+++ b/webroot/rsrc/css/phui/phui-object-item-list-view.css
@@ -39,7 +39,6 @@
border-color: {$lightgreyborder};
border-width: 0 0 0 4px;
margin: 5px 0;
- overflow: hidden;
}
.phui-object-item .phui-icon-view {
@@ -51,7 +50,6 @@
border-color: {$lightgreyborder};
border-bottom-color: {$greyborder};
border-width: 1px 1px 1px 0;
- overflow: hidden;
position: relative;
min-height: 29px;
}
diff --git a/webroot/rsrc/css/phui/phui-workboard-view.css b/webroot/rsrc/css/phui/phui-workboard-view.css
--- a/webroot/rsrc/css/phui/phui-workboard-view.css
+++ b/webroot/rsrc/css/phui/phui-workboard-view.css
@@ -6,20 +6,26 @@
width: 100%;
}
-.phui-workboard-view-shadow {
- overflow-x: auto;
- padding-bottom: 8px;
+.device-desktop .phui-workboard-view-shadow {
+ overflow: auto;
}
+/*
+
+May use in the future?
+
.phui-workboard-view-shadow::-webkit-scrollbar {
- height: 12px;
+ height: 10px;
}
.phui-workboard-view-shadow::-webkit-scrollbar-thumb {
background: {$lightbluetext};
border-radius: 10px;
+ margin: 0 8px;
}
+*/
+
.phui-workboard-action-list {
width: 60px;
float: left;
@@ -68,7 +74,7 @@
margin: 0 3px;
}
-.device-phone .project-board-wrapper {
- margin-left: 0;
- margin-right: 0;
+.device-tablet .project-board-wrapper {
+ margin-left: 8px;
+ margin-right: 8px;
}
diff --git a/webroot/rsrc/css/phui/phui-workpanel-view.css b/webroot/rsrc/css/phui/phui-workpanel-view.css
--- a/webroot/rsrc/css/phui/phui-workpanel-view.css
+++ b/webroot/rsrc/css/phui/phui-workpanel-view.css
@@ -53,7 +53,7 @@
}
.aphront-multi-column-fixed .phui-workpanel-body {
- width: 280px;
+ width: 300px;
}
.phui-workpanel-body .phui-object-item-list-view {

File Metadata

Mime Type
text/plain
Expires
Thu, Oct 17, 10:44 PM (3 w, 5 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6723748
Default Alt Text
D9005.id21392.diff (9 KB)

Event Timeline