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

Horstmann

Zitat von omar1979

Ich hab versucht, so wie du beschrieben hast es zu machen.

Code

let fp-statusbar-1 = document.getElementById('fp-statusbar-1');
        let bmbMenu = document.getElementById('bookmarks-menu-button');
        let bookMenu = document.getElementById('bookmarksMenu');
        let persToolBar = document.getElementById('PersonalToolbar');

        if(fp-statusbar-1)
            fp-statusbar-1.addEventListener('popupshowing', onPopupShowing );        
        if(bmbMenu)
            bmbMenu.addEventListener('popupshowing', onPopupShowing ); 
        if(bookMenu)
            bookMenu.addEventListener('popupshowing', onPopupShowing ); 
        if(persToolBar)
            persToolBar.addEventListener('popupshowing', onPopupShowing );

Alles anzeigen

das fp-statusbar-1 ist die ID von dieser Leiste, die jetzt der Lesezeichen-Ordner drauf ist auf dem Bild, aber es funkt nicht.

Hängt vermutlich von dem Script ab das die Leiste erzeugt, das könntest du mal posten.

Ich könnte mir auch vorstellen dass die ID nicht in der Form benutzt werden sollte - die ID gehört wohl nur an eine Stelle vom Code, die Bindestriche könnten auch ein Problem sein ausserhalb der Klammern; hab aber absolut keine Ahnung von Javascript...

Ganz wild geraten, evtl. mal sowas probieren? :

JavaScript

        let newbarMenu = document.getElementById('fp-statusbar-1');

        if(newbarMenu)
        newbarMenu.addEventListener('popupshowing', onPopupShowing );

grisu2099

Zitat von grisu2099

Könnte mir vorstellen, das sich auch andere dann darauf stürzen.

Ich hab doch geahnt, daß das wieder ausartet...

Aber immer wieder toll, was wir hier gemeinsam hinkriegen können.

omar1979

Zitat von Horstmann

Ganz wild geraten, evtl. mal sowas probieren? :

Du bist mein Retter! Dankeschön.

Horstmann

Zitat von omar1979

Prima!

Zitat von grisu2099

Ich hab doch geahnt, daß das wieder ausartet...

Der Weg ist ja das Ziel, wieder dazugelernt.

grisu2099

Ich habe die Anzeige noch mal durchgemischt...

Falls jemand Interesse hat:

JavaScript

