Minor Ponder spacing updates

Summary: Fix an issue where you've already answered, moved the summary section.

Test Plan: Review an answer with a wiki that i've already answered

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin

Differential Revision: https://secure.phabricator.com/D15395
This commit is contained in:
Chad Little
2016-03-03 15:18:41 -08:00
parent e6158391d2
commit 13813d2268
4 changed files with 6 additions and 5 deletions

View File

@@ -92,7 +92,7 @@ return array(
'rsrc/css/application/policy/policy-edit.css' => '815c66f7', 'rsrc/css/application/policy/policy-edit.css' => '815c66f7',
'rsrc/css/application/policy/policy-transaction-detail.css' => '82100a43', 'rsrc/css/application/policy/policy-transaction-detail.css' => '82100a43',
'rsrc/css/application/policy/policy.css' => '957ea14c', 'rsrc/css/application/policy/policy.css' => '957ea14c',
'rsrc/css/application/ponder/ponder-view.css' => '4e321d68', 'rsrc/css/application/ponder/ponder-view.css' => 'fbd45f96',
'rsrc/css/application/project/project-card-view.css' => '9418c97d', 'rsrc/css/application/project/project-card-view.css' => '9418c97d',
'rsrc/css/application/project/project-view.css' => '9ce99f21', 'rsrc/css/application/project/project-view.css' => '9ce99f21',
'rsrc/css/application/releeph/releeph-core.css' => '9b3c5733', 'rsrc/css/application/releeph/releeph-core.css' => '9b3c5733',
@@ -857,7 +857,7 @@ return array(
'policy-css' => '957ea14c', 'policy-css' => '957ea14c',
'policy-edit-css' => '815c66f7', 'policy-edit-css' => '815c66f7',
'policy-transaction-detail-css' => '82100a43', 'policy-transaction-detail-css' => '82100a43',
'ponder-view-css' => '4e321d68', 'ponder-view-css' => 'fbd45f96',
'project-card-view-css' => '9418c97d', 'project-card-view-css' => '9418c97d',
'project-view-css' => '9ce99f21', 'project-view-css' => '9ce99f21',
'releeph-core' => '9b3c5733', 'releeph-core' => '9b3c5733',

View File

@@ -108,9 +108,9 @@ final class PonderQuestionViewController extends PonderController {
'class' => 'ponder-question-content', 'class' => 'ponder-question-content',
), ),
array( array(
$answer_wiki,
$footer, $footer,
$comment_view, $comment_view,
$answer_wiki,
$answers, $answers,
$answer_add_panel, $answer_add_panel,
)); ));

View File

@@ -22,13 +22,14 @@ final class PonderAddAnswerView extends AphrontView {
$authors = mpull($question->getAnswers(), null, 'getAuthorPHID'); $authors = mpull($question->getAnswers(), null, 'getAuthorPHID');
if (isset($authors[$viewer->getPHID()])) { if (isset($authors[$viewer->getPHID()])) {
return id(new PHUIInfoView()) $view = id(new PHUIInfoView())
->setSeverity(PHUIInfoView::SEVERITY_NOTICE) ->setSeverity(PHUIInfoView::SEVERITY_NOTICE)
->setTitle(pht('Already Answered')) ->setTitle(pht('Already Answered'))
->appendChild( ->appendChild(
pht( pht(
'You have already answered this question. You can not answer '. 'You have already answered this question. You can not answer '.
'twice, but you can edit your existing answer.')); 'twice, but you can edit your existing answer.'));
return phutil_tag_div('ponder-add-answer-view', $view);
} }
$info_panel = null; $info_panel = null;

View File

@@ -117,7 +117,7 @@ body .phui-main-column .ponder-question-content .ponder-answer-view
} }
.ponder-answer-section { .ponder-answer-section {
margin-top: 32px; margin-top: 48px;
} }
.ponder-add-answer-header { .ponder-add-answer-header {