From 53d835e6ede69ffc9d7adfb0098f63002cae49c9 Mon Sep 17 00:00:00 2001
From: jvoisin <julien.voisin@dustri.org>
Date: Thu, 19 Sep 2019 20:49:45 +0200
Subject: [PATCH 1/3] Make the logout icon/image cliquable

This should close #1178
---
 .../src/main/webapp/WEB-INF/jsp/top.jsp       | 43 ++++++++++---------
 1 file changed, 22 insertions(+), 21 deletions(-)

diff --git a/airsonic-main/src/main/webapp/WEB-INF/jsp/top.jsp b/airsonic-main/src/main/webapp/WEB-INF/jsp/top.jsp
index 6b53eb78..36553d45 100644
--- a/airsonic-main/src/main/webapp/WEB-INF/jsp/top.jsp
+++ b/airsonic-main/src/main/webapp/WEB-INF/jsp/top.jsp
@@ -132,28 +132,29 @@
         </td>
 
         <td style="padding-left:15pt;padding-right:5pt;vertical-align: middle;width: 100%;text-align: center">
-
-          <c:choose>
-            <c:when test="${model.showAvatar}">
-            <sub:url value="avatar.view" var="avatarUrl">
-                <sub:param name="username" value="${model.user.username}"/>
-            </sub:url>
-                <div style="padding-bottom: 4px">
-                    <c:if test="${model.user.settingsRole}"><a href="personalSettings.view" target="main"></c:if>
-                        <img src="${avatarUrl}" alt="" width="30" height="30">
-                        <c:if test="${model.user.settingsRole}"></a></c:if>
+            <a href="<c:url value="/logout"/>" target="_top">
+                <c:choose>
+                    <c:when test="${model.showAvatar}">
+                        <sub:url value="avatar.view" var="avatarUrl">
+                            <sub:param name="username" value="${model.user.username}"/>
+                        </sub:url>
+                        <div style="padding-bottom: 4px">
+                            <c:if test="${model.user.settingsRole}"><a href="personalSettings.view" target="main"></c:if>
+                            <img src="${avatarUrl}" alt="" width="30" height="30">
+                            <c:if test="${model.user.settingsRole}"></a></c:if>
+                        </div>
+                    </c:when>
+                    <c:otherwise>
+                        <img src="<spring:theme code="logoutImage"/>" alt="logout" height="24">
+                    </c:otherwise>
+                </c:choose>
+
+                <div class="detail">
+                    <fmt:message key="top.logout" var="logout"><br/>
+                        <fmt:param value="${model.user.username}"/></fmt:message>
+                    ${fn:escapeXml(logout)}
                 </div>
-            </c:when>
-            <c:otherwise>
-            <img src="<spring:theme code="logoutImage"/>" alt="logout" height="24">
-            </c:otherwise>
-          </c:choose>
-
-            <div class="detail">
-              <fmt:message key="top.logout" var="logout"><br/>
-              <fmt:param value="${model.user.username}"/></fmt:message>
-                <a href="<c:url value="/logout"/>" target="_top">${fn:escapeXml(logout)}</a>
-            </div>
+            </a>
         </td>
 
     </tr></table>

From 45c6893f16764a91b66a910919d027462eec1aee Mon Sep 17 00:00:00 2001
From: "Andrew V. Louis" <13123840+avlouis@users.noreply.github.com>
Date: Tue, 1 Oct 2019 14:13:30 -0500
Subject: [PATCH 2/3] Making textareas vertical-align middle allows them to
 align with adjacent imgs.

Signed-off-by: Andrew V. Louis
---
 .../src/main/webapp/style/default-without-mediaelement.css       | 1 +
 1 file changed, 1 insertion(+)

diff --git a/airsonic-main/src/main/webapp/style/default-without-mediaelement.css b/airsonic-main/src/main/webapp/style/default-without-mediaelement.css
index 70f95c68..a671bf10 100644
--- a/airsonic-main/src/main/webapp/style/default-without-mediaelement.css
+++ b/airsonic-main/src/main/webapp/style/default-without-mediaelement.css
@@ -194,6 +194,7 @@ input, textarea, select {
     padding: 3px;
     border-radius: 3px;
     border: 1px solid lightgray;
+    vertical-align: middle;
 }
 
 input[type=submit], input[type=button] {

From f77a0d8b6788980a9d5cef80424146ca591765cc Mon Sep 17 00:00:00 2001
From: Jon Borglund <jon.borglund@gmail.com>
Date: Thu, 3 Oct 2019 22:13:35 +0200
Subject: [PATCH 3/3] fixed 'Settings Saved' notification when saving
 transcoding settings #1114 (#1269)

Signed-off-by: Jon <jon@powha.net>
---
 .../player/controller/TranscodingSettingsController.java       | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/airsonic-main/src/main/java/org/airsonic/player/controller/TranscodingSettingsController.java b/airsonic-main/src/main/java/org/airsonic/player/controller/TranscodingSettingsController.java
index 5009bdf3..d6f4662c 100644
--- a/airsonic-main/src/main/java/org/airsonic/player/controller/TranscodingSettingsController.java
+++ b/airsonic-main/src/main/java/org/airsonic/player/controller/TranscodingSettingsController.java
@@ -69,9 +69,10 @@ public class TranscodingSettingsController {
     public String doPost(HttpServletRequest request, RedirectAttributes redirectAttributes) throws Exception {
         String error = handleParameters(request, redirectAttributes);
         if (error != null) {
+            redirectAttributes.addFlashAttribute("error", error);
+        } else {
             redirectAttributes.addFlashAttribute("settings_toast", true);
         }
-        redirectAttributes.addFlashAttribute("error", error);
         return "redirect:transcodingSettings.view";
     }