From 73b4010fcd93e921b626a6f51de7a148de276283 Mon Sep 17 00:00:00 2001 From: John Olheiser Date: Sat, 25 Mar 2023 14:37:34 -0500 Subject: Remove row clicking from notification table (#22695) Resolves #22692 I don't think there's a need for this entire row to be clickable (and even different links depending on which segment you click) The links still point to the same spot, so no information is lost here. --------- Signed-off-by: jolheiser Co-authored-by: wxiaoguang --- web_src/js/features/common-global.js | 15 --------------- 1 file changed, 15 deletions(-) (limited to 'web_src/js/features') diff --git a/web_src/js/features/common-global.js b/web_src/js/features/common-global.js index 3546a052b3..a10ca74ab5 100644 --- a/web_src/js/features/common-global.js +++ b/web_src/js/features/common-global.js @@ -124,21 +124,6 @@ export function initGlobalCommon() { toggleElem($($(this).data('target'))); }); - // make table and elements clickable like a link - $('tr[data-href], td[data-href]').on('click', function (e) { - const href = $(this).data('href'); - if (e.target.nodeName === 'A') { - // if a user clicks on , then the or should not act as a link. - return; - } - if (e.ctrlKey || e.metaKey) { - // ctrl+click or meta+click opens a new window in modern browsers - window.open(href); - } else { - window.location = href; - } - }); - // prevent multiple form submissions on forms containing .loading-button document.addEventListener('submit', (e) => { const btn = e.target.querySelector('.loading-button'); -- cgit v1.2.3