Skip to content

Gumgum #4691

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
evanpelle wants to merge 4 commits into
base: main
Choose a base branch
from
+63 −3
Draft

Gumgum #4691

Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
7cdf856
fix: reserve top space for Playwire flex ad unit instead of letting i…
Celant Jul 16, 2026
fd2266b
gum gum footer
evanpelle Jul 23, 2026
bb10055
fix url
evanpelle Jul 23, 2026
e2d73f8
remove duplicate script
evanpelle Jul 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
69 changes: 67 additions & 2 deletions index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,12 +18,19 @@
transition: opacity 0.5s ease-out;
}

/* iOS safe area support */
/* iOS safe area support, plus reserved space for Playwire's "flex"
ad unit (a top banner injected as body's first child). Its
height is mirrored into --flex-ad-height by the script near
the end of <body>, so the nav bar is pushed below it instead
of being covered. */
body {
padding-top: env(safe-area-inset-top);
padding-top: calc(
env(safe-area-inset-top) + var(--flex-ad-height, 0px)
);
padding-right: env(safe-area-inset-right);
padding-bottom: env(safe-area-inset-bottom);
padding-left: env(safe-area-inset-left);
transition: padding-top 0.3s ease;
}

/* Prevent Playwire bottom rail from affecting flex layout,
Expand All @@ -33,6 +40,20 @@
z-index: 150 !important;
}

/* Prevent Playwire's "flex" ad unit (top banner) from being
treated as a flex item in body's flex-row layout. It's
injected as body's first child; the ad creative itself is
already position:fixed, but the wrapping container Playwire
adds is not, so — like bottom_rail above — it must be taken
out of flow or it shifts/clips the rest of the page under
body's overflow-hidden. */
[id^="pw-oop-flex"] {
position: fixed !important;
top: 0;
left: 0;
width: 100%;
}

/* Google Funding Choices CCPA link override */
.fc-dns-link,
.fc-ccpa-root,
Expand Down Expand Up @@ -463,6 +484,44 @@
});
</script>

<script>
// Keep --flex-ad-height in sync with Playwire's "flex" ad unit
// (a top banner injected as body's first child, id
// "pw-oop-flex_container"), so the CSS in <head> can reserve
// matching space above the nav bar. The ad is added/removed/
// resized asynchronously by Playwire's own script, so this
// watches for it rather than assuming a fixed height.
(function () {
if (typeof ResizeObserver === "undefined") return;

var root = document.documentElement;
var observedEl = null;
var resizeObserver = new ResizeObserver(function (entries) {
root.style.setProperty(
"--flex-ad-height",
Math.round(entries[0].contentRect.height) + "px",
);
});

function sync() {
var el = document.getElementById("pw-oop-flex_container");
if (el === observedEl) return;
if (observedEl) resizeObserver.unobserve(observedEl);
observedEl = el;
if (el) {
resizeObserver.observe(el);
} else {
root.style.setProperty("--flex-ad-height", "0px");
}
}

sync();
new MutationObserver(sync).observe(document.body, {
childList: true,
});
})();
</script>

<!-- Analytics -->
<script
defer
Expand All @@ -476,6 +535,12 @@
async
src="//cdn.intergient.com/1025558/75940/ramp.js"
></script>
<!--Gum Gum header ads-->
<script
data-cfasync="false"
async
src="cdn.intergient.com/1025558/78237/ramp.js"
></script>
Comment thread
coderabbitai[bot] marked this conversation as resolved.
</footer>
</body>
</html>
Loading