From 0b19f0d85e5d22c0ac3555282d9a3832246771d4 Mon Sep 17 00:00:00 2001
From: Bill Thornton <thornbill@users.noreply.github.com>
Date: Fri, 6 May 2022 13:30:10 -0400
Subject: [PATCH] Merge pull request #3604 from thornbill/fix-custom-elements

Fix custom element creation

(cherry picked from commit f2226ee74500af0ee48f61d1ca5641d2a9e411c8)
Signed-off-by: crobibero <cody@robibe.ro>
---
 .../dashboard/plugins/repositories/index.js   | 20 +++++++++----------
 src/controllers/session/login/index.js        |  1 +
 src/scripts/libraryMenu.js                    | 15 +++++++-------
 3 files changed, 19 insertions(+), 17 deletions(-)

diff --git a/src/controllers/dashboard/plugins/repositories/index.js b/src/controllers/dashboard/plugins/repositories/index.js
index 510a031716..eff2de40c4 100644
--- a/src/controllers/dashboard/plugins/repositories/index.js
+++ b/src/controllers/dashboard/plugins/repositories/index.js
@@ -2,9 +2,11 @@ import loading from '../../../../components/loading/loading';
 import libraryMenu from '../../../../scripts/libraryMenu';
 import globalize from '../../../../scripts/globalize';
 import dialogHelper from '../../../../components/dialogHelper/dialogHelper';
+
 import '../../../../elements/emby-button/emby-button';
 import '../../../../elements/emby-checkbox/emby-checkbox';
 import '../../../../elements/emby-select/emby-select';
+
 import '../../../../components/formdialog.scss';
 import '../../../../components/listview/listview.scss';
 
@@ -19,8 +21,8 @@ function reloadList(page) {
             noneElement: page.querySelector('#none'),
             repositories: repositories
         });
-    }).catch(() => {
-        console.error('error loading repositories');
+    }).catch(e => {
+        console.error('error loading repositories', e);
         page.querySelector('#none').classList.remove('hide');
         loading.hide();
     });
@@ -35,8 +37,8 @@ function saveList(page) {
         contentType: 'application/json'
     }).then(() => {
         reloadList(page);
-    }).catch(() => {
-        console.error('error saving repositories');
+    }).catch(e => {
+        console.error('error saving repositories', e);
         loading.hide();
     });
 }
@@ -64,9 +66,8 @@ function getRepositoryElement(repository) {
     const listItem = document.createElement('div');
     listItem.className = 'listItem listItem-border';
 
-    const repoLink = document.createElement('a');
-    repoLink.setAttribute('is', 'emby-linkbutton');
-    repoLink.className = 'clearLink listItemIconContainer';
+    const repoLink = document.createElement('a', 'emby-linkbutton');
+    repoLink.classList.add('clearLink', 'listItemIconContainer');
     repoLink.style.margin = '0';
     repoLink.style.padding = '0';
     repoLink.rel = 'noopener noreferrer';
@@ -90,10 +91,9 @@ function getRepositoryElement(repository) {
 
     listItem.appendChild(body);
 
-    const button = document.createElement('button');
+    const button = document.createElement('button', 'paper-icon-button-light');
     button.type = 'button';
-    button.setAttribute('is', 'paper-icon-button-light');
-    button.className = 'btnDelete';
+    button.classList.add('btnDelete');
     button.id = repository.Url;
     button.title = globalize.translate('Delete');
     button.innerHTML = '<span class="material-icons delete" aria-hidden="true"></span>';
diff --git a/src/controllers/session/login/index.js b/src/controllers/session/login/index.js
index 20a7c0a5b2..060324b450 100644
--- a/src/controllers/session/login/index.js
+++ b/src/controllers/session/login/index.js
@@ -289,6 +289,7 @@ import './login.scss';
                 disclaimer.innerHTML = DOMPurify.sanitize(marked(options.LoginDisclaimer || ''));
 
                 for (const elem of disclaimer.querySelectorAll('a')) {
+                    elem.rel = 'noopener noreferrer';
                     elem.target = '_blank';
                     elem.classList.add('button-link');
                     elem.setAttribute('is', 'emby-linkbutton');
diff --git a/src/scripts/libraryMenu.js b/src/scripts/libraryMenu.js
index b77aba42e2..4220b9dca9 100644
--- a/src/scripts/libraryMenu.js
+++ b/src/scripts/libraryMenu.js
@@ -1,4 +1,6 @@
 import escapeHtml from 'escape-html';
+import Headroom from 'headroom.js';
+
 import dom from './dom';
 import layoutManager from '../components/layoutManager';
 import inputManager from './inputManager';
@@ -12,14 +14,14 @@ import browser from './browser';
 import globalize from './globalize';
 import imageHelper from './imagehelper';
 import { getMenuLinks } from '../scripts/settings/webSettings';
+import Dashboard, { pageClassOn } from './clientUtils';
+import ServerConnections from '../components/ServerConnections';
+
 import '../elements/emby-button/paper-icon-button-light';
+
 import 'material-design-icons-iconfont';
 import '../assets/css/scrollstyles.scss';
 import '../assets/css/flexstyles.scss';
-import Dashboard, { pageClassOn } from '../utils/dashboard';
-import ServerConnections from '../components/ServerConnections';
-import Headroom from 'headroom.js';
-import { getParameterByName } from '../utils/url.ts';
 
 /* eslint-disable indent */
 
@@ -669,9 +671,8 @@ import { getParameterByName } from '../utils/url.ts';
         if (customMenuOptions) {
             getMenuLinks().then(links => {
                 links.forEach(link => {
-                    const option = document.createElement('a');
-                    option.setAttribute('is', 'emby-linkbutton');
-                    option.className = 'navMenuOption lnkMediaFolder';
+                    const option = document.createElement('a', 'emby-linkbutton');
+                    option.classList.add('navMenuOption', 'lnkMediaFolder');
                     option.rel = 'noopener noreferrer';
                     option.target = '_blank';
                     option.href = link.url;