Skip to content

Commit e4f85cc

Browse files
authored
Merge pull request #7669 from readthedocs/davidfischer/set-ad-config-values
Set ad configuration values if using explicit placement
2 parents ba966e3 + 463f453 commit e4f85cc

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

readthedocs/core/static-src/core/js/doc-embed/sponsorship.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,11 @@ function create_ad_placement() {
3232
var offset;
3333

3434
if ($(EXPLICIT_PLACEMENT_SELECTOR).length > 0) {
35+
$(EXPLICIT_PLACEMENT_SELECTOR).attr("data-ea-publisher", "readthedocs");
36+
$(EXPLICIT_PLACEMENT_SELECTOR).attr("data-ea-manual", "true");
37+
if ($(EXPLICIT_PLACEMENT_SELECTOR).attr("data-ea-type") !== "image" && $(EXPLICIT_PLACEMENT_SELECTOR).attr("data-ea-type") !== "text") {
38+
$(EXPLICIT_PLACEMENT_SELECTOR).attr("data-ea-type", "readthedocs-sidebar");
39+
}
3540
return $(EXPLICIT_PLACEMENT_SELECTOR);
3641
} else if ($(OLD_EXPLICIT_PLACEMENT_SELECTOR).length > 0) {
3742
selector = OLD_EXPLICIT_PLACEMENT_SELECTOR;

0 commit comments

Comments
 (0)