Make navigation structure more approachable to new users (#1083)

* Move event selector to sidebar

* Unify navigation

* Fix confusing icons

src/pretix/control/templates/pretixcontrol/event/base.html

@@ -2,158 +2,3 @@
 {% load i18n %}
 {% load static %}
 {% block title %}{{ request.event.name }}{% endblock %}
-
-{% block nav %}
-    <li>
-        <a href="{% url 'control:event.index' organizer=request.event.organizer.slug event=request.event.slug %}"
-                {% if url_name == "event.index" %}class="active"{% endif %}>
-            <i class="fa fa-dashboard fa-fw"></i>
-            {% trans "Event dashboard" %}
-        </a>
-    </li>
-    {% if 'can_change_event_settings' in request.eventpermset %}
-        <li>
-            <a href="{% url 'control:event.settings' organizer=request.event.organizer.slug event=request.event.slug %}"
-                    {% if is_event_settings or "event.settings" == url_name or "event.settings." in url_name %}class="active"{% endif %}>
-                <i class="fa fa-wrench fa-fw"></i>
-                {% trans "Settings" %}
-            </a>
-        </li>
-        {% if request.event.has_subevents %}
-            <li>
-                <a href="{% url 'control:event.subevents' organizer=request.event.organizer.slug event=request.event.slug %}"
-                        {% if "event.subevent" in url_name %}class="active"{% endif %}>
-                    <i class="fa fa-calendar fa-fw"></i>
-                    {% trans "Dates" context "subevent" %}
-                </a>
-            </li>
-        {% endif %}
-    {% endif %}
-    {% if 'can_change_items' in request.eventpermset %}
-        <li>
-            <a href="{% url 'control:event.items' organizer=request.event.organizer.slug event=request.event.slug %}"
-                class="has-children">
-                <i class="fa fa-ticket fa-fw"></i>
-                {% trans "Products" %}
-            </a>
-            <a href="#" class="arrow">
-                <span class="fa arrow"></span>
-            </a>
-            <ul class="nav nav-second-level">
-                <li>
-                    <a href="{% url 'control:event.items' organizer=request.event.organizer.slug event=request.event.slug %}"
-                            {% if "event.items" == url_name or "event.item." in url_name or "event.items.add" == url_name or url_name == "event.item" %}class="active"{% endif %}>
-                        {% trans "Products" %}</a>
-                </li>
-                <li>
-                    <a href="{% url 'control:event.items.quotas' organizer=request.event.organizer.slug event=request.event.slug %}"
-                            {% if "event.items.quotas" in url_name %}class="active"{% endif %}>
-                        {% trans "Quotas" %}
-                    </a>
-                </li>
-                <li>
-                    <a href="{% url 'control:event.items.categories' organizer=request.event.organizer.slug event=request.event.slug %}"
-                            {% if "event.items.categories" in url_name %}class="active"{% endif %}>
-                        {% trans "Categories" %}
-                    </a>
-                </li>
-                <li>
-                    <a href="{% url 'control:event.items.questions' organizer=request.event.organizer.slug event=request.event.slug %}"
-                            {% if "event.items.questions" in url_name %}class="active"{% endif %}>
-                        {% trans "Questions" %}
-                    </a>
-                </li>
-            </ul>
-        </li>
-    {% endif %}
-    {% if 'can_view_orders' in request.eventpermset %}
-        <li>
-            <a href="{% url 'control:event.orders' organizer=request.event.organizer.slug event=request.event.slug %}"
-                class="has-children">
-                <i class="fa fa-shopping-cart fa-fw"></i>
-                {% trans "Orders" %}
-            </a>
-            <a href="#" class="arrow">
-                <span class="fa arrow"></span>
-            </a>
-            <ul class="nav nav-second-level">
-                <li>
-                    <a href="{% url 'control:event.orders' organizer=request.event.organizer.slug event=request.event.slug %}"
-                            {% if url_name == "event.orders" or "event.order." in url_name or url_name == "event.order" %}class="active"{% endif %}>
-                        {% trans "All orders" %}
-                    </a>
-                </li>
-                <li>
-                    <a href="{% url 'control:event.orders.overview' organizer=request.event.organizer.slug event=request.event.slug %}"
-                            {% if url_name == "event.orders.overview" %}class="active"{% endif %}>
-                        {% trans "Overview" %}
-                    </a>
-                </li>
-                <li>
-                    <a href="{% url 'control:event.orders.refunds' organizer=request.event.organizer.slug event=request.event.slug %}"
-                            {% if url_name == "event.orders.refunds" %}class="active"{% endif %}>
-                        {% trans "Refunds" %}
-                    </a>
-                </li>
-                <li>
-                    <a href="{% url 'control:event.orders.export' organizer=request.event.organizer.slug event=request.event.slug %}"
-                            {% if url_name == "event.orders.export" %}class="active"{% endif %}>
-                        {% trans "Export" %}
-                    </a>
-                </li>
-                <li>
-                    <a href="{% url 'control:event.orders.waitinglist' organizer=request.event.organizer.slug event=request.event.slug %}"
-                       {% if url_name == "event.orders.waitinglist" %}class="active"{% endif %}>
-                        {% trans "Waiting list" %}
-                    </a>
-                </li>
-            </ul>
-        </li>
-    {% endif %}
-    {% if 'can_view_vouchers' in request.eventpermset %}
-        <li>
-            <a href="{% url 'control:event.vouchers' organizer=request.event.organizer.slug event=request.event.slug %}"
-                    {% if url_name == "event.voucher" %}class="active"{% endif %}>
-                <i class="fa fa-tags fa-fw"></i>
-                {% trans "Vouchers" %}
-            </a>
-        </li>
-    {% endif %}
-    {% if 'can_view_orders' in request.eventpermset %}
-        <li>
-            <a href="{% url 'control:event.orders.checkinlists' organizer=request.event.organizer.slug event=request.event.slug %}"
-                    {% if "event.orders.checkin" in url_name %}class="active"{% endif %}>
-                <i class="fa fa-check-square-o fa-fw"></i>
-                {% trans "Check-in lists" %}
-            </a>
-        </li>
-    {% endif %}
-    {% for nav in nav_event %}
-        <li>
-            <a href="{{ nav.url }}" {% if nav.active %}class="active"{% endif %}
-            {% if nav.children %}class="has-children"{% endif %}>
-                {% if nav.icon and "." in nav.icon %}
-                    <img src="{% static nav.icon %}" class="fa-img">
-                {% elif nav.icon %}
-                    <i class="fa fa-{{ nav.icon }} fa-fw"></i>
-                {% endif %}
-                {{ nav.label }}
-            </a>
-            {% if nav.children %}
-                <a href="#" class="arrow">
-                    <span class="fa arrow"></span>
-                </a>
-                <ul class="nav nav-second-level">
-                {% for item in nav.children %}
-                    <li>
-                        <a href="{{ item.url }}"
-                        {% if item.active %}class="active"{% endif %}>
-                            {{ item.label }}
-                        </a>
-                    </li>
-                {% endfor %}
-                </ul>
-            {% endif %}
-        </li>
-    {% endfor %}
-{% endblock %}

src/pretix/control/templates/pretixcontrol/event/display.html

@@ -3,6 +3,7 @@
 {% load bootstrap3 %}
 {% load hierarkey_form %}
 {% block inside %}
+    <h1>{% trans "Display settings" %}</h1>
     <form action="" method="post" class="form-horizontal" enctype="multipart/form-data">
         {% csrf_token %}
         {% bootstrap_form_errors form %}

src/pretix/control/templates/pretixcontrol/event/invoicing.html

@@ -2,11 +2,12 @@
 {% load i18n %}
 {% load bootstrap3 %}
 {% block inside %}
+    <h1>{% trans "Invoice settings" %}</h1>
     <form action="" method="post" class="form-horizontal" enctype="multipart/form-data">
         {% csrf_token %}
         {% bootstrap_form_errors form %}
         <fieldset>
-            <legend>{% trans "Invoice settings" %}</legend>
+            <legend>{% trans "General settings" %}</legend>
             {% bootstrap_field form.invoice_generate layout="control" %}
             {% bootstrap_field form.invoice_email_attachment layout="control" %}
             {% bootstrap_field form.invoice_numbers_prefix layout="control" %}

src/pretix/control/templates/pretixcontrol/event/mail.html

@@ -3,12 +3,13 @@
 {% load bootstrap3 %}
 {% load static %}
 {% block inside %}
+    <h1>{% trans "E-mail settings" %}</h1>
     <form action="" method="post" class="form-horizontal" enctype="multipart/form-data"
           mail-preview-url="{% url "control:event.settings.mail.preview" event=request.event.slug organizer=request.event.organizer.slug %}">
         {% csrf_token %}
         {% bootstrap_form_errors form %}
         <fieldset>
-            <legend>{% trans "E-mail settings" %}</legend>
+            <legend>{% trans "General settings" %}</legend>
             {% bootstrap_field form.mail_prefix layout="control" %}
             {% bootstrap_field form.mail_from layout="control" %}
             {% bootstrap_field form.mail_text_signature layout="control" %}

src/pretix/control/templates/pretixcontrol/event/payment.html

@@ -2,6 +2,7 @@
 {% load i18n %}
 {% load bootstrap3 %}
 {% block inside %}
+    <h1>{% trans "Payment settings" %}</h1>
     <form action="" method="post" class="form-horizontal form-plugins">
         {% csrf_token %}
         <fieldset>

src/pretix/control/templates/pretixcontrol/event/plugins.html

@@ -2,10 +2,10 @@
 {% load i18n %}
 {% load bootstrap3 %}
 {% block inside %}
+    <h1>{% trans "Installed plugins" %}</h1>
 	<form action="" method="post" class="form-horizontal form-plugins">
 		{% csrf_token %}
 		<fieldset>
-			<legend>{% trans "Installed plugins" %}</legend>
 			{% if "success" in request.GET %}
 				<div class="alert alert-success">
 					{% trans "Your changes have been saved." %}

src/pretix/control/templates/pretixcontrol/event/settings.html

@@ -2,6 +2,7 @@
 {% load i18n %}
 {% load bootstrap3 %}
 {% block inside %}
+    <h1>{% trans "General settings" %}</h1>
     <form action="" method="post" class="form-horizontal">
         {% csrf_token %}
         {% bootstrap_form_errors form %}

src/pretix/control/templates/pretixcontrol/event/settings_base.html

@@ -27,68 +27,6 @@
         </div>
     {% endif %}
 
-    <h1>{% trans "Settings" %}</h1>
-    <ul class="nav nav-pills">
-        {% if 'can_change_event_settings' in request.eventpermset %}
-            <li {% if "event.settings" == url_name %}class="active"{% endif %}>
-                <a href="{% url 'control:event.settings' organizer=request.event.organizer.slug event=request.event.slug %}">
-                    {% trans "General" %}
-                </a>
-            </li>
-            <li {% if "event.settings.payment" == url_name %}class="active"{% endif %}>
-                <a href="{% url 'control:event.settings.payment' organizer=request.event.organizer.slug event=request.event.slug %}">
-                    {% trans "Payment" %}
-                </a>
-            </li>
-            <li {% if "event.settings.plugins" == url_name %}class="active"{% endif %}>
-                <a href="{% url 'control:event.settings.plugins' organizer=request.event.organizer.slug event=request.event.slug %}">
-                    {% trans "Plugins" %}
-                </a>
-            </li>
-            <li {% if "event.settings.display" == url_name %}class="active"{% endif %}>
-                <a href="{% url 'control:event.settings.display' organizer=request.event.organizer.slug event=request.event.slug %}">
-                    {% trans "Display" %}
-                </a>
-            </li>
-            <li {% if "event.settings.tickets" == url_name %}class="active"{% endif %}>
-                <a href="{% url 'control:event.settings.tickets' organizer=request.event.organizer.slug event=request.event.slug %}">
-                    {% trans "Tickets" %}
-                </a>
-            </li>
-            <li {% if "event.settings.mail" == url_name %}class="active"{% endif %}>
-                <a href="{% url 'control:event.settings.mail' organizer=request.event.organizer.slug event=request.event.slug %}">
-                    {% trans "E-mail" %}
-                </a>
-            </li>
-            <li {% if "event.settings.tax" in url_name %}class="active"{% endif %}>
-                <a href="{% url 'control:event.settings.tax' organizer=request.event.organizer.slug event=request.event.slug %}">
-                    {% trans "Tax rules" %}
-                </a>
-            </li>
-            <li {% if "event.settings.invoice" == url_name %}class="active"{% endif %}>
-                <a href="{% url 'control:event.settings.invoice' organizer=request.event.organizer.slug event=request.event.slug %}">
-                    {% trans "Invoicing" %}
-                </a>
-            </li>
-            <li {% if "event.settings.permissions" == url_name %}class="active"{% endif %}>
-                <a href="{% url 'control:event.settings.permissions' organizer=request.event.organizer.slug event=request.event.slug %}">
-                    {% trans "Permissions" %}
-                </a>
-            </li>
-            <li {% if "event.settings.widget" == url_name %}class="active"{% endif %}>
-                <a href="{% url 'control:event.settings.widget' organizer=request.event.organizer.slug event=request.event.slug %}">
-                    {% trans "Widget" %}
-                </a>
-            </li>
-        {% endif %}
-        {% for nav in nav_event_settings %}
-            <li {% if nav.active %}class="active"{% endif %}>
-                <a href="{{ nav.url }}">
-                    {{ nav.label }}
-                </a>
-            </li>
-        {% endfor %}
-    </ul>
     {% block inside %}
     {% endblock %}
 {% endblock %}

src/pretix/control/templates/pretixcontrol/event/tax_delete.html

@@ -3,7 +3,7 @@
 {% load bootstrap3 %}
 {% block title %}{% trans "Delete tax rule" %}{% endblock %}
 {% block inside %}
-    <legend>{% trans "Delete tax rule" %}</legend>
+    <h1>{% trans "Delete tax rule" %}</h1>
     <form action="" method="post" class="form-horizontal">
         {% csrf_token %}
         {% if possible %}

src/pretix/control/templates/pretixcontrol/event/tax_edit.html

@@ -11,9 +11,9 @@
 {% endblock %}
 {% block inside %}
     {% if rule %}
-        <legend>{% blocktrans with name=rule.name %}Tax rule: {{ name }}{% endblocktrans %}</legend>
+        <h1>{% blocktrans with name=rule.name %}Tax rule: {{ name }}{% endblocktrans %}</h1>
     {% else %}
-        <legend>{% trans "Tax rule" %}</legend>
+        <h1>{% trans "Tax rule" %}</h1>
     {% endif %}
     <form action="" method="post" class="form-horizontal">
         {% csrf_token %}

src/pretix/control/templates/pretixcontrol/event/tax_index.html

@@ -2,7 +2,7 @@
 {% load i18n %}
 {% block title %}{% trans "Tax rules" %}{% endblock %}
 {% block inside %}
-    <legend>{% trans "Tax rules" %}</legend>
+    <h1>{% trans "Tax rules" %}</h1>
     {% if taxrules|length == 0 %}
         <div class="empty-collection">
             <p>

src/pretix/control/templates/pretixcontrol/event/tickets.html

@@ -4,8 +4,8 @@
 {% block inside %}
     <form action="" method="post" class="form-horizontal" enctype="multipart/form-data">
         {% csrf_token %}
+        <h1>{% trans "Ticket download" %}</h1>
         <fieldset>
-            <legend>{% trans "Ticket download" %}</legend>
             {% if request.event.settings.ticket_download and not any_enabled %}
                 <div class="alert alert-warning">
                     {% blocktrans trimmed %}

src/pretix/control/templates/pretixcontrol/event/widget.html

@@ -5,7 +5,7 @@
 {% load eventurl %}
 {% load eventsignal %}
 {% block inside %}
-    <legend>{% trans "Widget" %}</legend>
+    <h1>{% trans "Widget" %}</h1>
     <p>
         {% blocktrans trimmed %}
             The pretix widget is a way to embed your ticket shop into your event website. This way, your visitors can