From 26d8e36bdd386c370d5e223d17e300b9dbc7e33e Mon Sep 17 00:00:00 2001
From: "W. Trevor King" <wking@drexel.edu>
Date: Wed, 21 Jul 2010 14:20:16 -0400
Subject: [PATCH] Use POST for data-changing forms, GET otherwise

Also float the recipe list tag-selection form to the right.
---
 cookbook/template/recipe.html  | 4 ++--
 cookbook/template/recipes.html | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

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">
-- 
2.26.2