From 67c6fdba2982b90e67b99f51aba633c2b9cd0e6f Mon Sep 17 00:00:00 2001
From: Oskar Hahn <mail@oshahn.de>
Date: Wed, 31 Aug 2011 14:17:11 +0200
Subject: [PATCH] use a-tag instead of onclick for nojs use

---
 openslides/agenda/templates/agenda/overview.html | 10 +++++++---
 openslides/agenda/views.py                       |  2 +-
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/openslides/agenda/templates/agenda/overview.html b/openslides/agenda/templates/agenda/overview.html
index cc943ffc6..bc121d5f5 100644
--- a/openslides/agenda/templates/agenda/overview.html
+++ b/openslides/agenda/templates/agenda/overview.html
@@ -55,13 +55,15 @@
             </tr>
             <tr {% if overview %}class="activeline"{% endif %}>
                 <td {% if perms.agenda.can_manage_agenda %}
-                    class="select" onclick="window.location.href='{% url item_activate 0 %}'"
+                    class="select"
                     {% endif %}>
+                    <a href="{% url item_activate 0 %}">
                     {% if overview %}
                         <img class="center" src="/static/images/icons/task-accepted.png" title="{% trans 'Agenda selected' %}">
                     {% else %}
                         {% if perms.agenda.can_manage_agenda %}<img class="center" src="/static/images/icons/task-accepted-grey.png" title="{% trans 'Select agenda' %}">{% endif %}
                     {% endif %}
+                    </a>
                 </td>
                 <td><i>{% trans "Agenda" %} ({{ items|length }} {% trans "items" %})</i></td>
                 {% if perms.agenda.can_manage_agenda %}
@@ -77,13 +79,15 @@
                 {% if item.parent.active and summary %}activesummarychildline{% endif %}
                 {% endif %}">
                 <td {% if perms.agenda.can_manage_agenda %}
-                    class="select" onclick="window.location.href='{% url item_activate item.id %}'"
+                    class="select"
                     {% endif %}>
+                    <a id="activate_link_{{ item.id }}" class="activate_link" href="{% url item_activate item.id %}">
                     {% if item.active %}
                         <img class="center" src="/static/images/icons/task-accepted.png" title="{% trans 'Item selected' %}"></span>
                     {% else %}
                         {% if perms.agenda.can_manage_agenda %}<img class="center" src="/static/images/icons/task-accepted-grey.png" title="{% trans 'Select item' %}"> {% endif %}
                     {% endif %}
+                    </a>
                 </td>
                 <td>
                     {% for p in item.parents %}
@@ -135,7 +139,7 @@
                     {% if item.children.exists %}
                         <a href="{% url item_activate_summary item.id %}"><img src="/static/images/icons/view-list-tree.png" title="{% trans 'Select item overview' %}"></a>
                     {% endif %}
-                    {% endif %} 
+                    {% endif %}
                 </td>
                 {% if perms.agenda.can_manage_agenda %}
                 <td class="tabledrag-hide">
diff --git a/openslides/agenda/views.py b/openslides/agenda/views.py
index 41f2d57dc..bfbde10f2 100644
--- a/openslides/agenda/views.py
+++ b/openslides/agenda/views.py
@@ -43,7 +43,7 @@ def view(request, item_id):
     """
     item = Item.objects.get(id=item_id)
     votes = assignment_votes(item)
-    
+
     return render_to_response('beamer/%s.html' % item.type,
                              {
                                  'item': item.cast(),