From 0ba195f2efdacb674c8ff7affbea353abbe2a936 Mon Sep 17 00:00:00 2001
From: michal <michal.sipa@mirumee.com>
Date: Fri, 6 Oct 2017 14:22:50 +0200
Subject: Further refactoring of sort_by and sort_menu

Combine sort_by and sort_menu to one template and templatetag.
Move templatetags to shop.py.
Delete sort_menu.py and _sort_menu.html.
Update index.html with new templatetags.
---
 templates/category/_sort_by.html | 66 +++++++++++++++++++++++++++-------------
 1 file changed, 45 insertions(+), 21 deletions(-)

(limited to 'templates/category/_sort_by.html')

diff --git a/templates/category/_sort_by.html b/templates/category/_sort_by.html
index e0e4cf88..4f2f9a8c 100644
--- a/templates/category/_sort_by.html
+++ b/templates/category/_sort_by.html
@@ -1,27 +1,51 @@
+{% load staticfiles %}
 {% load i18n %}
 {% load shop %}
-{% load staticfiles %}
 
-<li>
-  <div class="row">
-    <div class="col-6">
-      Sort by: <strong>{{ label }}</strong>
+<div class="sort-by">
+  <div class="click-area d-none"></div>
+  <button class="btn btn-link">
+    <div>
+      <span>
+        Sort by:&nbsp;
+        <strong>
+          {{ sort_by }}
+        </strong>
+      </span>
+      <div class="sort-order-icon">
+        {% if arrow_down %}
+          <svg data-src="{% static "assets/arrow_down.svg" %}">
+        {% else %}
+          <svg data-src="{% static "assets/arrow_up.svg" %}">
+        {% endif %}
+      </div>
     </div>
-    <div class="col-6">
-      <div>
-        <a href="{% get_sort_by_url ascending_attribute %}">
-          <span>{% trans 'ascending' context 'sort by option' %}</span>
-          <div class="sort-order-icon float-right">
-            <img src="{% static "assets/arrow_up.svg" %}">
+  </button>
+  <ul class="sort-list d-none">
+    {% for choice in sort_by_choices %}
+      <li>
+        <div class="row">
+          <div class="col-6">
+            Sort by: <strong>{{ choice.label }}</strong>
           </div>
-        </a>
-        <a href="{% get_sort_by_url descending_attribute %}">
-          <span>{% trans 'descending' context 'sort by option' %}</span>
-          <div class="sort-order-icon float-right">
-            <img src="{% static "assets/arrow_down.svg" %}">
+          <div class="col-6">
+            <div>
+              <a href="{% get_sort_by_url choice.value %}">
+                <span>{% trans 'ascending' context 'sort by option' %}</span>
+                <div class="sort-order-icon float-right">
+                  <img src="{% static "assets/arrow_up.svg" %}">
+                </div>
+              </a>
+              <a href="{% get_sort_by_url choice.value descending=True %}">
+                <span>{% trans 'descending' context 'sort by option' %}</span>
+                <div class="sort-order-icon float-right">
+                  <img src="{% static "assets/arrow_down.svg" %}">
+                </div>
+              </a>
+            </div>
           </div>
-        </a>
-      </div>
-    </div>
-  </div>
-</li>
+        </div>
+      </li>
+    {% endfor %}
+  </ul>
+</div>
-- 
cgit v1.2.3