From b86688806267d0c063435116a9f5cad5dc80c2b3 Mon Sep 17 00:00:00 2001 From: Isaac Connor Date: Tue, 8 Feb 2022 17:51:21 -0500 Subject: [PATCH] Change title of settings button to give an indication WHY it isn't enabled --- web/skins/classic/views/js/watch.js | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/web/skins/classic/views/js/watch.js b/web/skins/classic/views/js/watch.js index 6c9a669fe..21026c3c8 100644 --- a/web/skins/classic/views/js/watch.js +++ b/web/skins/classic/views/js/watch.js @@ -805,7 +805,15 @@ function initPage() { if (monitorType == 'Local') getSettingsModal(); } // Only enable the settings button for local cameras - settingsBtn.prop('disabled', !(canView.Control && (monitorType == 'Local'))); + if (!canView.Control) { + settingsBtn.prop('disabled', true); + settingsBtn.prop('title', 'Disbled due to lack of Control View permission.'); + } else if (monitorType != 'Local') { + settingsBtn.prop('disabled', true); + settingsBtn.prop('title', 'Settings only available for Local monitors.'); + } else { + settingsBtn.prop('disabled', false); + } if (monitorType != 'WebSite') { monitorStream = new MonitorStream(monitorData[monIdx]);