Сценарий Greasemonkey, не удаляющий элемент - PullRequest
Новая
       28

Сценарий Greasemonkey, не удаляющий элемент

0 голосов
/ 24 мая 2018

Большая часть этого сценария в основном вырезана и вставлена ​​из сценариев других людей, которые работают на них, но у меня странная проблема с ошибкой запуска .remove или .removeChild.На этом этапе сценарий завершает работу обработчика пользовательских сценариев. 

// ==UserScript==
// @name     Strip Gocomics Sidebar
// @version  1
// @grant    none
// @include  https://www.gocomics.com/*
// ==/UserScript==

window.addEventListener('load', setkillsidebar);

function setkillsidebar() {
  var interval = Math.random() * 5000 + 1000;
  setTimeout(killsidebar, interval);
}

function killsidebar() {
  console.log("Start Session");
  // const adSidebar = document.querySelectorAll('.gc-container-fluid .layout-2col-sidebar, .gc-page-header--hero .layout-2col-sidebar');
  var adSidebar = document.getElementsByClassName('.gc-container-fluid .layout-2col-sidebar, .gc-page-header--hero .layout-2col-sidebar');
  console.log("Got Elements " + adSidebar.length );
  if (adSidebar) {
    console.log("Found SideBar");
    var myParent = adSidebar.parentNode;
    console.log("Made Parent");
    // myParent.remove();
    adSidebar.parentNode.removeChild(adSidebar);
    console.log("Stripped SideBar");
    var interval = Math.random() * 5000 + 1000;
    console.log("Timer Time " + interval );
    setTimeout(killsidebar, interval);
    console.log("Set Timer");
  }
}

Поэтому, добавив элементы console.log, я получаю в веб-консоли Firefox следующее:

  • Начать сеанс
  • Получил элементы
  • Найден SideBar
  • Сделан родительским

И это завершение, у меня смерть либо в .removeили .removeChild, поэтому либо я что-то делаю неправильно, либо у меня возникла проблема с настройкой безопасности, которая не позволяет мне удалять элементы с веб-страниц, о которых мне никто не говорил.

И для более интересногоинформация, хотя название этого поста - Greasemonkey, с Tampermonkey это тоже не получается.

PS Это используется в дополнение к некоторому Стильному CSS, который позволяет мне иметь большой комический вид на маленьком мониторе.Не имеет значения, работает Стильный или нет.

1 Ответ

0 голосов
/ 24 мая 2018

Существует много проблем с этим пользовательским сценарием, но они в основном сводятся к следующему: вам нужно записать сообщения об ошибках в консоли и прогуглить функции, которые их вызывают.
Например:

  • Это не то, как getElementsByClassName работает.
  • querySelectorAll не возвращает узел.
  • parentNode и removeChild оба действуют на один узел.

Также: второй setTimeout не нужен.И прослушиватель событий load также (вероятно) излишен.

Вот сценарий с исправленными недостатками: 

// ==UserScript==
// @name     Gocomics, Strip Sidebar
// @match    https://www.gocomics.com/*
// @version  2
// @grant    none
// ==/UserScript==

var interval = Math.random () * 5000 + 1000;
setTimeout (killsidebar, interval);

function killsidebar () {
    //-- querySelector() and querySelectorAll () are not the same.
    var adSidebar = document.querySelector ('.gc-container-fluid .layout-2col-sidebar, .gc-page-header--hero .layout-2col-sidebar');
    if (adSidebar) {
        adSidebar.parentNode.removeChild (adSidebar);
    }
}

Хотя этот сценарий, вероятно, будет работать лучше: 

// ==UserScript==
// @name     Gocomics, Strip Sidebar
// @match    https://www.gocomics.com/*
// @version  2
// @require  https://ajax.googleapis.com/ajax/libs/jquery/2.1.0/jquery.min.js
// @require  https://gist.github.com/raw/2625891/waitForKeyElements.js
// @grant    GM_addStyle
// @grant    GM.getValue
// ==/UserScript==
//- The @grant directives are needed to restore the proper sandbox.

waitForKeyElements (
    ".gc-container-fluid .layout-2col-sidebar, .gc-page-header--hero .layout-2col-sidebar",
    removeNode
);

function removeNode (jNode) {
    jNode.remove ();
}

Используется waitForKeyElements - который быстрее и надежнее, чем прямой setTimeout.

...