Merge pull request #1453 from AlecRust/fix-width-when-embedded

Fix popup width when embedded in page
This commit is contained in:
Ajay Ramachandran
2022-08-21 22:23:52 -04:00
committed by GitHub
2 changed files with 13 additions and 7 deletions

View File

@@ -100,6 +100,10 @@
margin-bottom: 16px;
}
#sponsorBlockPopupContainer iframe {
width: 100%;
}
/*
* Disable popup max height when displayed in-page (content.ts)
*/
@@ -110,7 +114,7 @@
/*
* Disable fixed popup width when displayed in-page (content.ts)
*/
#sponsorBlockPopupContainer #sponsorBlockPopupBody {
#sponsorBlockPopupBody.is-embedded {
width: auto;
}

View File

@@ -75,6 +75,7 @@ async function runThePopup(messageListener?: MessageListener): Promise<void> {
const PageElements: PageElements = {};
[
"sponsorBlockPopupBody",
"sponsorblockPopup",
"sponsorStart",
// Top toggles
@@ -132,7 +133,7 @@ async function runThePopup(messageListener?: MessageListener): Promise<void> {
getSegmentsFromContentScript(false);
await utils.wait(() => Config.config !== null && allowPopup, 5000, 5);
document.querySelector("body").style.removeProperty("visibility");
PageElements.sponsorBlockPopupBody.style.removeProperty("visibility");
if (!Config.configSyncListeners.includes(contentConfigUpdateListener)) {
Config.configSyncListeners.push(contentConfigUpdateListener);
}
@@ -145,6 +146,7 @@ async function runThePopup(messageListener?: MessageListener): Promise<void> {
if (window !== window.top) {
PageElements.sbCloseButton.classList.remove("hidden");
PageElements.sponsorBlockPopupBody.classList.add("is-embedded");
}
// Hide donate button if wanted (Safari, or user choice)