launchpad-reviewers team mailing list archive

[Raphaël Badin <raphael.badin@xxxxxxxxxxxxx>]

Raphaël Badin has proposed merging lp:~rvb/maas/maas-fix-node-view-css into lp:maas.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~rvb/maas/maas-fix-node-view-css/+merge/100558

This branch is a very simple cleanup branch that removes the <br /><br /> tags used for padding in node_view.  It uses existing css classes instead.
-- 
https://code.launchpad.net/~rvb/maas/maas-fix-node-view-css/+merge/100558
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~rvb/maas/maas-fix-node-view-css into lp:maas.

=== modified file 'src/maasserver/templates/maasserver/node_view.html'
--- src/maasserver/templates/maasserver/node_view.html	2012-04-02 16:38:56 +0000
+++ src/maasserver/templates/maasserver/node_view.html	2012-04-03 06:45:26 +0000
@@ -6,26 +6,28 @@
 {% block layout-modifiers %}sidebar{% endblock %}
 
 {% block sidebar %}
-  <div class="block size3">
-    {% if can_edit %}
+  {% if can_edit %}
+    <div class="block size3">
       <h4>Node details</h4>
       <a href="{% url 'node-edit' node.system_id %}" class="button secondary">
         Edit node
       </a>
-    {% endif %}
-    {% if form.transition_buttons %}
-      <form id="node_actions" method="post" action=".">
-      {% for transition in form.transition_buttons %}
-        {% if forloop.first %}<br /><br /><h4>Actions</h4>{% endif %}
-        <input class="secondary"
-               type="submit"
-               name="{{ form.input_name }}"
-               value="{{ transition.display }}" />
-        {% if not forloop.last %}<br /><br />{% endif %}
-      {% endfor %}
-      </form>
-    {% endif %}
-  </div>
+    </div>
+  {% endif %}
+  {% if form.transition_buttons %}
+    {% for transition in form.transition_buttons %}
+      {% if forloop.first %}
+      <div class="block size3">
+        <h4>Actions</h4>
+        <form id="node_actions" method="post" action=".">
+      {% endif %}
+      <input class="secondary {% if not forloop.first %}space-top{% endif %}"
+             type="submit"
+             name="{{ form.input_name }}"
+             value="{{ transition.display }}" />
+      {% if forloop.last %}</form></div>{% endif %}
+    {% endfor %}
+  {% endif %}
 {% endblock %}
 
 {% block content %}