From 154bb1749d7fa111cef024a3d29d3f8f3cdf021c Mon Sep 17 00:00:00 2001 From: HanYaodong Date: Thu, 29 Feb 2024 12:36:06 +0800 Subject: Stop refresh animation on popup when not on youtube page --- src/content.ts | 7 ++++++- src/popup.ts | 6 ++++-- 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/src/content.ts b/src/content.ts index 504ba8d5..f89fb369 100644 --- a/src/content.ts +++ b/src/content.ts @@ -258,7 +258,12 @@ function messageListener(request: Message, sender: unknown, sendResponse: (respo break; case "refreshSegments": // update video on refresh if videoID invalid - if (!getVideoID()) checkVideoIDChange(); + if (!getVideoID()) { + checkVideoIDChange().then(() => { + // if still no video ID found, return an empty info to the popup + if (!getVideoID()) chrome.runtime.sendMessage({ message: "infoUpdated" }); + }); + } // fetch segments sponsorsLookup(false); diff --git a/src/popup.ts b/src/popup.ts index 86e79f54..f954f168 100644 --- a/src/popup.ts +++ b/src/popup.ts @@ -465,8 +465,8 @@ async function runThePopup(messageListener?: MessageListener): Promise { return; } - //if request is undefined, then the page currently being browsed is not YouTube - if (request != undefined) { + // if request has no field other than message, then the page currently being browsed is not YouTube + if (request.found != undefined) { //remove loading text PageElements.mainControls.style.display = "block"; if (request.onMobileYouTube) PageElements.mainControls.classList.add("hidden"); @@ -490,6 +490,8 @@ async function runThePopup(messageListener?: MessageListener): Promise { PageElements.issueReporterImportExport.classList.remove("hidden"); } + } else { + displayNoVideo(); } //see if whitelist button should be swapped -- cgit v1.2.3