Skip to content

Commit 63b91e5

Browse files
committed
Revert "Reintroduce the stickybox on wide screens (#604)"
This reverts commit bc62388.
1 parent cd79b92 commit 63b91e5

File tree

1 file changed

+2
-13
lines changed

1 file changed

+2
-13
lines changed

src/ethicalads.js

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -198,22 +198,11 @@ export class EthicalAdsAddon extends AddonBase {
198198
if (elementToAppend) {
199199
elementToAppend.append(placement);
200200
}
201-
} else if (window.innerWidth > 1300 && Math.random() > 0.5) {
202-
// https://ethical-ad-client.readthedocs.io/en/latest/#stickybox
203-
placement.setAttribute("data-ea-type", "image");
204-
placement.setAttribute("data-ea-style", "stickybox");
205-
placement.setAttribute("id", "readthedocs-ea-image-stickybox");
206-
this.addEaPlacementToElement(placement);
207-
// `document.body` here is not too much relevant, since we are going to
208-
// use this selector only for a floating stickybox ad
209-
const elementInsertBefore = document.body;
210-
elementInsertBefore.insertBefore(
211-
placement,
212-
elementInsertBefore.lastChild,
213-
);
214201
} else {
215202
// Default to a text ad appended to the root selector when no known placement found
216203
placement.setAttribute("data-ea-type", "text");
204+
// TODO: Check this placement on the dashboard,
205+
// and see how this is performing.
217206
const docToolName = docTool.getDocumentationTool();
218207
const idSuffix = docToolName ? `-${docToolName}` : "";
219208
placement.setAttribute("id", `readthedocs-ea-text-footer${idSuffix}`);

0 commit comments

Comments
 (0)