guardian / guardian/dotcom-rendering

Adjust ad insertion script to give space around interactives

Open
#14,719 0 comments 0 reactions 1 assignee Claimed by @frederickobrien View on GitHub
fix
Dominant language
TypeScript
Stars
274
Forks
34
Avg merge
2d 22h
Merged PRs (30d)
121

Description

Ads in interactive articles sometimes disrupt the narrative flow by breaking up key information blocks. E.g.:

Image

Have spoken with Commercial and no initial objections to updating the ad placeholder script used on apps:

https://github.com/guardian/dotcom-rendering/blob/main/dotcom-rendering/src/model/enhance-ad-placeholders.ts#L154

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.