Div. Skripte funktionieren im aktuellem Nightly nicht mehr

Zitat von Horstmann

Sieht nicht einfach aus, da haste recht.

Sieht es nicht?

Aus:

element.setAttribute('onEvent', 'function();');

wird:

element.addEventListener('event', () => {
  function();
});

Das ist das ganze Prinzip, welches sich die ganze Zeit wiederholt. Manchmal entstehen wie gesagt Irritationen dadurch, dass es unterschiedliche Schreibweisen gibt. Oder der Code ist etwas „komplizierter“ dadurch, dass Attribute in einer Schleife gesetzt werden, ein Teil der Logik in eine eigene Methode ausgelagert ist und / oder Funktionen ineinander verschachtelt werden. Dann reicht es nicht mehr, nur zu kopieren. Dann muss man den Weg verstanden haben, genau hinsehen und das Gelernte unter abweichenden Voraussetzungen anwenden. Aber das Prinzip bleibt am Ende immer das Gleiche.

Zitat von Mira_Belle

Ich hab's, ich fress 'en Besen! Dharkness FuchsFan

Danke, aber alle meine Bemühungen scheitern, weil Dein Script nicht identisch ist mit dem meinen.

Das bei Endor gelistete wäre der Ursprung, aber das bekomme ich nicht gerichtet.

userChrome.js/Firefox 129/Appmenu.uc.js at master · Endor8/userChrome.js

Skripte für die Firefox-Erweiterung userChromeJS. Contribute to Endor8/userChrome.js development by creating an account on GitHub.

github.com

Zitat von FuchsFan

aber das bekomme ich nicht gerichtet.

Auch hier immer das Gleiche:

ExternalAppBtn.setAttribute("onclick", "event.preventDefault();event.stopPropagation();");
key.setAttribute('oncommand', 'document.getElementById("AMpopup").openPopup();');
appItem.setAttribute('oncommand', "Appmenu.exec(this.path, this.args);");
configItem.setAttribute('oncommand', config.command);

=>

ExternalAppBtn.addEventListener('click', event => {
  event.preventDefault();
  event.stopPropagation();
});

key.addEventListener('command', () => {
  document.getElementById('AMpopup').openPopup();
});

appItem.addEventListener('command', function () {
  Appmenu.exec(this.path, this.args);
});

configItem.addEventListener('command', () => {
  Function(config.command)();
});

Nachtrag: Nummer 4 korrigiert, ich hatte mir das Original-Script nicht gut genug angesehen. Das funktioniert tatsächlich etwas anders.

Zitat von Sören Hentzschel

Auch hier immer das Gleiche:

Danke, Sören Hentzschel , dann starte ich mal den nächsten Versuch.

Zitat von Sören Hentzschel

Zitat von Horstmann

Sieht nicht einfach aus, da haste recht.

Sieht es nicht?

Aus:

JavaScript

element.setAttribute('onEvent', 'function();');

wird:

JavaScript

element.addEventListener('event', () => {
  function();
});

Das ist das ganze Prinzip, welches sich die ganze Zeit wiederholt.

Hab ich gesehen, aber wenn du dir hier Zeilen 282+, und 319+ anschaust, da hat Mia ein paar Dinge umgebastelt.

