From 11bef3d7f44901764343eaab05788a9d86294551 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ricardo=20Monta=C3=B1ana?= <rmontanana@gmail.com>
Date: Mon, 5 Jun 2023 17:33:20 +0200
Subject: [PATCH] Finish select benchmark

---
 app/app.db                               | Bin 28672 -> 28672 bytes
 app/main.py                              |   3 +++
 app/results/templates/_table_select.html |   2 +-
 app/results/templates/select.html        |   7 +++++
 app/templates/_benchmarks.html           |   9 +++++++
 app/templates/_nav.html                  |  33 +++++++++++++----------
 app/templates/base.html                  |   9 +++++--
 app/templates/index.html                 |  11 +-------
 app/templates/login.html                 |   6 +++--
 9 files changed, 51 insertions(+), 29 deletions(-)
 create mode 100644 app/templates/_benchmarks.html

diff --git a/app/app.db b/app/app.db
index 88ad1e0bd71fa4212f80bc486f64f5b36dd0a8e4..9544a8c8d33ec94ce599936ae89f2c0319d333ea 100644
GIT binary patch
delta 61
zcmZp8z}WDBae_4Cp@}lijE6QREVSq5FJxfii(=r5;(N7OP{5CmrIDABV{)8dHzV`p
RSAOj*3=9nYn@jz>6#!$f63PGo

delta 61
zcmZp8z}WDBae_2s_e2?I#_o*?3+=fX7#NuNUNP_&^1a$DDB#D(!e7WRVRD>bHzU*J
RSAOj*jXaDTn@jz>6#!hy69WJM

diff --git a/app/main.py b/app/main.py
index b10d5d6..270f3d4 100644
--- a/app/main.py
+++ b/app/main.py
@@ -32,6 +32,9 @@ def index():
 @main.route("/set_benchmark/<benchmark_id>")
 @login_required
 def set_benchmark(benchmark_id):
+    if int(benchmark_id) == current_user.benchmark_id:
+        flash("Benchmark already selected.")
+        return redirect(url_for(INDEX))
     if current_user.admin:
         benchmark = Benchmark.query.filter_by(id=benchmark_id).first()
         if benchmark is None:
diff --git a/app/results/templates/_table_select.html b/app/results/templates/_table_select.html
index 26fa05f..7bfe53e 100644
--- a/app/results/templates/_table_select.html
+++ b/app/results/templates/_table_select.html
@@ -5,7 +5,7 @@
 <table id="file-table"
        class="table table-striped table-hover table-bordered">
     <thead>
-        <tr>
+        <tr class="table-primary">
             <th>Model</th>
             <th>Metric</th>
             <th>Platform</th>
diff --git a/app/results/templates/select.html b/app/results/templates/select.html
index 14cf448..b220b69 100644
--- a/app/results/templates/select.html
+++ b/app/results/templates/select.html
@@ -1,9 +1,16 @@
 {% extends "base.html" %}
+{% block styles %}
+    {{ super() }}
+    <link rel="stylesheet"
+          href="https://cdn.datatables.net/1.10.25/css/jquery.dataTables.min.css">
+{% endblock %}
 {% block content %}
     {% include "_table_select.html" %}
 {% endblock %}
 {% block jscript %}
     {{ super() }}
     <script src="https://cdn.datatables.net/1.10.25/js/jquery.dataTables.min.js"></script>
+    <script rel="stylesheet"
+            href="https://cdn.datatables.net/1.13.4/js/dataTables.bootstrap5.min.js"></script>
     <script src="{{ url_for('static', filename="js/select.js") }}"></script>
 {% endblock %}
diff --git a/app/templates/_benchmarks.html b/app/templates/_benchmarks.html
new file mode 100644
index 0000000..7ce43dc
--- /dev/null
+++ b/app/templates/_benchmarks.html
@@ -0,0 +1,9 @@
+<div class="container col-md-4 col-sm-4 col-lg-offset-4">
+    <h5>Benchmarks</h5>
+    <div class="list-group">
+        {% for benchmark in benchmarks %}
+            <a href="{{ url_for('main.set_benchmark', benchmark_id=benchmark.id) }}"
+               class="list-group-item list-group-item-action {% if benchmark == current_user.benchmark %}active{% endif %}">{{ benchmark.name + " - " + benchmark.description }}</a>
+        {% endfor %}
+    </div>
+</div>
diff --git a/app/templates/_nav.html b/app/templates/_nav.html
index a5e6c92..62c081b 100644
--- a/app/templates/_nav.html
+++ b/app/templates/_nav.html
@@ -1,30 +1,35 @@
 {% from 'bootstrap5/nav.html' import render_nav_item %}
-<nav class="navbar navbar-expand-sm navbar-dark bg-dark mb-4 justify-content-end">
-    <div class="container">
+<nav class="navbar navbar-expand-lg navbar-dark bg-dark">
+    <div class="container-fluid">
+        <a class="navbar-brand" href="{{ url_for("main.index") }}">Benchmark</a>
         <button class="navbar-toggler"
                 type="button"
                 data-bs-toggle="collapse"
-                data-bs-target="#navbarSupportedContent"
-                aria-controls="navbarSupportedContent"
+                data-bs-target="#navbarToggler"
+                aria-controls="navbarToggler"
                 aria-expanded="false"
                 aria-label="Toggle navigation">
             <span class="navbar-toggler-icon"></span>
         </button>
-        <div class="collapse navbar-collapse" id="navbarSupportedContent">
-            <!-- Left side of navbar -->
-            <ul class="navbar-nav me-auto">
-                {{ render_nav_item('main.index', 'Home') }}
-            </ul>
-            <ul class="navbar-nav justify-content-end">
-                {{ render_nav_item('results.select', 'Results') }}
-                {{ render_nav_item('results.datasets', 'Datasets') }}
-                {{ render_nav_item('main.config', 'Config') }}
+        <div class="collapse navbar-collapse" id="navbarToggler">
+            <ul class="navbar-nav me-auto mb-2 mb-lg-0">
                 {% if current_user.is_authenticated %}
-                    {{ render_nav_item('main.logout', current_user.username + '(Logout) ') }}
+                    <li>{{ render_nav_item('main.index', "on " + current_user.benchmark.name) }}</li>
+                {% endif %}
+                <li>{{ render_nav_item('results.select', 'Results') }}</li>
+                <li>{{ render_nav_item('results.datasets', 'Datasets') }}</li>
+                <li>{{ render_nav_item('main.config', 'Config') }}</li>
+            </ul>
+            <div class="navbar-nav justify-content-end">
+                {% if current_user.is_authenticated %}
+                    <img src='{{ current_user.avatar(40) }}' height=35 alt="gravatar" />
+                    {{ render_nav_item('main.logout',
+                                        current_user.username + '(Logout) ')|safe }}
                 {% else %}
                     {{ render_nav_item('main.login', 'Login') }}
                 {% endif %}
             </ul>
         </div>
     </div>
+</div>
 </nav>
diff --git a/app/templates/base.html b/app/templates/base.html
index 765ddec..3444773 100644
--- a/app/templates/base.html
+++ b/app/templates/base.html
@@ -7,6 +7,11 @@
                   content="width=device-width, initial-scale=1, shrink-to-fit=no">
             {% block styles %}
                 {{ bootstrap.load_css() }}
+                <link rel="stylesheet"
+                      href="https://cdnjs.cloudflare.com/ajax/libs/MaterialDesign-Webfont/7.1.96/css/materialdesignicons.css"
+                      integrity="sha512-lD1LHcZ8tFHvMFNeo6qOLY/HjzSPCasPJOAoir22byDxlZI1R71S5lZel8zRL2TZ+Dut1wOHfYgSU2lHXuL00w=="
+                      crossorigin="anonymous"
+                      referrerpolicy="no-referrer" />
                 <link rel="stylesheet"
                       href="{{ url_for('static', filename='css/main.css') }}">
             {% endblock %}
@@ -14,7 +19,7 @@
         {% endblock %}
     </head>
     <body>
-        {% include "_nav.html" %}
+        <div class="container-fluid">{% include "_nav.html" %}</div>
         {% with messages = get_flashed_messages(with_categories=true) %}
             {% if messages %}
                 {% for category, message in messages %}
@@ -25,7 +30,7 @@
                 {% endfor %}
             {% endif %}
         {% endwith %}
-        <div class="container">
+        <div class="container-fluid">
             {% block content %}{% endblock %}
         </div>
     </body>
diff --git a/app/templates/index.html b/app/templates/index.html
index ec42095..a79ade0 100644
--- a/app/templates/index.html
+++ b/app/templates/index.html
@@ -1,16 +1,7 @@
 {% extends "base.html" %}
 {% block content %}
     {% if benchmarks %}
-        <h5>Benchmarks</h5>
-        <ul>
-            {% for benchmark in benchmarks %}
-                <li>
-                    <a href="{{ url_for('main.set_benchmark', benchmark_id=benchmark.id) }}">{{ benchmark.name }}
-                        {% if benchmark == current_user.benchmark %}*{% endif %}
-                    </a>
-                </li>
-            {% endfor %}
-        </ul>
+        {% include "_benchmarks.html" %}
     {% else %}
         <img src="{{ url_for('static', filename='img/robert-lukeman-_RBcxo9AU-U-unsplash.jpg') }}"
              class="img-fluid"
diff --git a/app/templates/login.html b/app/templates/login.html
index 00b45f3..d5139fb 100644
--- a/app/templates/login.html
+++ b/app/templates/login.html
@@ -1,6 +1,8 @@
 {% extends 'base.html' %}
 {% from 'bootstrap5/form.html' import render_form %}
 {% block content %}
-    <h2>Login</h2>
-    {{ render_form(form) }}
+    <div class="col-md-3 col-lg-offset-4">
+        <h2>Login</h2>
+        {{ render_form(form) }}
+    </div>
 {% endblock content %}