[Script] [Nightly] Autopopup für Lesezeichenleiste funktioniert nicht mehr...

BrokenHeart

Hallo,

eben festgestellt, dass das oben genannte Script von aborix in der Nightly (bei mir?) nicht mehr funktioniert. Das Überfahren der Verzeichnisse in der Lesezeichenleiste mit der Maus öffnet keine Popups mehr.

Der Grund scheint zu sein, dass die entsprechenden Listener für die mouse-Events gar nicht registriert werden, weil der Befehl

let items = document.querySelectorAll('#PlacesToolbarItems > toolbarbutton[type="menu"]'); in Zeile 41

anscheinend eine leere Node-Liste erzeugt. Ich kann aber im Inspector nicht erkennen, dass sich da irgendwelche Bezeichner oder Abhängigkeiten in der Lesezeichenleiste geändert hätten. Vielleicht habe ich auch nur was übersehen...

Vielleicht hat ja der Autor oder eine anderer schlauer Mensch eine Idee/Lösung...

Edit: Vergessen...hier das Script:

JavaScript

(function() {

  if (!window.gBrowser)
    return;

  let itemToClose = null;
  let timeoutEnter, timeoutLeave;
  let placesContext = document.getElementById('placesContext');

  placesContext.addEventListener('popuphidden', function() {
    if (itemToClose) {
      itemToClose.open = false;
      itemToClose = null;
    };
  });

  function onMouseenter(event) {
    clearTimeout(timeoutLeave);
    timeoutEnter = setTimeout(function() {
      itemToClose = null;
      if (placesContext._view && !placesContext._view._openedMenuButton) {
        placesContext.hidePopup();
      };
      event.target.open = true;
    }, 100);
  };

  function onMouseleave(event) {
    clearTimeout(timeoutEnter);
    timeoutLeave = setTimeout(function() {
      if (placesContext.state == 'closed') {
        event.target.open = false;
        itemToClose = null;
      } else {
        itemToClose = event.target;
      };
    }, 100);
  };

  function addListeners() {
    let items = document.querySelectorAll('#PlacesToolbarItems > toolbarbutton[type="menu"]');
    for (let item of items) {
      item.addEventListener('mouseenter', onMouseenter);
      item.addEventListener('mouseleave', onMouseleave);
    };
  };

  setTimeout(addListeners, 500);

  window.addEventListener('aftercustomization', function() {
    setTimeout(addListeners, 100);
  });

})();

Alles anzeigen

Ok , "Fehler" gefunden.

Wenn man den Timeout in Zeile 48 von 500 auf z.B. 1500 erhöht dann läuft's wieder.

milupo

Zitat von BrokenHeart

Wenn man den Timeout in Zeile 48 von 500 auf z.B. 1500 erhöht dann läuft's wieder.

Wenn sich nur alle Probleme so schnell lösen ließen ...

Sören Hentzschel

Noch besser, wenn es gar kein Timeout gäbe. Denn alles, was von Timeouts wirklich abhängt, ist letztlich nur ein Workaround in der Hoffnung, dass die eingestellte Zeit ausreicht. Das ist also ziemlich fragil, wie auch dieser Fall zeigt. Was gestern noch reichte, reicht heute plötzlich nicht mehr. Für einen anderen Nutzer könnte es aber noch reichen, für wieder einen anderen könnten 1500 auch noch zu wenig sein. 1500 bedeutet, dass man 1 1/2 Sekunden auf etwas warten muss, das ist schon echt lange. Ich bin leider zu weit von diesem Thema entfernt, um eine bessere Lösung anzubieten, aber Timeouts sind halt irgendwo zwischen Glücks- und Geduldspiel: Je niedriger der Wert, desto eher kommt es zu keiner Ausführung (Glücksspiel), je höher, desto länger muss man warten (Geduldsspiel).

BrokenHeart

Zitat von Sören Hentzschel

1500 bedeutet, dass man 1 1/2 Sekunden auf etwas warten muss, das ist schon echt lange. Ich bin leider zu weit von diesem Thema entfernt, um eine bessere Lösung anzubieten, aber Timeouts sind halt irgendwo zwischen Glücks- und Geduldspiel

