From 6ce4e6cb3710e4684f2623301d597b7b0581aea7 Mon Sep 17 00:00:00 2001
From: Bernd Bestel <bernd@berrnd.de>
Date: Fri, 20 Sep 2019 10:22:59 +0200
Subject: [PATCH] An icon is not needed when not showing an userentity in the
 sidebar menu (references #242)

---
 changelog/52_UNRELEASED_2019-xx-xx.md |  3 +++
 public/viewjs/userentityform.js       | 16 ++++++++++++++++
 2 files changed, 19 insertions(+)

diff --git a/changelog/52_UNRELEASED_2019-xx-xx.md b/changelog/52_UNRELEASED_2019-xx-xx.md
index 3d6b7a23..5ac98039 100644
--- a/changelog/52_UNRELEASED_2019-xx-xx.md
+++ b/changelog/52_UNRELEASED_2019-xx-xx.md
@@ -73,3 +73,6 @@
   - Field `stock_amount` of endpoint `/stock/products/{productId}` now returns `0` instead of `null` when the given product is not in stock (thanks @Forceu)
   - Fixed that `/system/db-changed-time` always returned the current time (more or less) due to that that time is the database file modification time and the database is effectively changed on each request because of session information tracking - which now explicitly does not change the database file modification time, so this should work again to determine if any data changes happened
   - It's now also possible to provide the API key via a query parameter (same name as the header, so `GROCY-API-KEY`)
+
+#### Say thanks
+Because there were some questions about that in the past: If grocy is useful for you, [say thanks}(https://grocy.info/#say-thanks)!
diff --git a/public/viewjs/userentityform.js b/public/viewjs/userentityform.js
index 4c376ceb..8f6af6ed 100644
--- a/public/viewjs/userentityform.js
+++ b/public/viewjs/userentityform.js
@@ -64,5 +64,21 @@ $('#userentity-form input').keydown(function(event)
 	}
 });
 
+$("#show_in_sidebar_menu").on("click", function()
+{
+	if (this.checked)
+	{
+		$("#icon_css_class").removeAttr("disabled");
+	}
+	else
+	{
+		$("#icon_css_class").attr("disabled", "");
+	}
+});
+
 $('#name').focus();
 Grocy.FrontendHelpers.ValidateForm('userentity-form');
+
+// Click twice to trigger on-click but not change the actual checked state
+$("#show_in_sidebar_menu").click();
+$("#show_in_sidebar_menu").click();