z.B.:

        if (item.command) {
        //  menuItem.setAttribute("oncommand", item.command);
          menuItem.addEventListener("command", function() {
            eval(item.command);
          });

Ob oder wieso das so umgesetzt sein soll kann ich nicht einschätzen, das ist der komplizierte Teil für mich; vermutlich aus den Gründen die du gerade beschrieben hast, und halt wg. mangelden JS Kentnissen.

Du hast schon Recht. Das ist tatsächlich zur Abwechslung mal ein Fall, der sich nicht automatisch aus den anderen Fällen ergibt, sondern etwas Zusatzwissen benötigt.

In den anderen Fällen hast du immer JavaScript-Funktionen, die innerhalb eines Strings angegeben sind, erkennbar an den Anführungszeichen. Zum Beispiel:

key.setAttribute('oncommand', 'document.getElementById("AMpopup").openPopup();');

Die kannst du einfach aus den Anführungszeichen herausnehmen und dann ausführen:

key.addEventListener('command', () => {
  document.getElementById('AMpopup').openPopup();
});

In dem Fall hast du keine Anführungszeichen:

menuItem.setAttribute("oncommand", item.command);

Das item.command ist quasi eine Variable, die das auszuführende Script beinhaltet. Nun könnte man meinen, das macht keinen Unterschied. Und es wäre, basierend auf den bisherigen Erkenntnissen, folgender Gedanke naheliegend:

menuItem.addEventListener('command', function () {
  item.command;
});

Also gleich, wie es die ganze Zeit mit den Funktionen geschieht. Aber das funktioniert so nicht. Eine Variable kann man nicht einfach so in den Raum stellen, damit etwas passiert.

Die Funktion eval() führt einen Text, in dem Fall also den Inhalt von item.command als Script aus:

menuItem.addEventListener('command', function () {
  eval(item.command);
});

Folgendes würde auch funktionieren:

menuItem.addEventListener('command', function () {
  Function(item.command)();
});

Das ist sogar die bessere Variante, da deutlich performanter.

Zitat von Sören Hentzschel

Die Funktion eval() führt einen Text, in dem Fall also den Inhalt von item.command als Script aus:

Sollte eval eigentlich nicht mehr verwendet werden oder verwechsele ich da etwas? Vor Jahren haben wir deswegen auch Skripte umgeschrieben.

eval() gilt vor allem aus Sicherheitsgründen als absolutes No-Go in JavaScript - auf Grund genau dieser Eigenschaft, die hier genutzt wird: aus einem String ausführbares JavaScript zu machen. In der Hinsicht ist der Weg über Function auch nicht besser. Die andere Sache ist, dass für eval() sämtliche Optimierungen der Engine außer Kraft gesetzt sind, was es sehr langsam macht.

Was die Sicherheit betrifft: Wenn du das auf einer Website nutzt und dann möglicherweise noch Nutzer-Eingaben verarbeitest, kann das fatale Folgen haben. In dem Fall denke ich, dass das nicht so viel ausmacht, weil keine Fremdeingaben verarbeitet werden und die Kontrolle vollständig beim Anwender liegt. Würde jemand Drittes Zugriff auf das System haben und das Script manipulieren wollen, wäre das bei einem direkten Aufruf der Funktionen ohne eval() genauso möglich. Dann wird eben das manipuliert.

Möchte man auf eval() oder Function() verzichten, muss man das Script umbauen und ähnlich lösen, wie in den anderen Fällen. Aus Performance-Sicht wäre das empfehlenswert. Und dass Mozilla früher oder später die Verwendung von eval() unterbindet, ist durchaus denkbar. Dafür gibt es ebenfalls eine CSP-Direktive. Zukunftssicherer wäre es definitiv ohne. Aber aktuell funktioniert es damit noch.

Danke Auf Anraten von

Beitrag

RE: Div. Skripte funktionieren im aktuellem Nightly nicht mehr

[…]

Auch hier immer das Gleiche:

(Quelltext, 4 Zeilen)

=>

(Quelltext, 16 Zeilen)

Nachtrag: Nummer 4 korrigiert, ich hatte mir das Original-Script nicht gut genug angesehen. Das funktioniert tatsächlich etwas anders.

Sören Hentzschel

14. Januar 2025 um 12:05

funktioniert dieses AppMenu.uc.js wieder

Beitrag

RE: userChrome.js Scripte für den Fuchs (Diskussion)

[…]

Teste bitte mal:

(Quelltext, 366 Zeilen)

2002Andreas

2. Dezember 2020 um 15:39

Zitat von Sören Hentzschel

Zitat von FuchsFan

aber das bekomme ich nicht gerichtet.

Auch hier immer das Gleiche:

JavaScript

ExternalAppBtn.setAttribute("onclick", "event.preventDefault();event.stopPropagation();");
key.setAttribute('oncommand', 'document.getElementById("AMpopup").openPopup();');
appItem.setAttribute('oncommand', "Appmenu.exec(this.path, this.args);");
configItem.setAttribute('oncommand', config.command);

=>

JavaScript

ExternalAppBtn.addEventListener('click', event => {
  event.preventDefault();
  event.stopPropagation();
});

key.addEventListener('command', () => {
  document.getElementById('AMpopup').openPopup();
});

appItem.addEventListener('command', function () {
  Appmenu.exec(this.path, this.args);
});

configItem.addEventListener('command', () => {
  Function(config.command)();
});

Alles anzeigen

Nachtrag: Nummer 4 korrigiert, ich hatte mir das Original-Script nicht gut genug angesehen. Das funktioniert tatsächlich etwas anders.

Ganz herzlichen Dank, funktioniert wieder, muss nur noch alles wieder hübsch machen.

Zitat von Sören Hentzschel

Die Frage hatte ich übersehen.

Kein Problem...

Zitat von Sören Hentzschel

Man kann natürlich auch diesen Weg gehen. Dann muss man halt in dieser Funktion den Namen der Eigenschaft prüfen und für die on*-Eigenschaften den Listener ensprechend setzen statt das Attribut. Dann darf man nur nicht vergessen, den Namen dynamisch anzupassen, denn das on gehört ja nicht in den Event-Namen und der erste Buchstabe muss klein sein.

Ja, so habe ich es ja in dem Beispielcode auch gemacht. Ganz verstehe ich nicht, was du meinst . 'oncommand' und 'onclick' werden nicht in die Attribute geschrieben, sondern stattdessen eben der entsprechende EventListener hinzugefügt mit 'command' und 'click'. Für alle anderen Eigenschaften des übergebenen Objekts wird die vorgegebene Standardzuweisung el.setAttribute(n, attr[n]);  verwendet.

Zitat von Sören Hentzschel

Wieso das $ verwendet wird, weiß ich nicht. Wenn ich so etwas sehe, vermute ich als erstes, dass der ursprüngliche Entwickler aus der jQuery-Entwicklung kommt und das daher hat. Aber vielleicht hatte der Entwickler auch eine ganz eigene Konvention, wann er das Dollar-Zeichen verwendet. Das müsste man den Entwickler mal fragen. Schlimmer finde ich, dass Funktionen als $, $A und $C benannt sind. Schön kurz, ja, aber nicht lesbar an den Stellen, an denen die Funktionen verwendet werden.

Ja sehe ich auch so. In einigen Scriptsprachen wird auch dieses $ vor dem Variablennamen verwendet. Wahrscheinlich noch nicht ganz in der 'neuen' Sprache angekommen, da nutzt man dann immer wieder gerne gewohnte Strukturen aus der "Heimatsprache". Habe da auch so ein paar Baustellen...

Würde mich schon interessieren, ob der Code auch funktioniert. Aber ich könnte Endor auch verstehen, wenn er es nicht nochmal testen möchte, wenn die andere Herangehensweise funktioniert hat.

Zitat von Sören Hentzschel

Das item.command ist quasi eine Variable, die das auszuführende Script beinhaltet. ......

Aha, herzlichen Dank für die Erklärung!

Das sind so die kleinen Stolpersteine, die einem als Amateur eine Menge Zeit kosten können.

Zitat von BrokenHeart

Ja, so habe ich es ja in dem Beispielcode auch gemacht. Ganz verstehe ich nicht, was du meinst . 'oncommand' und 'onclick' werden nicht in die Attribute geschrieben, sondern stattdessen eben der entsprechende EventListener hinzugefügt mit 'command' und 'click'. Für alle anderen Eigenschaften des übergebenen Objekts wird die vorgegebene Standardzuweisung el.setAttribute(n, attr[n]); verwendet.

Ich hatte nicht gesehen, dass du schon Code dafür hattest. Ich hatte mir nur den Teil des Beitrags angesehen, der an mich gerichtet war. In diesem Thema war so viel in so kurzer Zeit los und ich so beschäftigt, dass ich vermutlich nicht mal die Hälfte des gesamten Themas gelesen habe. 😬

Die Umsetzung, die ich im Kopf hatte, hätte so ausgesehen, dass der Name des Attributs ausgelesen wird und wenn dieser mit „on“ beginnt, der entsprechende Code kommt und für den Event-Namen die ersten zwei Buchstaben gestrichen und der dritte klein geschrieben wird. Das wäre dann auch flexibel genug für andere Events als nur diese zwei. Wenn aber ohnehin nur „oncommand“ und „onclick“ verwendet werden, hast du mit deiner Umsetzung sowieso schon alle Fälle abgedeckt. Das passt also, wie es ist.

Danke Sören Hentzschel

          menuItem.addEventListener('command', () => {
            Function(item.command)();
          });

Das habe ich übernommen und eingepflegt!
Aber bei der darauffolgenden Anpassung bleibt alles wie gehabt?

PS: Der ganze Codeschnipsel.

        if (item.command) {
        //  menuItem.setAttribute("oncommand", item.command);
        /*  menuItem.addEventListener("command", function() {   // oder so  */
        /*  menuItem.addEventListener("command", () => {
            eval(item.command);
          });   */

        // !!! Das ist eine bessere und performantere Variante !!! 
          menuItem.addEventListener('command', () => {
            Function(item.command)();
          });

        } else if (item.path) {
        //  menuItem.setAttribute("oncommand","Appmenu.exec(this.path, this.args);");
        /*  menuItem.addEventListener("command",  function () {   // oder so  */
          menuItem.addEventListener("command",  () => {  
            Appmenu.exec(this.path, this.args)
        });

2002Andreas Schau Dir dieses Skript einmal an, das nutze ich!
Wenn Du daran Änderungen haben möchtest, gerne werde ich sie durchfahren.

// ==UserScript==
// @name           Fenster_schließen.us.js
// @charset        UTF-8
// @author         @mira
// @version        v2022.09.10
// @note           Entwickelt von Mira Bellenbaum (mit Hilfe des Fuchsforums 09.10.2022)
// @reviewURL      https://www.camp-firefox.de/forum/thema/135346/?postID=1210102#post1210102
// Source          https://www.camp-firefox.de/forum/thema/135346/?postID=1210195#post1210195 


(function() {
  if (location.href !== 'chrome://browser/content/browser.xhtml') {
    return;
  }
  const cs_Geschlossenen_Tab_wieder_öffnen = document.getElementById('context_undoCloseTab');
  const cs_Tab_schließen = document.getElementById('context_closeTab');
  const cs_Andere_Tabs_schließen = document.getElementById('context_closeOtherTabs');
  const cs_Linke_Tabs_schließen = document.getElementById('context_closeTabsToTheStart');
  const cs_Rechte_Tabs_schließen = document.getElementById('context_closeTabsToTheEnd');
  const cs_Tab_klonen = document.getElementById('context_duplicateTab');
  const cs_An_Anfang_verschieben = document.getElementById('context_moveToStart');
  const cs_An_Ende_verschieben = document.getElementById('context_moveToEnd');
  const cs_In_neues_Fenster_verschieben = document.getElementById('context_openTabInWindow');
  const cs_Alle_Tabs_auswählen = document.getElementById('context_selectAllTabs');
  const cs_Tab_als_Lesezeichen_hinzufügen = document.getElementById('context_bookmarkTab');
  const cs_Add = document.getElementById('fbcbdc47-ebd2-42f0-b2bb-7cc902ec2f40_-menuitem-_add-current-tab-menu');
  const cs_separator = document.createXULElement('menuseparator');
  
  tabContextMenu.insertBefore(cs_Andere_Tabs_schließen, cs_Geschlossenen_Tab_wieder_öffnen);
  tabContextMenu.insertBefore(cs_Rechte_Tabs_schließen, cs_Andere_Tabs_schließen);
  tabContextMenu.insertBefore(cs_Linke_Tabs_schließen, cs_Rechte_Tabs_schließen);
  tabContextMenu.insertBefore(cs_Tab_als_Lesezeichen_hinzufügen, cs_Tab_klonen);
  tabContextMenu.insertBefore(cs_In_neues_Fenster_verschieben, cs_Alle_Tabs_auswählen);
  tabContextMenu.insertBefore(cs_An_Ende_verschieben, cs_In_neues_Fenster_verschieben);
  tabContextMenu.insertBefore(cs_An_Anfang_verschieben, cs_An_Ende_verschieben);
  tabContextMenu.insertBefore(cs_Tab_schließen, cs_Add);
  tabContextMenu.insertBefore(cs_separator, cs_Tab_schließen);

  //Components.utils.import("resource://gre/modules/Services.jsm");
  let ss =  Components.classes["@mozilla.org/content/style-sheet-service;1"].getService(Components.interfaces.nsIStyleSheetService);
  let custom_label = {
    init: function() {
      let uri = Services.io.newURI("data:text/css;charset=utf-8," + encodeURIComponent(`
       
        @namespace html url("http://www.w3.org/1999/xhtml");
       
        #context_moveToStart label,
        #context_moveToEnd label,
        #context_openTabInWindow label,
        #context_closeOtherTabs label {
          display: none !important;
        }

        #context_moveToStart::after {
          content: "Tab an Anfang verschieben" !important;
          font-size: 13px !important          
        }

        #context_moveToEnd::after {
          content: "Tab an Ende verschieben" !important;
          font-size: 13px !important
        }

        #context_openTabInWindow::after {
          content: "Tab in neues Fenster verschieben" !important;
          font-size: 13px !important
        }

        #context_closeOtherTabs::after {
          content: "Alle anderen Tabs schließen" !important;
          font-size: 13px !important
        }

        #context_bookmarkTab,
        #context_toggleMuteTab,
        #context_pinTab,
        #context_duplicateTab,
        #context_moveTabOptions,
        #context_selectAllTabs,
        #context_closeTabOptions,
        #context_undoCloseTab,
        #_fbcbdc47-ebd2-42f0-b2bb-7cc902ec2f40_-menuitem-_add-current-tab-menu {
          display: none !important;
        }
        #tabContextMenu > menuseparator:nth-child(37) {
          display: none !important;
        }

      `), null, null);
      ss.loadAndRegisterSheet(uri, ss.AGENT_SHEET);
    }
  };
  custom_label.init();
})();

2002Andreas Dieses Skript funktioniert korrekt

(async (el, url) => {
   if (location != url) return;
   el.before(document.getElementById("context_closeTabsToTheStart"));
   el.before(document.getElementById("context_closeTabsToTheEnd"));
   el.before(document.getElementById("context_closeOtherTabs"));
   el.style.visibility="collapse";
})(document.getElementById("context_closeTabOptions"), "chrome://browser/content/browser.xhtml"); 

Bitte mal helfen, im Appmenu-Script gibt es den Eintrag zum öffnen von Downloads. Der eingetragene Wert command reagiert nicht mehr. Wie müsste der jetzt richtig lauten?

name: 'Downloads',
command: "BrowserDownloadsUI();",
id: 'AMdownloads',

Danke, hat sich erledigt.

command:"PlacesCommandHook.showPlacesOrganizer('Downloads');",