From eb255e84d868ff9bf17365e774c0b53552391814 Mon Sep 17 00:00:00 2001
From: Oliver <oliver.henry.walters@gmail.com>
Date: Sat, 18 Jun 2022 17:33:59 +1000
Subject: [PATCH] Small UI improvements to the settings interface (#3223)

---
 InvenTree/templates/InvenTree/settings/user.html     | 12 +++++++-----
 .../templates/InvenTree/settings/user_display.html   | 11 +++++++----
 2 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/InvenTree/templates/InvenTree/settings/user.html b/InvenTree/templates/InvenTree/settings/user.html
index b9feb555f5..948c84bcac 100644
--- a/InvenTree/templates/InvenTree/settings/user.html
+++ b/InvenTree/templates/InvenTree/settings/user.html
@@ -154,7 +154,9 @@
         </form>
 
         {% else %}
-        <p>{% trans 'You currently have no social network accounts connected to this account.' %}</p>
+        <div class='alert alert-block alert-warning'>
+            {% trans 'There are no social network accounts connected to this account.' %}
+        </div>
         {% endif %}
     </div>
 
@@ -198,9 +200,9 @@
     </table>
 
     {% else %}
-    <p><strong>{% trans 'Warning:'%}</strong>
-        {% trans "You currently do not have any factors set up." %}
-    </p>
+    <div class='alert alert-block alert-warning'>
+        {% trans "Multifactor authentication is not configured for your account" %}
+    </div>
 
     {% endif %}
     </div>
@@ -224,7 +226,7 @@
                 <form method="post" action="{% url 'session_delete_other' %}">
                     {% csrf_token %}
                     <button type="submit" class="btn btn-sm btn-default btn-danger" title='{% trans "Log out active sessions (except this one)" %}'>
-                        {% trans "Log Out Active Sessions" %}
+                        <span class='fas fa-sign-out-alt'></span> {% trans "Log Out Active Sessions" %}
                     </button>
                 </form>
                 {% endif %}
diff --git a/InvenTree/templates/InvenTree/settings/user_display.html b/InvenTree/templates/InvenTree/settings/user_display.html
index bab90aadf9..1e0b4a1ca3 100644
--- a/InvenTree/templates/InvenTree/settings/user_display.html
+++ b/InvenTree/templates/InvenTree/settings/user_display.html
@@ -34,7 +34,7 @@
             {% csrf_token %}
             <input name='next' type='hidden' value='{% url "settings" %}'>
             <label for='theme' class=' requiredField'>
-                {% trans "Select theme" %}
+                <h6>{% trans "Select theme" %}</h6>
             </label>
             <div class='form-group input-group mb-3'>
                 <select id='theme' name='theme' class='select form-control'>
@@ -62,7 +62,7 @@
             {% csrf_token %}
             <input name="next" type="hidden" value="{% url 'settings' %}">
             <label for='language' class=' requiredField'>
-                {% trans "Select language" %}
+                <h6>{% trans "Select language" %}</h6>
             </label>
             <div class='form-group input-group mb-3'>
                 <select name="language" class="select form-control w-25">
@@ -100,8 +100,11 @@
     </form>
     </div>
     <div class="col-sm-6">
-        <h4>{% trans "Help the translation efforts!" %}</h4>
-        <p>{% blocktrans with link="https://crowdin.com/project/inventree" %}Native language translation of the web application is <a href="{{link}}">community contributed via crowdin</a>. Contributions are welcomed and encouraged.{% endblocktrans %}</p>
+        <div class='alert alert-block alert-success'>
+            <h5>{% trans "Help the translation efforts!" %}</h5>
+            <p>{% trans "Native language translation of the web application is community contributed via crowdin. Contributions are welcomed and encouraged." %}</p>
+            <p><a href="https://crowdin.com/project/inventree">{% trans "InvenTree Translation Project" %}</a></p>
+        </div>
     </div>
 </div>