From 0215f11d5c734cb818ed7b988b8f73e745e45179 Mon Sep 17 00:00:00 2001
From: Byron Jones <glob@mozilla.com>
Date: Tue, 12 May 2015 21:35:45 +0800
Subject: Bug 1146770: implement comment preview

---
 .../template/en/default/bug_modal/edit.html.tmpl   | 12 ++++----
 .../en/default/bug_modal/new_comment.html.tmpl     | 32 ++++++++++++++++++++--
 2 files changed, 36 insertions(+), 8 deletions(-)

(limited to 'extensions/BugModal/template/en/default')

diff --git a/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl b/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
index dd673537c..67c94f499 100644
--- a/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
+++ b/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
@@ -978,6 +978,13 @@
   INCLUDE bug_modal/activity_stream.html.tmpl stream=bug.activity_stream;
   IF user.id;
     INCLUDE bug_modal/new_comment.html.tmpl;
+  ELSE;
+    %]
+      <div id="new-comment-notice">
+        You need to <a href="show_bug.cgi?id=[% bug.bug_id FILTER none %]&amp;GoAheadAndLogIn=1">log in</a>
+        before you can comment on or make changes to this [% terms.bug %].
+      </div>
+    [%
   END;
 %]
 
@@ -1007,11 +1014,6 @@
     </div>
   </div>
   </form>
-[% ELSE %]
-  <div id="login-required">
-    You need to <a href="show_bug.cgi?id=[% bug.bug_id FILTER none %]&amp;GoAheadAndLogIn=1">log in</a>
-    before you can comment on or make changes to this [% terms.bug %].
-  </div>
 [% END %]
 
 [%# === blocks === %]
diff --git a/extensions/BugModal/template/en/default/bug_modal/new_comment.html.tmpl b/extensions/BugModal/template/en/default/bug_modal/new_comment.html.tmpl
index f6c3bb3fa..17bf38200 100644
--- a/extensions/BugModal/template/en/default/bug_modal/new_comment.html.tmpl
+++ b/extensions/BugModal/template/en/default/bug_modal/new_comment.html.tmpl
@@ -12,8 +12,15 @@
   #%]
 
 <div id="add-comment">
-  <div id="add-comment-label">Add Comment:</div>
-  [% IF user.is_insider && bug.check_can_change_field('longdesc', 0, 1) %]
+
+  [% IF !bug.check_can_change_field('longdesc', 0, 1) %]
+    <div id="new-comment-notice">
+      You are not allowed to make an additional comment on this [% terms.bug %].
+    </div>
+    [% RETURN %]
+  [% END %]
+
+  [% IF user.is_insider %]
     <div id="add-comment-private"
       title="Make comment visible only to members of the '[% Param('insidergroup') FILTER html %]' group"
     >
@@ -22,7 +29,26 @@
       <label for="add-comment-private-cb">Private</label>
     </div>
   [% END %]
-  <textarea rows="5" cols="80" name="comment" id="comment"></textarea>
+
+  <ul id="comment-tabs">
+    <li id="comment-edit-tab" data-tab="comment-tab-add" data-focus="comment" class="current">
+      Add Comment
+    </li>
+    [%~ ~%]
+    <li id="comment-preview-tab" data-tab="comment-tab-preview" data-focus="">
+      Preview
+      <img id="preview-throbber" src="extensions/BugModal/web/throbber.gif" width="16" height="11" style="display:none">
+    </li>
+  </ul>
+
+  <div id="comment-tab-add" class="comment-tab">
+    <textarea rows="5" cols="80" name="comment" id="comment"></textarea>
+  </div>
+  <div id="comment-tab-preview" class="comment-tab" style="display:none">
+    <pre id="comment-preview" class="comment-text"></pre>
+    </div>
+  </div>
+
   <div id="after-comment-commit-button">
     [% Hook.process("after_comment_commit_button", 'bug/edit.html.tmpl') %]
 
-- 
cgit v1.2.3-24-g4f1b