Page MenuHomePhabricator

D14582.id35283.diff
No OneTemporary

D14582.id35283.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' => '91bbffc2',
+ 'core.pkg.css' => 'ef642419',
'core.pkg.js' => '47dc9ebb',
'darkconsole.pkg.js' => 'e7393ebb',
'differential.pkg.css' => '2de124c9',
@@ -36,7 +36,7 @@
'rsrc/css/application/base/notification-menu.css' => 'f31c0bde',
'rsrc/css/application/base/phabricator-application-launch-view.css' => '95351601',
'rsrc/css/application/base/phui-theme.css' => '6b451f24',
- 'rsrc/css/application/base/standard-page-view.css' => 'a1096ed4',
+ 'rsrc/css/application/base/standard-page-view.css' => '3c99cdf4',
'rsrc/css/application/calendar/calendar-icon.css' => 'c69aa59f',
'rsrc/css/application/chatlog/chatlog.css' => 'd295b020',
'rsrc/css/application/conduit/conduit-api.css' => '7bc725c4',
@@ -103,7 +103,7 @@
'rsrc/css/application/slowvote/slowvote.css' => 'da0afb1b',
'rsrc/css/application/tokens/tokens.css' => '3d0f239e',
'rsrc/css/application/uiexample/example.css' => '528b19de',
- 'rsrc/css/core/core.css' => '78e8d7ea',
+ 'rsrc/css/core/core.css' => 'a76cefc9',
'rsrc/css/core/remarkup.css' => '88e1ebb6',
'rsrc/css/core/syntax.css' => '9fd11da8',
'rsrc/css/core/z-index.css' => '57ddcaa2',
@@ -126,7 +126,7 @@
'rsrc/css/phui/phui-box.css' => 'a5bb366d',
'rsrc/css/phui/phui-button.css' => '16020a60',
'rsrc/css/phui/phui-crumbs-view.css' => '414406b5',
- 'rsrc/css/phui/phui-document-pro.css' => '5f75ed99',
+ 'rsrc/css/phui/phui-document-pro.css' => 'e0fad431',
'rsrc/css/phui/phui-document.css' => 'a4a1c3b9',
'rsrc/css/phui/phui-feed-story.css' => 'b7b26d23',
'rsrc/css/phui/phui-fontkit.css' => '9cda225e',
@@ -727,7 +727,7 @@
'phabricator-busy' => '59a7976a',
'phabricator-chatlog-css' => 'd295b020',
'phabricator-content-source-view-css' => '4b8b05d4',
- 'phabricator-core-css' => '78e8d7ea',
+ 'phabricator-core-css' => 'a76cefc9',
'phabricator-countdown-css' => 'e7544472',
'phabricator-dashboard-css' => 'eb458607',
'phabricator-drag-and-drop-file-upload' => 'ad10aeac',
@@ -755,7 +755,7 @@
'phabricator-side-menu-view-css' => 'bec2458e',
'phabricator-slowvote-css' => 'da0afb1b',
'phabricator-source-code-view-css' => 'cbeef983',
- 'phabricator-standard-page-view' => 'a1096ed4',
+ 'phabricator-standard-page-view' => '3c99cdf4',
'phabricator-textareautils' => '5c93c52c',
'phabricator-title' => 'df5e11d2',
'phabricator-tooltip' => '1d298e3a',
@@ -790,7 +790,7 @@
'phui-calendar-month-css' => '476be7e0',
'phui-crumbs-view-css' => '414406b5',
'phui-document-view-css' => 'a4a1c3b9',
- 'phui-document-view-pro-css' => '5f75ed99',
+ 'phui-document-view-pro-css' => 'e0fad431',
'phui-feed-story-css' => 'b7b26d23',
'phui-font-icon-base-css' => 'ecbbb4c2',
'phui-fontkit-css' => '9cda225e',
diff --git a/src/applications/diviner/controller/DivinerAtomController.php b/src/applications/diviner/controller/DivinerAtomController.php
--- a/src/applications/diviner/controller/DivinerAtomController.php
+++ b/src/applications/diviner/controller/DivinerAtomController.php
@@ -75,8 +75,7 @@
$document = id(new PHUIDocumentViewPro())
->setBook($book->getTitle(), $group_name)
->setHeader($header)
- ->addClass('diviner-view')
- ->setPropertyList($prop_list);
+ ->addClass('diviner-view');
if ($atom) {
$this->buildDefined($properties, $symbol);
@@ -243,10 +242,10 @@
array(
$crumbs,
$document,
+ $prop_list,
),
array(
'title' => $symbol->getTitle(),
- 'class' => 'pro-white-background',
));
}
diff --git a/src/applications/diviner/controller/DivinerBookController.php b/src/applications/diviner/controller/DivinerBookController.php
--- a/src/applications/diviner/controller/DivinerBookController.php
+++ b/src/applications/diviner/controller/DivinerBookController.php
@@ -111,7 +111,6 @@
),
array(
'title' => $book->getTitle(),
- 'class' => 'pro-white-background',
));
}
diff --git a/src/applications/diviner/controller/DivinerMainController.php b/src/applications/diviner/controller/DivinerMainController.php
--- a/src/applications/diviner/controller/DivinerMainController.php
+++ b/src/applications/diviner/controller/DivinerMainController.php
@@ -79,7 +79,6 @@
),
array(
'title' => pht('Documentation Books'),
- 'class' => 'pro-white-background',
));
}
}
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
@@ -331,7 +331,6 @@
),
array(
'title' => $title,
- 'class' => 'pro-white-background',
'pageObjects' => array($document->getPHID()),
));
}
diff --git a/src/applications/meta/controller/PhabricatorApplicationEmailCommandsController.php b/src/applications/meta/controller/PhabricatorApplicationEmailCommandsController.php
--- a/src/applications/meta/controller/PhabricatorApplicationEmailCommandsController.php
+++ b/src/applications/meta/controller/PhabricatorApplicationEmailCommandsController.php
@@ -147,7 +147,6 @@
),
array(
'title' => $title,
- 'class' => 'pro-white-background',
));
}
diff --git a/src/applications/phame/controller/post/PhamePostViewController.php b/src/applications/phame/controller/post/PhamePostViewController.php
--- a/src/applications/phame/controller/post/PhamePostViewController.php
+++ b/src/applications/phame/controller/post/PhamePostViewController.php
@@ -49,8 +49,7 @@
->addActionLink($action_button);
$document = id(new PHUIDocumentViewPro())
- ->setHeader($header)
- ->setPropertyList($properties);
+ ->setHeader($header);
if ($post->isDraft()) {
$document->appendChild(
@@ -94,15 +93,16 @@
$timeline = phutil_tag_div('phui-document-view-pro-box', $timeline);
$add_comment = $this->buildCommentForm($post);
+ $add_comment = phutil_tag_div('mlb mlt', $add_comment);
return $this->newPage()
->setTitle($post->getTitle())
- ->addClass('pro-white-background')
->setPageObjectPHIDs(array($post->getPHID()))
->setCrumbs($crumbs)
->appendChild(
array(
$document,
+ $properties,
$timeline,
$add_comment,
));
@@ -209,12 +209,6 @@
private function buildCommentForm(PhamePost $post) {
$viewer = $this->getViewer();
- $is_serious = PhabricatorEnv::getEnvConfig('phabricator.serious-business');
-
- $add_comment_header = $is_serious
- ? pht('Add Comment')
- : pht('Derp Text');
-
$draft = PhabricatorDraft::newFromUserAndKey(
$viewer, $post->getPHID());
@@ -222,7 +216,7 @@
->setUser($viewer)
->setObjectPHID($post->getPHID())
->setDraft($draft)
- ->setHeaderText($add_comment_header)
+ ->setHeaderText(pht('Add Comment'))
->setAction($this->getApplicationURI('post/comment/'.$post->getID().'/'))
->setSubmitButtonName(pht('Add Comment'));
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
@@ -216,7 +216,6 @@
$page_content = id(new PHUIDocumentViewPro())
->setHeader($header)
- ->setPropertyList($prop_list)
->setToc($toc)
->appendChild(
array(
@@ -229,12 +228,12 @@
array(
$crumbs->render(),
$page_content,
+ $prop_list,
$children,
),
array(
'pageObjects' => array($document->getPHID()),
'title' => $page_title,
- 'class' => 'pro-white-background',
));
}
diff --git a/src/applications/transactions/editengine/PhabricatorEditEngine.php b/src/applications/transactions/editengine/PhabricatorEditEngine.php
--- a/src/applications/transactions/editengine/PhabricatorEditEngine.php
+++ b/src/applications/transactions/editengine/PhabricatorEditEngine.php
@@ -860,7 +860,6 @@
return $controller->newPage()
->setTitle(pht('HTTP Parameters'))
->setCrumbs($crumbs)
- ->addClass('pro-white-background')
->appendChild($document);
}
diff --git a/src/view/phui/PHUIDocumentViewPro.php b/src/view/phui/PHUIDocumentViewPro.php
--- a/src/view/phui/PHUIDocumentViewPro.php
+++ b/src/view/phui/PHUIDocumentViewPro.php
@@ -6,7 +6,6 @@
private $bookname;
private $bookdescription;
private $fluid;
- private $propertyList;
private $toc;
public function setHeader(PHUIHeaderView $header) {
@@ -26,11 +25,6 @@
return $this;
}
- public function setPropertyList($view) {
- $this->propertyList = $view;
- return $this;
- }
-
public function setToc($toc) {
$this->toc = $toc;
return $this;
@@ -39,12 +33,13 @@
protected function getTagAttributes() {
$classes = array();
+ $classes[] = 'phui-document-container';
if ($this->fluid) {
$classes[] = 'phui-document-fluid';
}
return array(
- 'class' => $classes,
+ 'class' => implode(' ', $classes),
);
}
@@ -118,19 +113,13 @@
),
$content_inner);
- $view = phutil_tag(
- 'div',
- array(
- 'class' => implode(' ', $classes),
- ),
- $content);
-
- $list = null;
- if ($this->propertyList) {
- $list = phutil_tag_div('phui-document-properties', $this->propertyList);
- }
+ return phutil_tag(
+ 'div',
+ array(
+ 'class' => implode(' ', $classes),
+ ),
+ $content);
- return array($view, $list);
}
}
diff --git a/webroot/rsrc/css/application/base/standard-page-view.css b/webroot/rsrc/css/application/base/standard-page-view.css
--- a/webroot/rsrc/css/application/base/standard-page-view.css
+++ b/webroot/rsrc/css/application/base/standard-page-view.css
@@ -19,9 +19,9 @@
.phabricator-standard-page-footer {
text-align: right;
- margin: 4px 16px;
+ margin: 32px 16px 16px;
padding: 12px 0;
- border-top: 1px solid {$lightgreyborder};
+ border-top: 1px solid rgba(71, 87, 120, 0.20);
color: {$greytext};
}
diff --git a/webroot/rsrc/css/core/core.css b/webroot/rsrc/css/core/core.css
--- a/webroot/rsrc/css/core/core.css
+++ b/webroot/rsrc/css/core/core.css
@@ -173,7 +173,3 @@
height: 2px;
background: {$sky};
}
-
-html body.pro-white-background {
- background-color: white;
-}
diff --git a/webroot/rsrc/css/phui/phui-document-pro.css b/webroot/rsrc/css/phui/phui-document-pro.css
--- a/webroot/rsrc/css/phui/phui-document-pro.css
+++ b/webroot/rsrc/css/phui/phui-document-pro.css
@@ -2,23 +2,30 @@
* @provides phui-document-view-pro-css
*/
-.phui-document-view.phui-document-view-pro,
-.phui-document-view-pro-box,
-.phui-document-properties {
+.phui-document-view.phui-document-view-pro {
max-width: 800px;
- padding: 0 16px;
+ padding: 16px 16px 64px 16px;
+ margin: 0 auto;
+}
+
+.phui-document-container {
+ background-color: #fff;
position: relative;
- margin: 16px auto;
+ border-bottom: 1px solid #dedee1;
}
+.phui-document-view-pro-box,
.phui-document-properties {
- max-width: 768px;
- background-color: {$lightgreybackground};
+ max-width: 800px;
+ margin: 0 auto;
+}
+
+.phui-property-list-section {
+ max-width: 800px;
margin: 16px auto;
- border-radius: 3px;
}
-.device .phui-document-properties {
+.device .phui-property-list-section {
margin: 0 8px 16px;
}
@@ -29,7 +36,7 @@
.phui-document-view-pro .phui-document-toc {
position: absolute;
- top: 18px;
+ top: 34px;
left: -36px;
}
@@ -49,7 +56,7 @@
position: absolute;
z-index: 30;
background-color: #fff;
- top: 38px;
+ top: 52px;
left: -44px;
}
@@ -131,8 +138,9 @@
.phui-document-view-pro-box .phui-timeline-view {
- padding: 0;
+ padding: 16px 0 0 0;
background: none;
+ border-top: 1px solid rgba(71, 87, 120, 0.20);
}
.phui-document-view-pro-box .phui-timeline-image {
@@ -157,8 +165,10 @@
}
.phui-document-view-pro-box .phui-timeline-title {
- border-radius: 3px;
- background-color: {$lightgreybackground};
+ border-top-right-radius: 3px;
+ border-top-left-radius: 3px;
+ background-color: #fff;
+ border-bottom: 1px solid #F1F1F4;
}
.phui-document-view-pro-box .phui-timeline-title-with-icon {
@@ -171,12 +181,10 @@
.phui-document-view-pro-box .phui-timeline-major-event .phui-timeline-content
.phui-timeline-core-content {
- padding-bottom: 24px;
+
}
.phui-document-view-pro-box .phui-object-box {
- background-color: {$lightgreybackground};
- border: none;
margin: 0;
}

File Metadata

Mime Type
text/plain
Expires
Sat, Feb 22, 7:03 PM (7 h, 38 m)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7178868
Default Alt Text
D14582.id35283.diff (12 KB)

Event Timeline