Disable full screen mode for inline comments

Summary:
Full screen is a little foobar so disabling it for inline comments
Fixes T5272

Test Plan:
View inline comment after change, make sure full screen option
has gone.

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: epriestley, Korvin

Maniphest Tasks: T5272

Differential Revision: https://secure.phabricator.com/D9579

Author: garoevans

src/infrastructure/diff/PhabricatorInlineCommentController.php

@@ -243,7 +243,8 @@ private function renderTextArea($text) {
       ->setUser($this->getRequest()->getUser())
       ->setSigil('differential-inline-comment-edit-textarea')
       ->setName('text')
-      ->setValue($text);
+      ->setValue($text)
+      ->setDisableFullScreen(true);
   }
 
 }

src/view/form/control/PhabricatorRemarkupControl.php

@@ -3,11 +3,18 @@
 final class PhabricatorRemarkupControl extends AphrontFormTextAreaControl {
   private $disableMacro = false;
 
+  private $disableFullScreen = false;
+
   public function setDisableMacros($disable) {
     $this->disableMacro = $disable;
     return $this;
   }
 
+  public function setDisableFullScreen($disable) {
+    $this->disableFullScreen = $disable;
+    return $this;
+  }
+
   protected function renderInput() {
     $id = $this->getID();
     if (!$id) {
@@ -90,15 +97,17 @@ protected function renderInput() {
         'href'  => PhabricatorEnv::getDoclink('Remarkup Reference'),
       );
 
-    $actions[] = array(
-      'spacer' => true,
-      'align' => 'right',
-    );
+    if (!$this->disableFullScreen) {
+      $actions[] = array(
+        'spacer' => true,
+        'align' => 'right',
+      );
 
-    $actions['fa-arrows-alt'] = array(
-      'tip' => pht('Fullscreen Mode'),
-      'align' => 'right',
-    );
+      $actions['fa-arrows-alt'] = array(
+        'tip' => pht('Fullscreen Mode'),
+        'align' => 'right',
+      );
+    }
 
     $buttons = array();
     foreach ($actions as $action => $spec) {