Das Script "QuickLinkButton-about-config.uc.js" funktioniert nicht mehr.

Dharkness

Hallo zusammen,

JavaScript

// QuickLinkButton-about-config.uc.js
// aboutconfig-button.uc.js Fallback Test Mai 2025
// Eigenes Icon erwartet in Profilordner/chrome/icons , icons Ordner falls noetig erstellen.
// TEST, die Zweite !!!!!

(function() {
    if (location != 'chrome://browser/content/browser.xhtml') return;

    try {
        CustomizableUI.createWidget({
            id: 'aboutconfig_button',
            type: 'custom',
            defaultArea: CustomizableUI.AREA_NAVBAR,
            onBuild: function(aDocument) {

                let buttonicon = "wrench-s2.png";
                let fallbackIcon = "chrome://global/skin/icons/warning.svg";

                let iconFile = Services.dirsvc.get("ProfD", Ci.nsIFile);
                iconFile.append("chrome");
                iconFile.append("icons");
                iconFile.append(buttonicon);

                let iconExists = iconFile.exists();
                let iconURL = iconExists
                    ? "file://" + iconFile.path.replace(/\\/g, "/")
                    : fallbackIcon;

                // Nur das Fallback-Icon einfärben
                let iconStyle = 'list-style-image: url("' + iconURL + '");';
                if (!iconExists) {
                    iconStyle += ' fill: red;';
                }

                let toolbaritem = aDocument.createElementNS(
                    'http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul',
                    'toolbarbutton'
                );
                let props = {
                    id: 'aboutconfig_button',
                    class: 'toolbarbutton-1 chromeclass-toolbar-additional',
                    label: 'About:Config',
                    tooltiptext: 'About:Config',
                    style: iconStyle
                };

                for (let p in props)
                    toolbaritem.setAttribute(p, props[p]);
                return toolbaritem;
            }
        });
    } catch (e) { }

    document.getElementById('aboutconfig_button')?.addEventListener('click', event => {
        if (event.button === 0) {
            openTrustedLinkIn("about:config", "tab");
        }
    });

})();

Alles anzeigen

obiges Script funktioniert leider nicht mehr, Button ist vorhanden, hat aber keine Funktion. Funktion des Scripts ist, das öffnen von about:config in einem neue Tab.

2002Andreas

Zitat von Dharkness

obiges Script funktioniert leider nicht mehr

Kein Problem hier

Mira_Belle

Bitte testen und gegebenenfalls anpassen. Siehe Kommentare

JavaScript

// QuickLinkButton-about-config.uc.js
// Das Script erstellt einen Button, der einen Tab mit "About:Config" öffnet.

// Source file https://www.camp-firefox.de/forum/thema/140072/?postID=1282691#post1282691

/* ----------------------------------------------------------------------------------- */
/*     Zu beachten ist, dass die Grafiken sich im richtigen Ordner befinden müssen     */
/*            %appdata%\Mozilla\Firefox\Profiles\"Profilname"\chrome\icons             */
/* ----------------------------------------------------------------------------------- */

(function() {

  if (!window.gBrowser)
    return;

  const
  // ■■ START UserConfiguration ■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■
           id = 'aboutconfig_button', // Id des Buttons
         link = 'about:config', // Linkziel des Buttons
        label = 'Quicklink: about:config', // Bezeichnung des Buttons
  tooltiptext = 'Quicklink:\nabout:config',
  // Icon-------------------------------------------------------
         icon = 'wrench-s2.png',  // [Name.Dateiendung] des Symbols
     iconPath = '/chrome/icons/', // Pfad zum Ordner der das Icon beinhaltet
  // ■■ END UserConfiguration ■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■
  curProfDir = PathUtils.toFileURI(PathUtils.join(PathUtils.profileDir));

	if (Services.prefs.getBoolPref('svg.context-properties.content.enabled') == false) {
		Services.prefs.setBoolPref('svg.context-properties.content.enabled', true );
	}
  //BUTTON
	try {
    CustomizableUI.createWidget({
      id: id,
      defaultArea: CustomizableUI.AREA_NAVBAR,
      label: label,
      tooltiptext: tooltiptext,

      onCreated: (button) => {
        // Optik
        button.style.MozContextProperties = 'fill, stroke, fill-opacity';
      //  button.style.listStyleImage = 'url("' + curProfDir + iconPath + icon + '")';
        button.style.listStyleImage = 'url("chrome://global/skin/icons/heart.svg")';  // ist nur zum testen !!
        button.style.minWidth = 'fit-content';   
      }
    });
	} catch(e) {};

  //click
  (function add_button() {
    const button = document.getElementById(id);
    if (button) {
      // Das wird nur gebraucht, wenn eine SVG-Datei verwendet wird
      button.style.fill = '#e45aff';  // hier die Farbe setzen
      /* Oder diese Lösung wenn ganz sicher nur das Icon-Element angesprochen werden soll */ 
      /*
      let iconNode = button.querySelector('.toolbarbutton-icon');
      if (iconNode) {
        iconNode.style.fill = '#ff5a79';
      } */
      // Alles bis zum Kommentar wird nur gebraucht, wenn eine SVG-Datei verwendet wird

      // click
      button.addEventListener('click', (event) => {
        if (event.button === 0) {
          openTrustedLinkIn(link, "tab");
        }
      });
    } else {
      // Wenn der Button noch nicht existiert, später nochmal probieren
      setTimeout(add_button, 100);
    }
  })();

})();

Alles anzeigen

Dharkness

Zitat von 2002Andreas

Kein Problem hier

Hm, andere Script funktionieren hier ohne Probleme, z.B. Restart-Scripts, oder das Profil-Backup-Script, das Start-any-aplication-Script, das Extra-Config-Menü-Script und das Firefox-Menü-Script und andere.

Zitat von Mira_Belle

Bitte testen und gegebenenfalls anpassen.

Das läuft ohne Probleme, Dankeschön.

Horstmann

Zitat von Dharkness

obiges Script funktioniert leider nicht mehr, Button ist vorhanden, hat aber keine Funktion. Funktion des Scripts ist, das öffnen von about:config in einem neue Tab.

Kann es sein, dass der Button nicht funktioniert hatte, direkt nachdem du ihn im Anpassenfenster in eine Symbolleiste gezogen hast?

Dharkness

Zitat von Horstmann

Kann es sein, dass der Button nicht funktioniert hatte, direkt nachdem du ihn im Anpassenfenster in eine Symbolleiste gezogen hast?

Nein, der Button war schon sehr lange funktionstüchtig in der entsprechenden Leiste abgelegt.

milupo

Das Fragezeichen in Zeile 54 kommt mir etwas suspekt vor. Es gibt zwar wirklich den Fragezeichen-Operator, aber irgendwie passt er für mich hier nicht. Er ist wohl nur eine andere Schreibweise für die if … else-Konstruktion. Die aber kommt aber erst danach.

Dharkness

Zitat von milupo

Das Fragezeichen in Zeile 54 kommt mir etwas suspekt vor.

Mag sein, aber das Script hat hier in der Tat so ohne Probleme bis einschließlich 148.x funktioniert.

Sören Hentzschel

Das Fragezeichen ist korrekt. Damit wird überprüft, ob das Element überhaupt existiert.

JavaScript

document.getElementById('aboutconfig_button')?.addEventListener('click', event => {
  // code …
});

Ist das Gleiche wie:

JavaScript

if (
  document.getElementById('aboutconfig_button') !== undefined &&
  document.getElementById('aboutconfig_button') !== null
) {
  document.getElementById('aboutconfig_button').addEventListener('click', event => {
    // code …
  });
}

Auf die eine oder andere Weise muss man das prüfen, wenn man nicht garantieren kann, dass das Element existiert. Ansonsten bricht das Script beim Aufruf von addEventListener mit einem Fehler ab, wenn es das Element nicht gibt.

2002Andreas

Zitat von Dharkness

bis einschließlich 148.x funktioniert.

Funktioniert hier auch in Fx 151 Nightly einwandfrei:

milupo

Zitat von Sören Hentzschel

Auf die eine oder andere Weise muss man das prüfen, wenn man nicht garantieren kann, dass das Element existiert. Ansonsten bricht das Script beim Aufruf von addEventListener mit einem Fehler ab, wenn es das Element nicht gibt.

Ah, danke. Wieder etwas gelernt.

Mira_Belle

Dharkness Gerne doch.
Im Übrigen sind alle meine Buttonscripte so aufgebaut, da ich vor langer Zeit ähnliche Probleme hatte.
Und die Wartung ist, wenn alle Scripte "gleich" sind, erheblich einfacher.

Dharkness

Zitat von 2002Andreas

Funktioniert hier auch in Fx 151 Nightly einwandfrei:

Das ist in der Tat sehr merkwürdig, aber vielleicht liegt es auch an der Leiste, in der der Button liegt, ist teste es mal eben.

Nope, es liegt nicht an der Leiste, es funktioniert hier schlicht und ergreifend nicht.

Horstmann

Zitat von Sören Hentzschel

Das Fragezeichen ist korrekt. Damit wird überprüft, ob das Element überhaupt existiert.
Auf die eine oder andere Weise muss man das prüfen, wenn man nicht garantieren kann, dass das Element existiert. Ansonsten bricht das Script beim Aufruf von addEventListener mit einem Fehler ab, wenn es das Element nicht gibt.

Wobei in Mira's Code aus #3 die (function add_button ....) mit dem Timeout dieses Problem abdecken sollte; wir hatten das und weitere diesbezügliche Button Probleme mal hier diskutiert.

Sören Hentzschel

Ja, aber nicht wegen des Timeouts. Das löst unterschiedliche Probleme. Der Timeout-„Hack” stellt sicher, dass der Code ausgeführt wird, auch wenn das Element erst später verfügbar ist. Das macht der obige Code nicht. Der prüft nur, ob das Element existiert. Diese Überprüfung ist auch im Code von Mira_Belle vorhanden und hat nichts mit dem Timeout zu tun, sondern ist die Abfrage if (button) an dieser Stelle. Sowohl undefined als auch null sind „falsy”, womit diese Schreibweise kürzer für das ist, was ich ausführlich geschrieben habe, um die exakte Bedeutung des „Optional Chaining”-Operators zu erklären.

Mira_Belle

Im Prinzip könnte auch in meinem Skript
document.getElementById(id)?.addEventListener('click', event => { stehen.

Nur weiß ich nicht, wie ich dann mit der "Farbe" machen soll.

Horstmann

Zitat von Sören Hentzschel

Ja, aber nicht wegen des Timeouts.

Das ist soweit klar; wollte eher darauf hinweisen, dass es mit eigenen Buttons noch weitere Aspekte zu beachten gibt, was Mira's Version auch abdeckt.

Sören Hentzschel

Zitat von Mira_Belle

Im Prinzip könnte auch in meinem Skript
document.getElementById(id)?.addEventListener('click', event => { stehen.

Könnte es, dann bräuchtest du die zusätzliche if-Abfrage nicht mehr. Aber mit der Schreibweise gibt es eben auch kein else. Und zwei Mal prüfen, ist unnötig. Insofern sehe ich da keinen Bedarf, etwas zu verändern.

Mira_Belle

Zitat von Sören Hentzschel

Könnte es, dann bräuchtest du die zusätzliche if-Abfrage nicht mehr. Aber mit der Schreibweise gibt es eben auch kein else.

Habe ich getestet, nur so aus Neugierde.
Funktioniert nicht, da es dann eine Schleife gibt und der Tab zig mal geöffnet wird.
Aber das mit der "Farbe" geht auch an anderer Stelle. Musste ich einfach ausprobieren.

Sören Hentzschel

Wie gesagt, es gibt kein else für diese Schreibweise. Und du kannst die Methode nicht jedes Mal ohne Überprüfung sich selbst aufrufen lassen. Das gibt dann natürlich eine Endlosschleife. Wenn du es über ein if-Konstrukt löst, wie es aktuell der Fall ist, gibt es keinen Grund, das darin ein weiteres Mal zu prüfen. Deswegen sage ich ja, dass ich keinen Bedarf sehe, an deinem Code etwas zu ändern. Diese Schreibweise ist dann praktisch, weil viel kürzer, wenn du wirklich nur auf die Existenz prüfen musst, um dann direkt darauf etwas ausführen zu können, und du das Ergebnis ansonsten ignorieren kannst.

Das Script "QuickLinkButton-about-config.uc.js" funktioniert nicht mehr.

Ähnliche Themen

Das Script New_Addonbar.uc funktioniert nicht mehr.

Das "DeepL-Skript" funktioniert nicht mehr richtig!

Script funktioniert seit Update auf FF 142 nicht mehr

Seit FF139 geht das script nicht mehr: allow_search_oneoff_with_empty_text.uc.js

Script "External-Application.uc.js" funktioniert nicht mehr

Deepl Context Script funktioniert nicht mehr in Nightly