Im Prinzip hast du recht. Das Problem sehe ich hier aber in der zugrundeliegenden (temporären?) Implementation der Nightly. Bevor ich diesen "Fehler" mit diesem Skript überhaupt realisiert hatte, ist mir aufgefallen, dass es gefühlt Ewigkeiten gedauert hat, bis die Lesezeichenleiste dargestellt wurde. Daher ist die zusätzliche Wartezeit notwendig, weil sich eben die Nightly-Version sehr viel Zeit lässt, bis etwas dargestellt wird. Und wenn die Lesezeichenleiste und ihre Elemente noch nicht 'vorhanden' sind, läuft ein Skript, was diese Elemente anspricht, natürlich ins Leere. Der Zeitpunkt, wann die einzelnen Skripte gestartet werden, kann meines Wissens nicht verändert werden. Aber das Problem hatten wir ja schon bei etlichen Skripten und ist halt der Preis für diese Art des Hackens.

milupo

Ich weiß nicht, ob folgende Zeile hier hilft:

JavaScript

gBrowser.selectedBrowser.removeAttribute('blank');

Es gab mal das Problem, dass bei Clipping alle Dialogfenster nach dem Start der Erweiterung leer waren. Man musste den Timeout-Wert zurechtschaukeln. Irgendjemand (Aris? oder aborix?) hat diese Zeile - statt des setTimeout-Codes - mal eingebracht, die das Problem behob.

BrokenHeart

Zitat von milupo

Ich weiß nicht, ob folgende Zeile hier hilft:

Meinst du diesen Beitrag dazu? Link

So wie ich das verstanden habe, war diese Zeile dazu gedacht, zusätzliche (unbenutzbare) Tabs bei der Verwendung zusätzlicher Symbolleisten zu verhindern.... Aber ich hatte seinerzeit die Diskussion nicht mitverfolgt.

milupo

Zitat von BrokenHeart

Meinst du diesen Beitrag dazu?

Ja.

Zitat von BrokenHeart

So wie ich das verstanden habe, war diese Zeile dazu gedacht, zusätzliche (unbenutzbare) Tabs bei der Verwendung zusätzlicher Symbolleisten zu verhindern...

Das weiß ich auch nicht mehr so genau, ich hatte mir nur gemerkt, dass es da leere Fenster/Tabs in Erweiterungen waren und ich erinnere mich da an Clippings. Das konnte man mit einem veränderten Timeout-Wert lösen, man musste aber experimentieren. bis dann obige Zeile kam. Da konnte man sich den SetTimeout-Code sparen, wenn man obige Zeile in eine eigene Skriptdatei verfrachtete. So habe ich es auch noch. Aber das Gemeinsame mit deinem Problem scheint hier doch nur zu sein, dass beide Timeout-Probleme sind.

aborix

Die Listener werden jetzt registriert, sobald die Lesezeichen-Ordner vorhanden sind:

JavaScript

(function() {

  if (!window.gBrowser)
    return;

  let itemToClose = null;
  let timeoutEnter, timeoutLeave;
  let placesContext = document.getElementById('placesContext');

  placesContext.addEventListener('popuphidden', function() {
    if (itemToClose) {
      itemToClose.open = false;
      itemToClose = null;
    };
  });

  function onMouseenter(event) {
    clearTimeout(timeoutLeave);
    timeoutEnter = setTimeout(function() {
      itemToClose = null;
      if (placesContext._view && !placesContext._view._openedMenuButton) {
        placesContext.hidePopup();
      };
      event.target.open = true;
    }, 100);
  };

  function onMouseleave(event) {
    clearTimeout(timeoutEnter);
    timeoutLeave = setTimeout(function() {
      if (placesContext.state == 'closed') {
        event.target.open = false;
        itemToClose = null;
      } else {
        itemToClose = event.target;
      };
    }, 100);
  };

  function addListeners(items) {
    for (let item of items) {
      item.addEventListener('mouseenter', onMouseenter);
      item.addEventListener('mouseleave', onMouseleave);
    };
  };

  let observer = new MutationObserver(mutations => {
    for (let mutation of mutations) {
      let items = [];
      for (let node of mutation.addedNodes) {
        if (node.type == 'menu') {
          items.push(node);
        };
      };
      addListeners(items);
    };
  });

  addListeners(document.querySelectorAll('#PlacesToolbarItems > toolbarbutton[type="menu"]'));

  observer.observe(document.getElementById('PlacesToolbarItems'), {childList: true});
  setTimeout(() => observer.disconnect(), 5000);

  window.addEventListener('aftercustomization', () =>
    setTimeout(() =>
      addListeners(document.querySelectorAll('#PlacesToolbarItems > toolbarbutton[type="menu"]')),
      100
    )
  );

})();

