From 1345c37789bb56fbe4c6c21344f6fc19846f488d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?David=20C=C3=B4t=C3=A9-Tremblay?=
 <david.cote-tremblay@savoirfairelinux.com>
Date: Fri, 27 Jan 2017 16:30:56 -0500
Subject: [PATCH] New issue : Show watchers in spoiler

---
 app/views/issues/new.html.erb      | 27 +++++++++++++++------------
 public/stylesheets/application.css |  5 +++++
 public/stylesheets/responsive.css  |  4 ++++
 3 files changed, 24 insertions(+), 12 deletions(-)

diff --git a/app/views/issues/new.html.erb b/app/views/issues/new.html.erb
index ec7f262..23e6057 100644
--- a/app/views/issues/new.html.erb
+++ b/app/views/issues/new.html.erb
@@ -33,18 +33,21 @@
     <p id="attachments_form"><label><%= l(:label_attachment_plural) %></label><%= render :partial => 'attachments/form', :locals => {:container => @issue} %></p>
 
     <% if @issue.safe_attribute? 'watcher_user_ids' -%>
-      <p id="watchers_form"><label><%= l(:label_issue_watchers) %></label>
-      <span id="watchers_inputs">
-        <%= watchers_checkboxes(@issue, users_for_new_issue_watchers(@issue)) %>
-      </span>
-      <span class="search_for_watchers">
-      <%= link_to l(:label_search_for_watchers),
-                  {:controller => 'watchers', :action => 'new', :project_id => @issue.project},
-                  :class => 'icon icon-add-bullet',
-                  :remote => true,
-                  :method => 'get' %>
-      </span>
-      </p>
+    <fieldset id="watchers_form" class="collapsible collapsed">
+      <legend onclick="toggleFieldset(this);"><%= l(:label_issue_watchers) %></legend>
+      <div style="display: none;">
+        <span id="watchers_inputs">
+          <%= watchers_checkboxes(@issue, users_for_new_issue_watchers(@issue)) %>
+        </span>
+        <span class="search_for_watchers">
+          <%= link_to l(:label_search_for_watchers),
+                      {:controller => 'watchers', :action => 'new', :project_id => @issue.project},
+                      :class => 'icon icon-add-bullet',
+                      :remote => true,
+                      :method => 'get' %>
+        </span>
+      </div>
+    </fieldset>
     <% end %>
   </div>
 
diff --git a/public/stylesheets/application.css b/public/stylesheets/application.css
index 9cde8f7..4456026 100644
--- a/public/stylesheets/application.css
+++ b/public/stylesheets/application.css
@@ -652,6 +652,11 @@ span.pagination>span {white-space:nowrap;}
   min-height: 2em;
   clear:left;
 }
+.tabular fieldset {
+  padding-left: 163px;
+  min-height: 3em;
+  margin-top: 10px;
+}
 
 html>body .tabular p {overflow:hidden;}
 
diff --git a/public/stylesheets/responsive.css b/public/stylesheets/responsive.css
index 8e7f746..98be7c2 100644
--- a/public/stylesheets/responsive.css
+++ b/public/stylesheets/responsive.css
@@ -698,6 +698,10 @@
     padding-left: 0;
   }
 
+  .tabular fieldset {
+    padding-left: 0;
+  }
+
   .tabular label, .tabular.settings label {
     display: block;
     width: 100%;
-- 
2.1.4