From: W. Trevor King <wking@drexel.edu>
Date: Wed, 21 Jul 2010 18:20:16 +0000 (-0400)
Subject: Use POST for data-changing forms, GET otherwise
X-Git-Tag: v0.1~18
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=26d8e36bdd386c370d5e223d17e300b9dbc7e33e;p=cookbook.git

Use POST for data-changing forms, GET otherwise

Also float the recipe list tag-selection form to the right.
---

diff --git a/cookbook/template/recipe.html b/cookbook/template/recipe.html
index e65a15e..b335014 100644
--- a/cookbook/template/recipe.html
+++ b/cookbook/template/recipe.html
@@ -28,12 +28,12 @@
 	</span><br />
     {% endif %}
     </p>
-    <form action="add_tag" method="get">
+    <form action="add_tag" method="post">
       <input type="hidden" name="name" value="{{ recipe.name }}"/>
       <input type="text" name="tag" value=""/>
       <input type="submit" value="Add tag" />
     </form>
-    <form action="remove_tag" method="get">
+    <form action="remove_tag" method="post">
       <input type="hidden" name="name" value="{{ recipe.name }}"/>
       <input type="text" name="tag" value=""/>
       <input type="submit" value="Remove tag" />
diff --git a/cookbook/template/recipes.html b/cookbook/template/recipes.html
index 52e2fed..a9bfc07 100644
--- a/cookbook/template/recipes.html
+++ b/cookbook/template/recipes.html
@@ -5,12 +5,12 @@
 {% endblock %}
 
 {% block content %}
-    <form action="">
+    <form action="" method="get" style="float:right;">
       <select name="tag" size="5" multiple="multiple">
 	{% for tag in cookbook.tags() %}
 	<option value="{{ tag }}">{{ tag }}</option>
 	{% endfor %}
-      </select>
+      </select><br />
       <input type="submit" value="Show only selected tags" />
     </form>
     <ul id="recipe-list">