Fix Remarkup styling in Maniphest and Differential

Summary:
This makes some of the line spacing, paragraph spacing and layout
less terrible. In particular, fixes code blocks inside Differential inline
comments.

Test Plan:
Looked at Maniphest Tasks, Differential Revisions and Differential
inline comments with various flavors of remarkup in them.

Reviewed By: jungejason
Reviewers: tuomaspelkonen, jungejason
CC: jungejason
Differential Revision: 89

src/applications/differential/view/inlinecomment/DifferentialInlineCommentView.php

@@ -140,7 +140,9 @@ final class DifferentialInlineCommentView extends AphrontView {
         '<span class="differential-inline-comment-line">'.$line.'</span>'.
         phutil_escape_html($handles[$inline->getAuthorPHID()]->getName()).
       '</div>'.
-      $content);
+      '<div class="phabricator-remarkup">'.
+        $content.
+      '</div>');
 
     return $this->scaffoldMarkup($markup);
   }

webroot/rsrc/css/application/maniphest/transaction-detail.css

@@ -71,9 +71,3 @@
   padding: 4px 1em;
   background: #f3f3f3;
 }
-
-
-.maniphest-task-description p,
-.maniphest-transaction-comments p {
-  margin: .25em 0 .5em;
-}

webroot/rsrc/css/core/remarkup.css

@@ -2,8 +2,12 @@
  * @provides phabricator-remarkup-css
  */
 
+.phabricator-remarkup {
+  line-height: 1.45em;
+}
+
 .phabricator-remarkup p {
-  margin: 1em 0;
+  margin: 0 0 .75em;
 }
 
 .phabricator-remarkup code {
@@ -28,3 +32,7 @@
   margin: 1em 0 1em 3em;
 }
 
+.phabricator-remarkup h1,
+.phabricator-remarkup h2 {
+  margin: .75em 0 .5em;
+}