From 29977c2c173d40d40e0302561741c2c9b3ab9368 Mon Sep 17 00:00:00 2001
From: Fabrice Bellingard <bellingard@gmail.com>
Date: Wed, 25 Jan 2012 12:21:06 +0100
Subject: [PATCH] SONAR-1492 Improve rendering

---
 .../_active_rule_note.html.erb                 | 18 +++++++++---------
 .../rules_configuration/_rule_note.html.erb    | 16 ++++++++--------
 2 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_active_rule_note.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_active_rule_note.html.erb
index f85e08f1d7b..e93a22b73ab 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_active_rule_note.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_active_rule_note.html.erb
@@ -35,18 +35,17 @@
 
 <% if is_admin %>
 <div id="<%= active_note_form_div_id -%>" style="display: none" class="admin">
-  <table>
+  <%= form_remote_tag :url => {:action => 'update_active_rule_note', :active_rule_id => active_rule.id},
+                      :update => "active_rule_note_#{active_rule.id}" %>
+  <table style="width:100%">
     <tbody>
       <tr>
         <td class="max-width">
-          <%= form_remote_tag :url => {:action => 'update_active_rule_note', :active_rule_id => active_rule.id},
-                              :update => "active_rule_note_#{active_rule.id}" %>
-            <textarea name="text" id="<%= active_note_textarea_id -%>" rows="10" cols="100%"
-                      onkeyup="if (this.value=='') $('<%= submit_active_note_update_button_id -%>').disabled=true; else $('<%= submit_active_note_update_button_id -%>').disabled=false;"><%= h(note.plain_text) if note -%></textarea>
-            <br/>
-            <input type="submit" value="<%= note ? message('update_verb') : message('rules_configuration.add_note') -%>" name="commit" id="<%= submit_active_note_update_button_id -%>"/>
-            <a href="#" onclick="$('<%= active_note_detail_div_id -%>').show();$('<%= active_note_form_div_id -%>').hide();return false;"><%= message('cancel') %></a>
-          </form>
+          <textarea name="text" id="<%= active_note_textarea_id -%>" rows="10" style="width:100%"
+                    onkeyup="if (this.value=='') $('<%= submit_active_note_update_button_id -%>').disabled=true; else $('<%= submit_active_note_update_button_id -%>').disabled=false;"><%= h(note.plain_text) if note -%></textarea>
+          <br/>
+          <input type="submit" value="<%= note ? message('update_verb') : message('rules_configuration.add_note') -%>" name="commit" id="<%= submit_active_note_update_button_id -%>"/>
+          <a href="#" onclick="$('<%= active_note_detail_div_id -%>').show();$('<%= active_note_form_div_id -%>').hide();return false;"><%= message('cancel') %></a>
         </td>
         <td class="markdown-help">
           <%= render :partial => 'markdown/help' -%>
@@ -54,5 +53,6 @@
       </tr>
     </tbody>
   </table>
+  </form>
 </div>
 <% end %>
\ No newline at end of file
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule_note.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule_note.html.erb
index ecda4d7b332..cdb46cf6c7f 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule_note.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule_note.html.erb
@@ -28,20 +28,19 @@
 
 <% if is_admin %>
 <div id="<%= note_form_div_id -%>" style="display: none" class="admin">
-  <table>
+  <%= form_remote_tag :url => {:action => 'update_rule_note', :rule_id => rule.id},
+                      :update => "rule_note_#{rule.id}" %>
+  <table style="width:100%">
     <tbody>
       <tr>
         <td style="padding-bottom:20px;" colspan="2"><%= rule.description %></td>
       </tr>
       <tr>
         <td class="max-width">
-          <%= form_remote_tag :url => {:action => 'update_rule_note', :rule_id => rule.id},
-                              :update => "rule_note_#{rule.id}" %>
-            <textarea name="text" id="<%= note_textarea_id -%>" rows="10" cols="100%"><%= h(note.plain_text) if note -%></textarea>
-            <br/>
-            <input type="submit" value="<%= message('rules_configuration.update_description') -%>" name="commit" id="<%= submit_note_update_button_id -%>"/>
-            <a href="#" onclick="$('<%= note_detail_div_id -%>').show();$('<%= note_form_div_id -%>').hide();return false;"><%= message('cancel') %></a>
-          </form>
+          <textarea name="text" id="<%= note_textarea_id -%>" rows="10" style="width:100%"><%= h(note.plain_text) if note -%></textarea>
+          <br/>
+          <input type="submit" value="<%= message('rules_configuration.update_description') -%>" name="commit" id="<%= submit_note_update_button_id -%>"/>
+          <a href="#" onclick="$('<%= note_detail_div_id -%>').show();$('<%= note_form_div_id -%>').hide();return false;"><%= message('cancel') %></a>
         </td>
         <td class="markdown-help">
           <%= render :partial => 'markdown/help' -%>
@@ -49,5 +48,6 @@
       </tr>
     </tbody>
   </table>
+  </form>
 </div>
 <% end %>
-- 
2.39.5