Made "subscribe" and "hide" in tags as buttons with forms.

boards/static/css/base.css

@@ -79,3 +79,13 b' textarea, input {'
     height: 100px;
     width: 100%;
 }
+
+.post-button-form {
+    display: inline;
+}
+
+.post-button-form > input[type="submit"] {
+    border: none;
+    margin: 0;
+    padding: 0;
+}

boards/static/js/main.js

@@ -81,6 +81,21 b' function translate_time(node) {'
     });
 }
 
+/**
+ * We use buttons viewed as a form with link and hidden inputs to ensure
+ * the search crawlers don't index or click them.
+ *
+ * But we need to submit the form by clicking the links. That's why these
+ * handlers are here.
+ */
+function addPostButtonActions() {
+    $('.post-button-form > a').each(function() {
+        $(this).click(function() {
+            $(this).parent().submit();
+        });
+    });
+}
+
 $( document ).ready(function() {
     hideEmailFromForm();
 
@@ -96,4 +111,6 b' function translate_time(node) {'
     highlightCode($(document));
 
     translate_time(null);
+
+    addPostButtonActions();
 });

boards/templates/boards/posting_general.html

@@ -40,22 +40,25 b''
 {% if tag %}
     <div class="tag_info">
         <h2>
-            {% if is_favorite %}
-                <a href="{% url 'tag' tag.name %}?method=unsubscribe&next={{ request.path }}"
-                   class="fav" rel="nofollow">★</a>
-            {% else %}
-                <a href="{% url 'tag' tag.name %}?method=subscribe&next={{ request.path }}"
-                   class="not_fav" rel="nofollow">★</a>
-            {% endif %}
-            {% if is_hidden %}
-                <a href="{% url 'tag' tag.name %}?method=unhide&next={{ request.path }}"
-                   title="{% trans 'Show tag' %}"
-                   class="fav" rel="nofollow">H</a>
-            {% else %}
-                <a href="{% url 'tag' tag.name %}?method=hide&next={{ request.path }}"
-                   title="{% trans 'Hide tag' %}"
-                   class="not_fav" rel="nofollow">H</a>
-            {% endif %}
+            <form action="{% url 'tag' tag.name %}" method="post" class="post-button-form">
+                <input type="hidden" name="method"
+                {% if is_favorite %}
+                    <input type="hidden" name="method" value="unsubscribe" />
+                    <a class="fav" href="#">★</a>
+                {% else %}
+                    <input type="hidden" name="method" value="subscribe" />
+                    <a class="not_fav" href="#">★</a>
+                {% endif %}
+            </form>
+            <form action="{% url 'tag' tag.name %}" method="post" class="post-button-form">
+                {% if is_hidden %}
+                    <input type="hidden" name="method" value="unhide" />
+                    <a class="fav" href="#">H</a>
+                {% else %}
+                    <input type="hidden" name="method" value="hide" />
+                    <a class="not_fav" href="#">H</a>
+                {% endif %}
+            </form>
             {% autoescape off %}
                 {{ tag.get_view }}
             {% endautoescape %}

boards/views/tag_threads.py

@@ -43,22 +43,25 b' class TagView(AllThreadsView, Dispatcher'
     def get(self, request, tag_name, page=DEFAULT_PAGE, form=None):
         self.tag_name = tag_name
 
-        dispatch_result = self.dispatch_method(request)
-        if dispatch_result:
-            return dispatch_result
-        else:
-            return super(TagView, self).get(request, page, form)
+        return super(TagView, self).get(request, page, form)
+
 
     def post(self, request, tag_name, page=DEFAULT_PAGE):
-        form = ThreadForm(request.POST, request.FILES,
-                          error_class=PlainErrorList)
-        form.session = request.session
+        self.tag_name = tag_name
 
-        if form.is_valid():
-            return self.create_thread(request, form)
-        if form.need_to_ban:
-            # Ban user because he is suspected to be a bot
-            self._ban_current_user(request)
+        if 'method' in request.POST:
+            self.dispatch_method(request)
+            form = None
+        else:
+            form = ThreadForm(request.POST, request.FILES,
+                              error_class=PlainErrorList)
+            form.session = request.session
+
+            if form.is_valid():
+                return self.create_thread(request, form)
+            if form.need_to_ban:
+                # Ban user because he is suspected to be a bot
+                self._ban_current_user(request)
 
         return self.get(request, tag_name, page, form)
 
@@ -68,16 +71,12 b' class TagView(AllThreadsView, Dispatcher'
         settings_manager = get_settings_manager(request)
         settings_manager.add_fav_tag(tag)
 
-        return self.redirect_to_next(request)
-
     def unsubscribe(self, request):
         tag = get_object_or_404(Tag, name=self.tag_name)
 
         settings_manager = get_settings_manager(request)
         settings_manager.del_fav_tag(tag)
 
-        return self.redirect_to_next(request)
-
     def hide(self, request):
         """
         Adds tag to user's hidden tags. Threads with this tag will not be