From 9e9a6c9921f14d64b18f73485b149bfed8e4cf68 Mon Sep 17 00:00:00 2001 From: Maxr1998 Date: Fri, 4 Dec 2020 17:20:12 +0100 Subject: [PATCH] Simplify hiding menu items based on supported features --- src/controllers/user/menu/index.js | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/src/controllers/user/menu/index.js b/src/controllers/user/menu/index.js index 1b4afd70ea..062b1bd6da 100644 --- a/src/controllers/user/menu/index.js +++ b/src/controllers/user/menu/index.js @@ -29,17 +29,11 @@ export default function (view, params) { page.querySelector('.lnkSubtitlePreferences').setAttribute('href', '#!/mypreferencessubtitles.html?userId=' + userId); page.querySelector('.lnkQuickConnectPreferences').setAttribute('href', '#!/mypreferencesquickconnect.html'); - if (window.NativeShell && window.NativeShell.AppHost.supports('clientsettings')) { - page.querySelector('.clientSettings').classList.remove('hide'); - } else { - page.querySelector('.clientSettings').classList.add('hide'); - } + const supportsClientSettings = appHost.supports('clientsettings'); + page.querySelector('.clientSettings').classList.toggle('hide', !supportsClientSettings); - if (appHost.supports('multiserver')) { - page.querySelector('.selectServer').classList.remove('hide'); - } else { - page.querySelector('.selectServer').classList.add('hide'); - } + const supportsMultiServer = appHost.supports('multiserver'); + page.querySelector('.selectServer').classList.toggle('hide', !supportsMultiServer); ApiClient.getUser(userId).then(function (user) { page.querySelector('.headerUsername').innerHTML = user.Name;