(function() {

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

    function setFunction() {

        const css =`
            .countClass::after {
                content: attr(data-value);
                color: yellow;
                padding-right: 20px;
            }
        `;

        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[3]; //label.menu-iconic-text
  
              label.classList.add('countClass');
              let strCountOut =  + menuCount + " Ordner / " + String(menuitemCount).padStart(2, '0') + " Links"
              label.setAttribute('data-value', strCountOut);

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

Alles anzeigen

Nochmal Danke an alle Beteiligten.

Endor

Hallo BrokenHeart .

Danke für das tolle Script. Funktioniert bestens.

Darf ich diese Version bei Github hoch laden?

Beitrag

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

Ich glaube doch ein Fehler meinerseits. Wenn der Lesezeichen-Button überhaupt nicht vorhanden ist bzw. eingefügt wurde, dann ist das entsprechende Objekt natürlich auch NULL und es wird eine Exception geschmissen.

Hier der geänderte Code:

(Quelltext, 68 Zeilen)

BrokenHeart

7. Juli 2023 um 18:44

Mfg.
Endor

Boersenfeger

Zitat von grisu2099

Ich habe die Anzeige noch mal durchgemischt...

Kann man die Bezeichnungen "Ordner" und "Links" auch entfernen?

2002Andreas

Zitat von Boersenfeger

Kann man die Bezeichnungen "Ordner" und "Links" auch entfernen?

So war es in den ersten Skripten aus diesem Thread

Beitrag

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

Nochmal als Test direkt von der bei mir funktionierenden Version aus NP++ kopiert:

(Quelltext, 65 Zeilen)

BrokenHeart

7. Juli 2023 um 18:09

grisu2099

Kann man

Nimmst du diese Version:

Code

(function() {

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

    function setFunction() {

        const css =`
            .countClass::after {
                content: attr(data-value); 
//                color: lightgreen;
                color: yellow;
                padding-right: 11px;
            }
        `;

        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');
        let bookMenu = document.getElementById('bookmarksMenu');
        
        persToolBar.addEventListener('popupshowing', onPopupShowing );
        bookMenu.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[3]; //label.menu-iconic-text
  
              label.classList.add('countClass');
              let strCountOut =  + menuCount + " / " + String(menuitemCount).padStart(2, '0')
              label.setAttribute('data-value', strCountOut);

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

Alles anzeigen

Boersenfeger

Prima, jetzt kloppt mir nur wieder ein anderer Code dazwischen, den muss ich erstmal suchen... auf die Schnelle wurde er nicht identifiziert

grisu2099

Nein, diese Anzeige ist mit dem Skript aus #151 richtig!

Wenn du bei den Ordnern die führende 0 willst, mußt du Zeile 58 wie folgt ändern:

Code

              let strCountOut =  + String(menuCount).padStart(2, '0') + " Ordner / " + String(menuitemCount).padStart(2, '0') + " Links"

Boersenfeger

Nee,

Zitat von Boersenfeger

Es geht um die Auflistung vorn in schwarz...

..als Verursacher konnte ich dieses zusätzliche Script verantwortlich machen:

JavaScript

(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

Nun passt es. Danke für deine Hilfe!

grisu2099

Das links hatte ich ganz übersehen...

2 Skripte für den gleichen Zweck sind meistens kontraproduktiv...

BrokenHeart

Zitat von Boersenfeger

..als Verursacher konnte ich dieses zusätzliche Script verantwortlich machen:

Das war ja das alte Skript, was dann noch parallel dazu lief. Da verwundert es nicht, dass dann die Zahlenangaben auch doppelt im Popup stehen...

Zitat von Endor

Darf ich diese Version bei Github hoch laden?

Hallo Endor,

natürlich, sehr gerne...

Zitat von Dharkness

Im Lesezeichen-Menü bzw. in der Lesezeichensymbolleiste werden über lange Untermenüs leider automatisch ans Ende des Untermenüs gescrollt.

Da muss ich jetzt auch leider sagen, dass das wirklich ein 'Showstopper' ist . Ich habe auch ein paar sehr große Verzeichnisse und dieses Verhalten würde mich auf die Dauer schon nerven. Der Grund ist, dass alle Child-Popups (also die Unterverzeichnisse)

kurz geöffnet werden um die jeweiligen Eigenschaften (Menu/Menuitem) auszulesen und anzuzeigen.

Eine Lösung dafür habe ich momentan auch nicht. Aber die gesamte Lesezeichen-Struktur, mit allen Informationen, ist ja bekannt und entweder schon direkt im Speicher oder über ein SQL-Select über die 'places.sql' zu erreichen. Vielleicht gibt es da noch eine bessere Lösung, als alle Popups einmal zu öffnen und damit das Scrollproblem zu verursachen?

Mitleser

grisu2099

Zitat von grisu2099

Wenn du bei den Ordnern die führende 0 willst, mußt du Zeile 58 wie folgt ändern

Hast du das mal getestet?

Funzt bei mir nämlich nicht.

Siehe Erklärung aus Beitrag #140.

Endor

Hallo BrokenHeart .

Danke!!

@all

Hat jemand einen Vorschlag wie man dieses Script nennen soll?

Zbs.

Bookmarkscount.uc.js
Bookmarkscounter.uc.js

Count-bookmarks.uc.js

Was meint Ihr?

Mfg.
Endor

harff182

Zitat von Endor

Hat jemand einen Vorschlag wie man dieses Script nennen soll?

FolderContentCounter.uc.js

grisu2099

Läuft bei mir als "BookmarkCount.uc.js"

Endor

Finde ich sehr gut.

Mal abwarten was sonst noch für Vorschläge kommen.

Danke harff182 !

Mfg.
Endor

2002Andreas

Lesezeichenzähler.uc.js