Alles anzeigen

BrokenHeart

Cool, vielen Dank aborix !

Ich hatte auch schon an einen 'Observer' gedacht, hätte es aber in der Form wohl nicht (in vernünftiger Zeit) hinbekommen.

Das Prinzip lässt sich wohl auch auf zeitkritische Aktionen (vor allem beim Start) in so manch anderen Skripten anwenden...

harff182

Kann nachfolgendes Verhalten bitte mal jemand gegenchecken?

Ich benutze den Code von aborix aus #8.

Ich lasse die Lesezeichen-Symbolleiste mit Strg+Umschalt+B verschwinden und hol sie wieder.

Autopopup funktioniert nicht.

Überseh ich was bzw. mach ich was falsch?

BrokenHeart

Zitat von harff182

Ich lasse die Lesezeichen-Symbolleiste mit Strg+Umschalt+B verschwinden und hol sie wieder.
Autopopup funktioniert nicht.

Das war schon immer so. Durch das Verbergen der Leiste werden wohl auch die registrierten Listener gelöscht und müssen dann beim Anzeigen wieder neu gesetzt werden. So wie es aborix auch beim Aufruf von 'Symbolleiste anpassen' im Skript gemacht hat.

Falls du das unbedingt geändert haben möchtest, dann kannst du ja aborix fragen, ob er das Skript (für dich) anpasst und diese Abfrage (Aktivierung/Deaktivierung) mit einbaut ...

Mich stört dieses Verhalten überhaupt nicht, da ich die Lesezeichenleiste immer aktiviert lasse.

harff182

Zitat von BrokenHeart

Das war schon immer so.
...
Mich stört dieses Verhalten überhaupt nicht, da ich die Lesezeichenleiste immer aktiviert lasse.

Danke, alles Klar...

Kann ich mit Leben, Extrawurst nich notwendig.

aborix

Extrawurst ist glücklicherweise leicht machbar:

JavaScript

(function() {

  if (!window.gBrowser)
    return;

  let itemToClose = null;
  let timeoutEnter, timeoutLeave;
  let placesContext = document.getElementById('placesContext');

  placesContext.addEventListener('popuphidden', function() {
    if (itemToClose) {
      itemToClose.open = false;
      itemToClose = null;
    };
  });

  function onMouseenter(event) {
    clearTimeout(timeoutLeave);
    timeoutEnter = setTimeout(function() {
      itemToClose = null;
      if (placesContext._view && !placesContext._view._openedMenuButton) {
        placesContext.hidePopup();
      };
      event.target.open = true;
    }, 100);
  };

  function onMouseleave(event) {
    clearTimeout(timeoutEnter);
    timeoutLeave = setTimeout(function() {
      if (placesContext.state == 'closed') {
        event.target.open = false;
        itemToClose = null;
      } else {
        itemToClose = event.target;
      };
    }, 100);
  };

  function addListeners(items) {
    for (let item of items) {
      item.addEventListener('mouseenter', onMouseenter);
      item.addEventListener('mouseleave', onMouseleave);
    };
  };

  let observer = new MutationObserver(mutations => {
    let items = [];
    for (let mutation of mutations) {
      for (let node of mutation.addedNodes) {
        if (node.type == 'menu') {
          items.push(node);
        };
      };
    };
    addListeners(items);
  });

  addListeners(document.querySelectorAll('#PlacesToolbarItems > toolbarbutton[type="menu"]'));

  observer.observe(document.getElementById('PlacesToolbarItems'), {childList: true});
  setTimeout(() => observer.disconnect(), 5000);

  window.addEventListener('aftercustomization', () =>
    setTimeout(() =>
      addListeners(document.querySelectorAll('#PlacesToolbarItems > toolbarbutton[type="menu"]')),
      100
    )
  );

  document.getElementById('PersonalToolbar').addEventListener('toolbarvisibilitychange', event => {
    if (event.detail.visible) {
      setTimeout(() =>
        addListeners(document.querySelectorAll('#PlacesToolbarItems > toolbarbutton[type="menu"]')),
        100
      )
    }
  });

})();

Alles anzeigen

harff182

Zitat von aborix

Extrawurst ist glücklicherweise leicht machbar:

Funktionuckelt latürnich

Du hast mir nach einem besch.... gelaufenem Tag noch den Abend versüßt.

BrokenHeart 17. Dezember 2020 um 22:29