From bffd39afe5465b22217e31447063082af372b42d Mon Sep 17 00:00:00 2001 From: abuyoyo Date: Sat, 14 Jan 2023 15:01:45 +0200 Subject: [PATCH] js - remove all semi-colons --- js/notice_manager_panel.js | 128 ++++++++++++++++++------------------- 1 file changed, 64 insertions(+), 64 deletions(-) diff --git a/js/notice_manager_panel.js b/js/notice_manager_panel.js index d0fecee..b08cb50 100644 --- a/js/notice_manager_panel.js +++ b/js/notice_manager_panel.js @@ -3,58 +3,58 @@ * */ var NoticeManager = (function ($, document) { - let options = window.notice_manager_options; + let options = window.notice_manager_options let selectors_notice = [ "div.notice", "div.updated", - ]; + ] let selectors_warning = [ "div.notice-warning", "div.update-nag", - ]; + ] let selectors_error = [ "div.error", "div.notice-error", - ]; + ] - let selectors_all = selectors_notice.concat(selectors_warning, selectors_error); + let selectors_all = selectors_notice.concat(selectors_warning, selectors_error) // wait function used with autoCollapse let wait = function (ms) { - var dfd = $.Deferred(); - setTimeout(dfd.resolve, ms); //callback, timeout till callback - return dfd.promise(); - }; + var dfd = $.Deferred() + setTimeout(dfd.resolve, ms) //callback, timeout till callback + return dfd.promise() + } - let notices; + let notices - let button; - let panel; - let haveClosed; // set to true on first close/collect - let dismissNoticesButton; + let button + let panel + let haveClosed // set to true on first close/collect + let dismissNoticesButton // bootstrap // some of these need to run BEFORE document.ready - don't know why really - button = $("#meta-link-notices"); - panel = $("#meta-link-notices-wrap"); - haveClosed = false; - dismissNoticesButton = $("#meta-link-notices-wrap button.notice-dismiss"); + button = $("#meta-link-notices") + panel = $("#meta-link-notices-wrap") + haveClosed = false + dismissNoticesButton = $("#meta-link-notices-wrap button.notice-dismiss") dismissNoticesButton.on("click", () => { - screenMeta.close(panel, button); + screenMeta.close(panel, button) if (! haveClosed){ - NoticeManager.collectNotices(); + NoticeManager.collectNotices() } - NoticeManager.addCounter(); - }); + NoticeManager.addCounter() + }) //original wp focus on click function button.on("focus.scroll-into-view", (e) => { - if (e.target.scrollIntoView) e.target.scrollIntoView(false); - }); + if (e.target.scrollIntoView) e.target.scrollIntoView(false) + }) // scroll page to top when closing notice panel // function used to work with $(this) @@ -63,47 +63,47 @@ var NoticeManager = (function ($, document) { button.on("click", (e) => { if ($(e.target).hasClass("screen-meta-active")) { if (haveClosed) { - NoticeManager.addCounter(); + NoticeManager.addCounter() } - // $(window).scrollTop(true); + // $(window).scrollTop(true) } else { - NoticeManager.removeCounter(); + NoticeManager.removeCounter() // wait (500).then(function(){ //still jumpy sometimes - but scrolls to correct position 400 ~ 600 - // $(window).scrollTop(true); - // }); + // $(window).scrollTop(true) + // }) } - }); + }) /** * document.on.ready */ $(() => { - console.log("NoticeManager.on.ready"); - console.log("options"); - console.log(options); + console.log("NoticeManager.on.ready") + console.log("options") + console.log(options) // bootstrap notices // get all notices that are not explicitly marked as `.inline` or `.below-h2` // we add .update-nag.inline for WordPress Update notice notices = $( selectors_all.join(', ') ) .not(".inline, .below-h2") - .add("div.update-nag"); + .add("div.update-nag") /** * Remove panel if there are no notices on this page */ if (options.screen_panel) { - NoticeManager.maybeRemoveNoticesPanel(); + NoticeManager.maybeRemoveNoticesPanel() } if (options.screen_panel && options.auto_collect) { - NoticeManager.collectNotices(); + NoticeManager.collectNotices() } else { if (options.above_title) { - NoticeManager.moveAboveTitle(); + NoticeManager.moveAboveTitle() } } @@ -111,9 +111,9 @@ var NoticeManager = (function ($, document) { * auto-open notices panel */ if (button.length) { - panel.toggle(); - button.addClass("screen-meta-active"); - screenMeta.open(panel, button); + panel.toggle() + button.addClass("screen-meta-active") + screenMeta.open(panel, button) } /** @@ -124,17 +124,17 @@ var NoticeManager = (function ($, document) { if (options.auto_collapse) { wait(4000).then(() => { if (haveClosed && NoticeManager.getNoticesTopPriority() != 'error') { - screenMeta.close(panel, button); - NoticeManager.addCounter(); + screenMeta.close(panel, button) + NoticeManager.addCounter() } - }); + }) } - }); // end document.on.ready + }) // end document.on.ready // prevent jumpy scrollRestoration on reload page // fixed below on 'beforeunload' // if (history.scrollRestoration) { - // history.scrollRestoration = 'manual'; + // history.scrollRestoration = 'manual' //} /** * Set history.scrollTop to prevent jump on page refresh when scrollRestoration = auto @@ -144,18 +144,18 @@ var NoticeManager = (function ($, document) { { scrollTop: document.body.scrollTop }, document.title, document.location.pathname - ); - }); + ) + }) return { getNotices: () => notices, getNoticesTopPriority: () => { if (notices.filter(":visible").filter(selectors_error.join(", ")).length) - return 'error'; + return 'error' if (notices.filter(":visible").filter(selectors_warning.join(", ")).length) - return 'warning'; - return 'notice'; + return 'warning' + return 'notice' }, /** @@ -163,10 +163,10 @@ var NoticeManager = (function ($, document) { * Remove dismiss-notices button. */ collectNotices: () => { - notices.appendTo(".notice_container").eq(0); - $(".notice_container").removeClass("empty"); // .empty removes padding + notices.appendTo(".notice_container").eq(0) + $(".notice_container").removeClass("empty") // .empty removes padding - haveClosed = true; // initial collection has occurred. + haveClosed = true // initial collection has occurred. /** * When dismissible notices are dismissed, check if any notices are left on page. @@ -178,10 +178,10 @@ var NoticeManager = (function ($, document) { () => { notices = panel .find(selectors_all.join(", ")) - .filter(":visible"); - NoticeManager.maybeRemoveNoticesPanel(); + .filter(":visible") + NoticeManager.maybeRemoveNoticesPanel() } - ); + ) }, addCounter: () => { @@ -190,12 +190,12 @@ var NoticeManager = (function ($, document) { $("").text(notices.filter(":visible").length).attr({ class: "plugin-count", }).addClass(NoticeManager.getNoticesTopPriority()) - ); + ) } }, removeCounter: () => { - button.children(".plugin-count").remove(); + button.children(".plugin-count").remove() }, /** @@ -204,13 +204,13 @@ var NoticeManager = (function ($, document) { */ maybeRemoveNoticesPanel: () => { if (!notices.length) { - screenMeta.close(panel, button); + screenMeta.close(panel, button) - $("#meta-link-notices-link-wrap").detach(); - $("#meta-link-notices-wrap").detach(); + $("#meta-link-notices-link-wrap").detach() + $("#meta-link-notices-wrap").detach() if (!$("#screen-meta-links").children().length) - $("#screen-meta-links").detach(); + $("#screen-meta-links").detach() } }, @@ -220,7 +220,7 @@ var NoticeManager = (function ($, document) { * I HATE it when WordPress moves notices below title. */ moveAboveTitle: () => { - notices.insertBefore(".wrap:first"); + notices.insertBefore(".wrap:first") }, - }; + } }(jQuery,document) ) \ No newline at end of file