Fix Harper's Magazine (modal)

merge-requests/14/head
magnolia1234 1 year ago
parent 4b506a5622
commit 59f2997bc9

@ -6,6 +6,7 @@ Post-release
Add LosAndes.com.ar (Grupo Clarin)
Remove Hln.be (fix obsolete)
Remove Lequipe.fr (fix obsolete)
Fix Harper's Magazine (modal)
Fix NK News (pro)
Fix Quartz (newsletter)

@ -3344,14 +3344,6 @@ else if (matchDomain('griffithreview.com')) {
removeDOMElement(subscribe);
}
else if (matchDomain('harpers.org')) {
let overlay = document.querySelector('div[id^="pum-"]');
removeDOMElement(overlay);
let entry_content = document.querySelectorAll('.entry-content');
for (let elem of entry_content)
elem.setAttribute('style', 'display: block !important');
}
else if (matchDomain('hbr.org')) {
let popup = document.querySelector('.persistent-banner');
removeDOMElement(popup);

@ -857,7 +857,8 @@ var defaultSites = {
useragent: "googlebot"
},
"Harper's Magazine": {
domain: "harpers.org"
domain: "harpers.org",
block_regex: /\/harpers\.org\/wp-content\/themes\/timber\/static\/js\/modal.+\.js/
},
"Harvard Business Review": {
domain: "hbr.org",

@ -31,6 +31,10 @@
"allow_cookies": 1,
"block_regex": "(\\.abril\\.com\\.br\\/.+\\/abril-paywall\\/|cdn\\.ampproject\\.org\\/v\\d\\/amp-subscriptions-.+\\.js)"
},
"Harper's Magazine": {
"domain": "harpers.org",
"block_regex": "\\/harpers\\.org\\/wp-content\\/themes\\/timber\\/static\\/js\\/modal.+\\.js"
},
"Inside Higher Ed": {
"domain": "insidehighered.com",
"allow_cookies": 1,

Loading…
Cancel
Save