Skip to content

Reserve 90px space for top-above-nav advert on tablet#16282

Draft
domlander wants to merge 2 commits into
mainfrom
doml/fix-tablet-top-ad-space
Draft

Reserve 90px space for top-above-nav advert on tablet#16282
domlander wants to merge 2 commits into
mainfrom
doml/fix-tablet-top-ad-space

Conversation

@domlander

@domlander domlander commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

What does this change?

At the tablet breakpoint, between 740px and 980px, reserve 160px less vertical space for the top-above-nav advert.

Why?

We currently reserve enough space for a 970x250 (billboard) advert from the tablet breakpoint onwards. However, we only serve this ad size from the desktop breakpoint onwards. The only fixed ad size we serve on tablet, which is also the most commonly served, is the 728x90 (leaderboard) ad size1.

Screenshots

Before After
before after

Screenshares

Before

Screen.Recording.2026-06-29.at.10.02.12.mov

After

Screen.Recording.2026-06-29.at.11.07.29.mov

Footnotes

  1. https://github.com/guardian/commercial/blob/381fab35ac53d782469671f60b05d3eb7fd18025/core/src/ad-sizes.ts#L269

@domlander domlander self-assigned this Jun 29, 2026
@domlander domlander force-pushed the doml/fix-tablet-top-ad-space branch from 0802701 to 11b4ea4 Compare June 29, 2026 09:26
@domlander

Copy link
Copy Markdown
Contributor Author

Leaving this in draft until I have data on the breakdown of the ad sizes served in the top-above-nav slot at the tablet breakpoint. As long as more than half of the ads served in this slot are a maximum of 90px tall (the height of a leaderboard size advert), then it makes sense to make this change. Since we do serve other ad sizes in this slot that are larger than 90px in height, this change should be run as an AB test, to ensure that we are improving CLS for users, as we are intending.

@domlander domlander added the maintenance Departmental tracking: maintenance work, not a fix or a feature label Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Departmental tracking: maintenance work, not a fix or a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant