Nur die Anzeige der Ordner/Lesezeichenanzahl in einer anderen Farbe darstellen

BrokenHeart

Zitat von Mira_Belle

Funktioniert, schaut genau so aus, wie im Beitrag #52.

Vielen Dank , dann glaube ich jetzt zu wissen, woran es liegt. Anscheinend ist die Anordnung der Kindknoten des Menü-Items anders bzw. hier gibt es noch einen zusätzlichen Eintrag. Ob das auch was mit Win11 zu tun hat, kann ich mir irgendwie nicht vorstellen. Auf jeden Fall gibt es, bei den Leuten, bei denen es bisher nicht funktioniert hat, Unterschiede im Aufbau des Menü-Items. Ich muss den Knoten, hinter dem etwas eingefügt werden soll, anders auswählen...

Zitat von Mira_Belle

Berichtigung!
Habe den Firefox neu gestartet, nun sind die "Einträge" hinten direkt vor den Pfeilen.
So wie es wohl sein soll.

Alles klar. Ja, so sollte es aussehen. Ändert nichts an der Ursache, die es zu beheben gilt...

Zitat von 2002Andreas

PS:
Auch wenn ich das Skript für mich nicht brauche.

Ich werde es ja auch nicht dauerhaft nutzen...

BarbaraZ-

Moin,

bei mir funktioniert das Skript aus #48 in der Menuleiste bei Lesezeichen.

Allerdings lasse ich mir meine Lesezeichenordner mit deren Inhalt in der Sidebar anzeigen.

Wir kann das Skript umgebaut werden, damit es in der Sidebar angezeigt wird?

Code

// Anzahl Dateien im Ordner
// Erstellt von: @BrokenHeart

(function() {

  if (!window.gBrowser)
    return;

  function setFunction() {
    PlacesViewBase.prototype._onPopupShowing = function PVB__onPopupShowing(aEvent) {

      let popup = aEvent.originalTarget;
      this._ensureMarkers(popup);
      if ("_delayedRemovals" in popup) {
        while (popup._delayedRemovals.length) {
          popup.removeChild(popup._delayedRemovals.shift());
        }
      }
      if (popup._placesNode && PlacesUIUtils.getViewForNode(popup) == this) {
        if (!popup._placesNode.containerOpen) {
          popup._placesNode.containerOpen = true;
        }
        if (!popup._built) {
          this._rebuildPopup(popup);
        }
        this._mayAddCommandsItems(popup);
      }

      /* original function end */

      for (let item of popup.children) {
        if (item.localName != 'menu' || item.id?.startsWith('history'))
          continue;
        setTimeout(() => {
          let itemPopup = item.menupopup;
          itemPopup.openPopup();
          itemPopup.hidePopup();
          let menuitemCount = 0;
          let menuCount = 0;
          for (let subitem of itemPopup.children) {
            if (subitem.classList.contains('bookmark-item') && !subitem.disabled && !subitem.hidden) {
              if (subitem.localName == 'menuitem') {
                menuitemCount++;
              } else if (subitem.localName == 'menu') {
                menuCount++;
              }
            }
          }
          if (!item.labelOriginal) {
            item.labelOriginal = item.label;
          }
          item.labelSuffix = '  (' + menuCount + '/' + menuitemCount + ')';
//          item.setAttribute("id", "menu-menuitem-count");
//          item.labelSuffix.setAttribute("color", "red");
//          item.labelSuffix = className("menu-menuitem-count");
//          item.labelSuffix.className = "menu-menuitem-count";
//          menuitemCount.setAttribute("id", "menu-menuitem-count");
          item.label = item.labelOriginal + item.labelSuffix;
        }, 100);
      }

      popup.addEventListener('popuphidden', function onPopuphidden(event) {
        if (event.target != this)
          return;
        popup.removeEventListener('popuphidden', onPopuphidden);
        for (let item of popup.children) {
          if (item.localName == 'menu') {
            if (item.labelOriginal) {
              item.label = item.labelOriginal;
              item.labelOriginal = undefined;
            }
          }
        }
      });

    }

  }

  setTimeout(function() {
      setFunction();
  },50);
  
})();

Alles anzeigen

omar1979

Ich hab jetzt festgestellt, dass bei mir das Script nur funktioniert, wenn ich die lesezeichen-ordner auf der Lesezeichen-Symbolleiste habe.

wenn man die Lesezeichen-ordner auf einer anderen Leiste hat funktioniert es nicht.

Oben im Menü "Lesezeichen" funktioniert es auch nicht. Hauptsächlich NUR im Lesezeichen-Symbolleiste.

2002Andreas

Zitat von omar1979

auf der Lesezeichen-Symbolleiste habe.

So steht es ja auch in dem Skript

Mira_Belle

Zitat von 2002Andreas

Zitat von omar1979

auf der Lesezeichen-Symbolleiste habe.

So steht es ja auch in dem Skript

Er benutzt ja aber immer noch das Skript von Beitrag #34 oder #48

BrokenHeart

Zitat von omar1979

auf der Lesezeichen-Symbolleiste habe.

Nur das, nämlich die Lesezeichenleiste, ist bis jetzt auch implementiert. Welche Version des Skriptes hast du denn getestet (#BeitragsNummer)?

2002Andreas

Zitat von Mira_Belle

Er benutzt ja aber immer noch das Skript von

Auch da wird nur die Lesezeichensymbolleiste angesprochen.

let persToolBar = document.getElementById('PersonalToolbar');

omar1979

Weiss Leider nicht mehr welches script aber sicher mal eins von der 1.ten Seite hier in diesem thread.

2002Andreas

Zitat von omar1979

Weiss Leider nicht mehr welches script

Evtl. das

Beitrag

RE: mehrzeilige Lesezeichen Symbolleiste

[…]

Gerne doch

Wenn du magst, dann teste bitte mal diesen Code:

(Quelltext, 31 Zeilen)

2002Andreas

9. Januar 2023 um 11:31

Wobei das kein Skript ist

omar1979

Ich bräuchte eigentlich dieses Script ja auch nicht, wollte aber es testen und mithelfen wo es funkt und wo nicht.

2002Andreas

Zitat von omar1979

wollte aber es testen und mithelfen

omar1979

Zitat von 2002Andreas

Evtl. das

Nein, nicht das. Das von BrokenHeart

Das hier:

Code

(function() {

    if (!window.gBrowser)
        return;

    function setFunction() {

        const css =`
            .countClass::after {
                content: attr(data-value); 
                color: red;
            }
        `;

        const sss = Cc['@mozilla.org/content/style-sheet-service;1'].getService(Ci.nsIStyleSheetService);
        const uri = Services.io.newURI('data:text/css,' + encodeURIComponent(css));
        sss.loadAndRegisterSheet(uri, sss.AGENT_SHEET);

        let persToolBar = document.getElementById('PersonalToolbar');
        
        persToolBar.addEventListener('popupshowing', function onPopupShowing(aEvent) {
            let popup = aEvent.originalTarget;

            for (let item of popup.children) {
                if (item.localName != 'menu' || item.id?.startsWith('history'))
                    continue;
                setTimeout(() => {
                  let itemPopup = item.menupopup;
                  itemPopup.openPopup();
                  itemPopup.hidePopup();
                  let menuitemCount = 0;
                  let menuCount = 0;
                  for (let subitem of itemPopup.children) {
                    if (subitem.classList.contains('bookmark-item') && !subitem.disabled && !subitem.hidden) {
                      if (subitem.localName == 'menuitem') {
                        menuitemCount++;
                      } else if (subitem.localName == 'menu') {
                        menuCount++;
                      }
                    }
                  }
                  
                  let label = item.childNodes[1]; //label.menu-iconic-text
      
                  label.classList.add('countClass');
                  let strCountOut = "  (" + menuCount + "/" + menuitemCount + ")"
                  label.setAttribute('data-value', strCountOut);

                }, 100);
            }
        });
    }

    setTimeout(function() {
        setFunction();
    },50);
  
})();

Alles anzeigen

2002Andreas

Zitat von omar1979

Nein, nicht das.

Ok, dann hatte ich deine Antwort falsch verstanden, sorry dafür.

omar1979

Zitat von 2002Andreas

sorry dafür.

Nichts zu entschuldigen. Ich schreibe ja immer wirres Zeug.

BarbaraZ-

Hat jemand eine Idee zu meiner Frage aus Beitrag #62?

2002Andreas

Zitat von BarbaraZ-

Hat jemand eine Idee zu meiner Frage

Immer mit der Ruhe

Dazu musst das Skript erst wieder angepasst werden.

Und zur Zeit wird noch getestet, warum es bei einigen User nicht funktioniert.

BarbaraZ-

Oki, Danke.

Dachte schon, es wurde überlesen.

Dann warte ich gerne ab.

BrokenHeart

Mit diesem Script sollte es jetzt auch im Lesezeichen-Menu funktionieren:

JavaScript

(function() {

    if (!window.gBrowser)
        return;
    
    setTimeout(function() {
        setFunction();
    },50);

    function setFunction() {

        const css =`
            .countClass::after {
                content: attr(data-value); 
                color: red;
            }
        `;

        const sss = Cc['@mozilla.org/content/style-sheet-service;1'].getService(Ci.nsIStyleSheetService);
        const uri = Services.io.newURI('data:text/css,' + encodeURIComponent(css));
        sss.loadAndRegisterSheet(uri, sss.AGENT_SHEET);

        let bookMenu = document.getElementById('bookmarksMenu');
        let persToolBar = document.getElementById('PersonalToolbar');
        
        bookMenu.addEventListener('popupshowing', onPopupShowing ); 
        persToolBar.addEventListener('popupshowing', onPopupShowing ); 
    }
    
    function onPopupShowing(aEvent) {
        
        let popup = aEvent.originalTarget;

        for (let item of popup.children) {
            if (item.localName != 'menu' || item.id?.startsWith('history'))
                continue;
            setTimeout(() => {
              let itemPopup = item.menupopup;
              itemPopup.openPopup();
              itemPopup.hidePopup();
              let menuitemCount = 0;
              let menuCount = 0;
              for (let subitem of itemPopup.children) {
                if (subitem.classList.contains('bookmark-item') && !subitem.disabled && !subitem.hidden) {
                  if (subitem.localName == 'menuitem') {
                    menuitemCount++;
                  } else if (subitem.localName == 'menu') {
                    menuCount++;
                  }
                }
              }
              
              let label = item.childNodes[1]; //label.menu-iconic-text
  
              label.classList.add('countClass');
              let strCountOut = "  (" + menuCount + "/" + menuitemCount + ")"
              label.setAttribute('data-value', strCountOut);

            }, 100);
        }
    }
    
})();

Alles anzeigen

Für Leute, die Probleme mit der Anzeige der Werte haben oder die Anzeige rechtsbündig vor dem Pfeil haben wollen, habe ich momentan nur diese Lösung:

In obigem Skript in der Zeile 53 ein 3 eintragen, also

let label = item.childNodes[3]; //label.menu-iconic-text

Dann werden die Werte eben weiter rechts angezeigt, was mir persönlich sehr viel besser gefällt. Natürlich Geschmackssache.

Zitat von BarbaraZ-

Dachte schon, es wurde überlesen.

Nein, natürlich nicht , aber ich habe eine Antwort, die dir wahrscheinlich nicht gefallen wird: -> '42'.

Nein im Ernst, ich habe keinerlei Ahnung, wie das funktionieren soll.

2002Andreas

Zitat von BrokenHeart

Mit diesem Script sollte

Ist es auch

omar1979

Zitat von BrokenHeart

In obigem Skript in der Zeile 53 ein 3 eintragen, also

let label = item.childNodes[3]; //label.menu-iconic-text

Ja ich finde es sieht besser aus mit einer 3.

Was jetzt nurnoch fehlt, ist es, dass es für Mira_Belle auch im Sidebar funktioniert.

Für mich auch in den Zusatzleisten. Also eine Variante, die in allen Leisten funktioniert.

Bis jetzt auch eine tolle Leistung BrokenHeart. DankeSchön.

UuuPs es war BarbaraZ-, die es im Sidebar wollte. Sorry Mira_Belle