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

Horstmann

Zitat von Mira_Belle

Mh.

Also, ich habe keine Probleme mehr!
Es funktioniert super mit Version #2!

Wir schreiben aber hier ja Code für möglichst alle User und Fälle.
Für mich klappt #2 nicht sauber, käme also nicht in mein Script.

So, hier mal was zum Testen für Windozer und alle anderen; löst bei mir am Mac das Animationsproblem beim Lesezeichen-Menü-Button, und schmalem Popup.
Menüleiste -> Lesezeichen müsstet ihr mal selber testen.

JavaScript

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

        let bmbMenuPop = document.getElementById('BMB_bookmarksPopup');      // neu

        if(bmbMenu){
           bmbMenu.addEventListener('popupshown', onPopupShowing );    // popupshowing => popupshown
           bmbMenuPop.addEventListener('popupshowing', onPopupShowing );  // neu
           }

        if(bookMenu)
           bookMenu.addEventListener('popupshown', onPopupShowing );    // popupshowing => popupshown

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

Alles anzeigen

Oder so:

JavaScript

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

        let bmbMenuPop = document.getElementById('BMB_bookmarksPopup');      // neu
        
        if(bmbMenu)
           bmbMenu.addEventListener('popupshown', onPopupShowing );    // popupshowing => popupshown

        if(bmbMenuPop)
           bmbMenuPop.addEventListener('popupshowing', onPopupShowing );   //  neu

        if(bookMenu)
           bookMenu.addEventListener('popupshown', onPopupShowing );    // popupshowing => popupshown

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

Alles anzeigen

Mira_Belle

Zitat von Horstmann

Wir schreiben aber hier ja Code für möglichst alle User und Fälle.

Da will ich Dir auch gar nicht widersprechen

Es ist nur so, dass ich bei allen Events von 'popupshowing' auf 'popupshown' umstellen musste.
Bei bmbMenu funktionierte auch noch alles mit "doppeltem" Aufruf,
jedoch beim Aufruf des Lesezeichenmenüs (bookMenu) schon nicht mehr.
Da wurden die Counter gar nicht mehr angezeigt.
Und nutze ich da nur 'popupshowing', tritt halt das beschriebene Problem auf.
In der Lesezeichenleiste gleiches! Also überall 'popupshown' in Verwendung.

Horstmann Und Du hast den gleichen Fehler wie ich begangen!
BrokenHeart hat mich darauf hingewiesen! Klammern, es fehlen Klammern.

Hier noch einmal Deine zweite Version mit diesen Klammern.

JavaScript

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

        let bmbMenuPop = document.getElementById('BMB_bookmarksPopup');      // neu
        
        if(bmbMenu) {
           bmbMenu.addEventListener('popupshown', onPopupShowing );    // popupshowing => popupshown
		  }
        if(bmbMenuPop) {
           bmbMenuPop.addEventListener('popupshowing', onPopupShowing );   //  neu
		  }
        if(bookMenu) {
           bookMenu.addEventListener('popupshown', onPopupShowing );    // popupshowing => popupshown
		  }
        if(persToolBar) {
            persToolBar.addEventListener('popupshowing', onPopupShowing );
		  }

Alles anzeigen

Und da diese Version ohne Nebenwirkungen funktioniert,
werde ich das so übernehmen.
Nur als Kommentar, anstatt /* neu */ schreibe ich da /* extra für Mac erforderlich */

So etwas hatten wir schon einmal in einem Skript, dass es für Mac extra Code sein musste.

Boersenfeger

Ich nutze im Nightly die Version, die Endor derzeit hostet. Ich habe nun alle Versionsfragmente, die hier im Thread vorgestellt wurden durch und doch muss ich das Menü Lesezeichen 2x aufrufen, damit die Anzahl angezeigt wird. Während der gleichen Sitzung braucht es dann nur noch einmal den Aufruf. Ergänzend nutze ich das Script AutoPopUp, so dass ich nur drüberhovern muss, um das Menü zu öffnen.

Hier AutoPopUp.uc.js

Code

/* AutoPopUp */

// ==UserScript==
// @name           AutoPopup.uc.js
// @description    Auto popup menulist/menupopup
// @compatibility  Firefox 142.0a1
// @author         GOLF-AT, modified by gsf & aborix
// @version        2025.06.30
// ==UserScript==

(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);
    };
  };

  addListeners(document.querySelectorAll('#main-menubar > menu'));

})();

Alles anzeigen

und hier BookmarkCount.uc.js

CSS

//bookmark_count.uc.js, RC_5 Rudolf
//Ende Juni 2025
//Zeigt Anzahl der Lesezeichenordner und Links an in Lesezeichenpopups
//RC_5 R: Verblassen bei 0 leichter einstellbar, Fehlerkorrekturen, Standardfarben sollten sich jetzt an Themes anpassen, Aufraeumen
//basiert auf Script von BrokenHeart =>
//https://www.camp-firefox.de/forum/thema/136572-nur-die-anzeige-der-ordner-lesezeichenanzahl-in-einer-anderen-farbe-darstellen/?postID=1269879#post1269879
//Release Candidate 5.R =>
//https://www.camp-firefox.de/forum/thema/136572-nur-die-anzeige-der-ordner-lesezeichenanzahl-in-einer-anderen-farbe-darstellen/?postID=1274447#post1274447

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

    //Einstellungen Javascript =>
    //Nach Script Aenderungen Neustart mit Start-Cache loeschen => about:support

    //Eigenes Icon erwartet in Profilordner/chrome/icons, icons Ordner falls noetig erstellen
    //Eigene Icons eintragen, falls vorhanden
    let ProfilePath = PathUtils.toFileURI(PathUtils.join(PathUtils.profileDir, 'chrome', 'icons'));
    let bm_icon1 = 'YourFolderIcon.svg';        //  Custom  Folder Icon
    let bm_icon2 = 'YourLinkIcon.svg';          //  Custom  Link Icon
    //Variablen zur Auswahl der Icons; kein Auto Fallback falls eigenes Icon nicht vorhanden
    let bm_icon_folder = 0;      //  Ordner: 0 Firefox Fallback Icon / 1 eigenes Icon
    let bm_icon_link = 0;        //  Links:  0 Firefox Fallback Icon / 1 eigenes Icon

    //Variablen zur Auswahl der Klammern
    let bm_bracket = 0;           //  0 keine Klammern / 1 runde Klammern / 2 eckige Klammern
    //Variablen zur Auswahl der Reihenfolge
    let bm_order = 0;             //  0 Icons links Ziffern rechts / 1 Ziffern links Icons rechts

    //Variablen zur Auswahl der Farben
    let bm_colors = 0;            //  0 Standardfarben / 1 eigene Farben benutzen

    //Verblassen bei Zahl = 0
    let bm_opacity_null = 1;      //  0 kein Verblassen / 1 Verblassen Zahl, Icon, Trennlinie / 2 Verblassen nur Zahl

    //Variablen zur Auswahl der Trennlinie
    //  0 ohne Trennlinie
    //  1 ohne Trennlinie / Zaehler #1 = 0 => Zaehler #1 + Icon #2 wird versteckt
    //  2 mit Trennlinie
    //  3 mit Trennlinie nur bei Zaehler #1 > 0 / Zaehler #1 = 0 => Zaehler #1 + Icon #2 wird versteckt
    let trenner = 2;
    //  Trennlinie Inhalt, beliebige Textzeichen
    let trennzeichen = '|';

    function setFunction() {
        const css =`

       /*
       Einstellungen CSS , OPTIONAL =>
       #A Zaehler (Counter) anpassen,
       #B Feinabstimmung allgemein,
       #C Feinabstimmung Zentrierung Trennlinie,
       #D Eigene Farben, Verblassen,
       #E Allgemeines,
       #F Optionale Extras.
       */

       /*** User Einstellungen ***/

       /* ------ A ------ */

       /** Feste Breite der beiden Counter, abhaengig von Anzahl Ziffern, AUTO Anpassung für mit/ohne Klammern.
          Falls noetig => Werte erhoehen NUR bis Icons untereinander auf gleicher Hoehe sind **/

       /* Beispiel: 2 Ziffern plus Klammern, Systemfont Mac */
       #bmContent:is(
       [data-value1^='['],
       [data-value1^='(']) {
          --bm_width_one: 2.1em;
          --bm_width_two: 2.1em;
          }
       /* Beispiel: 2 Ziffern ohne Klammern, Systemfont Mac */
       #bmContent {
          --bm_width_one: 1.3em;
          --bm_width_two: 1.3em;

       /* ------ B ------ */

       /** Abstaende / Groessen fuer die Counter = Basisanpassungen **/

          /* Abstand mittig zwischen Counter #1 <=> Counter #2 / wird x2 benutzt */
          --bm_margin_mid: 8px;

          /* Groesse Icons = 16px Firefox Standard */
          --bm_icon_size: 16px;

          /* Abstand zwischen Icon und Ziffer */
          --bm_space: 4px;
          }

       /* ------ C ------ */

       /** Trennlinie , fuer trenner = 2/3 **/

       /* Feinabstimmung Zentrierung Mitte / Standard = 0px */
       #bmContent.trennclass {
          --bm_divider: 0px;
          }
       /* Fuer Reihenfolge/ Order = 1 */
       #bmContent.trennclass.order_2 {
          --bm_divider: 0.3em;
          }
       /* Element Trennlinie allgemein */
       #trennID {
          display: flex;
          align-items: center;
          font-size: calc(1em - 1px);        /* Hoehe, optional */
          padding-bottom: 1px;               /* ausrichten vertikal, optional */
          color: var(--trenn_color, currentColor);
          }

       /* ------ D ------ */

       /* Eigene Farben, fuer bm_colors = 1 */
       /* Farben Text / kompatible svg Icons aendern , anpassen nach Belieben */
       /* Info: HSL Farben: https://www.w3schools.com/css/css_colors_hsl.asp */
       /* Ordner Icon, Zahl / Links Icon, Zahl / Trennlinie */
       #bmContent.bm_my_colors {
          --folder_fill:  hsl(190, 60%, 20%, 1);
          --folder_color: hsl(190, 20%, 30%, 1);
          --link_fill:    hsl(35, 100%, 20%, 1);
          --link_color:   hsl(35, 60%, 30%, 1);
          --trenn_color:  hsl(250, 60%, 30%, 1);
          }

       /* Grad des Verblassens bei Zahl = 0, fuer bm_opacity_null = 1/2 */
       #bmContent {
          --bm_opacity: 50%;
          }

       /* ------ E ------ */

       /** Counter gesamt **/

       #bmContent {
          display: flex !important;
          -moz-context-properties: fill, fill-opacity;
          margin-left: auto !important;         /* Gesamt rechtsbuendig */
          /*margin-right: -8px !important;*/    /* Abstand rechts zu Pfeil > ist evtl. OS abhaengig, optional */
          height: var(--bm_icon_size);          /* Layout passt sich Icon Groesse an */
          padding-left: 8px;                    /* min. Abstand links Gesamt für enge Popups */

       /* Font Aenderungen , optional */
          /* font-family: Aenderungen nicht empfohlen => einheitlichen Standardfont behalten */
          /*font-size: 12px !important;*/       /* font-size */
          /*font-weight: 200 !important;*/      /* font-weight */
          }

       /* ------ F ------ */

       /** Optionale Extras **/

       /* Text vertikal ausrichten */
       /*
       #bmContent::after,
       #bmContent::before {
          padding-top: 3px;
          }
          */

       /* Pfeil rechts > bei Beidem = 0 */
       .bookmark-item.bm_chevron_00 > .menu-right,
       .bookmark-item.bm_chevron_00::after {
          fill: hsl(30, 100%, 50%, 1) !important;
          fill-opacity: 0.7 !important;
          }

       /* Hide empty Popup */
       .bookmark-item.bm_chevron_00 > menupopup[emptyplacesresult] {
          display: none !important;
          }

       /** Pfeil rechts > allgemein **/
       /*
       menu.bookmark-item .menu-right,
       menu.bookmark-item::after {
           fill: hsl(210, 100%, 50%, 1) !important;
           fill-opacity: 1 !important;
           }
           */

       /*** User Einstellungen Ende ***/


       /*** Feste Werte ***/

       /** Reihenfolge Varianten Icons / Ziffern **/
       /* order 0: Icons links / Ziffern rechts */
       #bmContent {
            --bm_padding_inline: calc(var(--bm_icon_size) + var(--bm_space)) 0;
            --bm_bg_position: center left;
            }
       /* order 1: Ziffern links / Icons rechts */
       #bmContent.order_2 {
            --bm_padding_inline: 0 calc(var(--bm_icon_size) + var(--bm_space));
            --bm_bg_position: center right;
            }

       /** Icons Varianten **/
       /* Eigene Icons, falls Icons existieren im icons Ordner */
       #bmContent {
            --bm_icon_image_1: url("${ProfilePath}/${bm_icon1}");
            --bm_icon_image_2: url("${ProfilePath}/${bm_icon2}");
            }
       /* Firefox Icons Fallback */
       #bmContent.icon_fallback_folder {
            --bm_icon_image_1: url("chrome://global/skin/icons/folder.svg");
            }
       #bmContent.icon_fallback_link {
            --bm_icon_image_2: url("chrome://browser/skin/bookmark-hollow.svg");
            }

       /** Anpassungen bei Zahl = 0 **/
       /* Verstecken #1 automatisch bei Trennlinie 1/3 , beides = 0 */
       #bmContent:is(.trennclass_1, .trennclass_3)::before {
            display: none !important;
            }
       /* Verstecken Icon #2 , Trennlinie 1/3 , beides = 0 */
       #bmContent.Null_1.Null_2:is(.trennclass_1, .trennclass_3)::after {
            /*content: "X";*/             /* eigener Inhalt */
            background-image: linear-gradient(transparent, transparent);
            }
       /* Verblassen Icon + Zahl, fuer #1 , #2 , Alles Zahl = 0 */
       #bmContent.opacity_Null:is(.Null_1:not(.Null_2), .Null_1.Null_2)::before,
       #bmContent.opacity_Null:is(.Null_2:not(.Null_1), .Null_1.Null_2)::after,
       #bmContent.opacity_Null:is(.Null_1, .Null_2) #trennID {
            filter: opacity(var(--bm_opacity));
            }
       /* Verblassen nur Zahl */
       #bmContent.opacity_Null_B:is(.Null_1:not(.Null_2), .Null_1.Null_2)::before {
            -webkit-text-fill-color: color-mix(in srgb, var(--folder_color, currentColor) var(--bm_opacity), transparent) !important;
            }
       #bmContent.opacity_Null_B:is(.Null_2:not(.Null_1), .Null_1.Null_2)::after {
            -webkit-text-fill-color: color-mix(in srgb, var(--link_color, currentColor) var(--bm_opacity), transparent) !important;
            }

       /** Counters (Zaehler) **/
       /* Counter #1 Ordner */
       #bmContent::before {
            content: attr(data-value1);
            display: flex;
            min-width: fit-content;
            width: var(--bm_width_one);
            padding-inline: var(--bm_padding_inline);
            align-items: center;
            justify-content: flex-end;
            margin-right: var(--bm_margin_mid);
            background-image: var(--bm_icon_image_1);
            background-position: var(--bm_bg_position);
            background-repeat: no-repeat;
            background-size: var(--bm_icon_size);
            color: var(--folder_color, currentColor) !important;
            fill: var(--folder_fill, currentColor) !important;
            }
       /* Counter #2 Links */
       #bmContent::after {
            content: attr(data-value2);
            display: flex;
            min-width: fit-content;
            width: var(--bm_width_two);
            padding-inline: var(--bm_padding_inline);
            align-items: center;
            justify-content: flex-end;
            margin-left: calc(var(--bm_margin_mid) - var(--bm_divider, 0px));
            background-image: var(--bm_icon_image_2);
            background-position: var(--bm_bg_position);
            background-repeat: no-repeat;
            background-size: var(--bm_icon_size);
            color: var(--link_color, currentColor) !important;
            fill: var(--link_fill, currentColor) !important;
            }
		`;
        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 bmbMenu = document.getElementById('bookmarks-menu-button');
        let bookMenu = document.getElementById('bookmarksMenu');
        let persToolBar = document.getElementById('PersonalToolbar');
        if(bmbMenu)
            bmbMenu.addEventListener('popupshowing', onPopupShowing );
        if(bookMenu)
            bookMenu.addEventListener('popupshowing', onPopupShowing );
        if(persToolBar)
            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.hidden = true;
			        itemPopup.collapsed = true;
              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++;
                  }
                }
              }
			        itemPopup.hidden = false;
			        itemPopup.collapsed = false;
              //Eigenes Element für Zaehler
              let bmCounta = item.childNodes[1];
              bmCounta.innerHTML = '';
              let bmVario = document.createElement("bmElement");
              bmVario.id = "bmContent";
              bmCounta.appendChild(bmVario);
              //Zaehler Design Optionen =>  ohne/ mit: runde, eckige Klammern
              if (bm_bracket === 0) {
                  strCountOut1 = '' + menuCount + '';
                  strCountOut2 = '' + menuitemCount + '';
                  }
              if (bm_bracket === 1) {
                  strCountOut1 = '(' + menuCount + ')';
                  strCountOut2 = '(' + menuitemCount + ')';
                  }
              if (bm_bracket === 2) {
                  strCountOut1 = '[' + menuCount + ']';
                  strCountOut2 = '[' + menuitemCount + ']';
                  }
              bmVario.setAttribute('data-value1', strCountOut1);
              bmVario.setAttribute('data-value2', strCountOut2);
              // Trennlinie Element
              let trennelementVario = document.createElement("trennelement");
              trennelementVario.id = "trennID";
              // Trennlinie Varianten
              if (trenner === 2 || trenner === 3 && menuCount !== 0) {
                  bmVario.appendChild(trennelementVario);
                  trennelementVario.textContent = trennzeichen;
                  bmVario.classList.add('trennclass');
                  }
              if (trenner === 3 && menuCount === 0) {
                  bmVario.classList.add('trennclass_3');
                  }
              if (trenner === 1 && menuCount === 0) {
                  bmVario.classList.add('trennclass_1');
                  }
              // Extra class item / ganzes menu fuer 00
              if (menuCount === 0 && menuitemCount === 0) {
                  item.classList.add('bm_chevron_00');
                  } else {
                     item.classList.remove('bm_chevron_00');
                     }
              // Anpassungen fuer 00, #1 = 0, #2 = 0
              if (menuCount === 0) {
                  bmVario.classList.add('Null_1');
                  }
              if (menuitemCount === 0) {
                  bmVario.classList.add('Null_2');
                  }
              // Reihenfolge Varianten Icons / Ziffern
              if (bm_order === 1) {
                  bmVario.classList.add('order_2');
                  }
              // Icons Auswahl
              if (bm_icon_folder === 0) {
                  bmVario.classList.add('icon_fallback_folder');
                  }
              if (bm_icon_link === 0) {
                  bmVario.classList.add('icon_fallback_link');
                  }
              // Farben Auswahl
              if (bm_colors === 1) {
                  bmVario.classList.add('bm_my_colors');
                  }
              // Verblassen bei Anzeige = 0
              if (bm_opacity_null === 1) {
                  bmVario.classList.add('opacity_Null');
                  }
              if (bm_opacity_null === 2) {
                   bmVario.classList.add('opacity_Null_B');
                   }
            }, 100);
        }
    }
})();

Alles anzeigen

Wo ist der Haken?

2002Andreas

Zitat von Boersenfeger

muss ich das Menü Lesezeichen 2x aufrufen

Mit deinen beiden Skripten kann ich das bestätigen.

Mit diesem Skript von BrokenHeart funktioniert es hier sofort:

JavaScript

(function() {

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

    function setFunction() {

        const css =`
            .countClass::after {
                content: attr(data-value) !important; 
				font-family:  Consolas, "Lucida Console", "Courier New", monospace !important;
				font-size: 11px !important;
				font-weight: 900 !important;
                color: rgb(100,200,255) !important;
			}
		`;

        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 bmbMenu = document.getElementById('bookmarks-menu-button');
        let bookMenu = document.getElementById('bookmarksMenu');
        let persToolBar = document.getElementById('PersonalToolbar');
        
        if(bmbMenu)
            bmbMenu.addEventListener('popupshown', onPopupShowing );  //--- 'popupshowing' -> 'popupshown' 
        if(bookMenu)
            bookMenu.addEventListener('popupshown', onPopupShowing ); //--- 'popupshowing' -> 'popupshown'
        if(persToolBar)
            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.hidden = true;
			  itemPopup.collapsed = true;
			  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++;
                  }
                }
              }
			  itemPopup.hidden = false;
			  itemPopup.collapsed = false;
              
              let label = item.childNodes[1]; //label.menu-iconic-text
  
              label.classList.add('countClass');
			  let strCountOut = String(menuCount).padStart(2, '\xa0') + "📁" + "\xa0" + String(menuitemCount).padStart(3, '\xa0') + "⭐"; //"\xa0"
			  label.setAttribute('data-value', strCountOut);
			  

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

Alles anzeigen

Horstmann

Zitat von Mira_Belle

Horstmann Und Du hast den gleichen Fehler wie ich begangen!
BrokenHeart hat mich darauf hingewiesen! Klammern, es fehlen Klammern.

Ahem, der Code funktionierte schon immer ohne diese Klammern {} - es sei denn, man fasst mehrere statements(?) unter einer condition(?) zusammen, das hatte BrokenHeart korrigiert in #531.
Deine Code Version von #542 sollte auch ohne die zusätzlichen Klammern funktionieren.
Oder?

Hattest du beide meiner Versionen aus #541 genau so getestet, und ohne eigene Modifikationen in dem Code Block?
Wir reden vermutlich etwas aneinander vorbei.

Und es ist nicht als extra Code für Mac gedacht, es ist extra Code für schmale Hauptpopups beim Lesezeichen-Menü-Button.

Zitat von Boersenfeger

Wo ist der Haken?

Wir arbeiten gerade daran.

Mira_Belle

Zitat von Horstmann

Hattest du beide meiner Versionen aus #541 genau so getestet, und ohne eigene Modifikationen in dem Code Block?
Wir reden vermutlich etwas aneinander vorbei.

Nein, glaube ich nicht!

Zitat von Horstmann

Hattest du beide meiner Versionen aus #541 genau so getestet,

Ja, und dann funktioniert die Anzeige der Counter im Popup vom LesezeichenButton, aber auch in der Lesezeichenleiste, nicht.
Mit Klammern schon.
Also noch einmal zusammen gefasst, beide Codes in Beitrag #541 funktionieren so bei mir nicht.
Setze ich die Klammern, funktionieren beide.

Sören Hentzschel

Zitat von Horstmann

der Code funktionierte schon immer ohne diese Klammern {} - es sei denn, man fasst mehrere statements(?) unter einer condition(?) zusammen

Die Klammern sind dann optional, wenn man alles in eine Zeile schreibt, ansonsten gilt das nur für die folgende Zeile (beziehungsweise bis das Statement abgeschlossen ist, falls man dieses über mehrere Zeile verteilt). Aber man sollte wirklich niemals auf die Klammern verzichten, weil das sehr leicht zur einer Fehlerquelle wird.

Zitat von Mira_Belle

Ja, und dann funktioniert die Anzeige der Counter im Popup vom LesezeichenButton eben nicht.
Mit Klammern schon.

Das kann überhaupt nicht sein. In den zwei Code-Blöcken von Beitrag #541 steht am Ende jeweils noch eine geschlossene Klammer, die nirgends dazu gehört. Vielleicht wurde diese mitkopiert, sodass es am Ende zu einem Fehler durch eine Klammer zu viel kam. Aber das Fehlen der Klammern für die if-Statements ist wie gesagt zwar nicht empfohlen, kann technisch allerdings keinen Unterschied machen.

PS: Klammern setzen heißt übrigens nicht, dass man die Einrückung vernachlässigen sollte. Das ist technisch genauso irrelevant, aber der Lesbarkeit schadet es, wenn jede Zeile an einer willkürlich gewählten Position startet.

Boersenfeger

2002Andreas Hatte die Datei nicht wieder zurück umbenannt...

Ja so klappt das, allerdings passt die Farbe nicht. Ich probiere da mal ein bisschen und melde mich wieder..

Edit:
Mit Schwarz sieht es besser aus, allerdings waren vorher die Zahlen untereinander und rechtsbündig. Die Schrift gefällt mir auch noch nicht.

Vorerst sieht es so aus:

Mira_Belle

Sören Hentzschel Du hast wie immer recht! Danke.
Bei dem vielen Testen mit Codefragmenten hatte ich eine Klammer, wo gar keine sein sollte.

Also Berichtigung Horstmann
Beide Varianten aus Beitrag #541 funktionieren.
Nur muss es bei mir anstatt persToolBar.addEventListener('popupshowing', onPopupShowing );
persToolBar.addEventListener('popupshown', onPopupShowing ); heißen,
sonst tritt eben genau DAS Problem auf.
Counter wird erst beim zweiten Mal öffnen des Menüs angezeigt.

grisu2099

Zitat von Boersenfeger

Wo ist der Haken?

ersetze mal in deinem oben geposteten Skript in den Zeilen 285-289 popupshowing durch popupshown und teste...

Boersenfeger

Jo, das ists dann.

Dann bleibe ich bei diesem Code/Script.

Vielen Dank!

Horstmann

Zitat von Sören Hentzschel

Das kann überhaupt nicht sein. In den zwei Code-Blöcken von Beitrag #541 steht am Ende jeweils noch eine geschlossene Klammer, die nirgends dazu gehört. Vielleicht wurde diese mitkopiert, sodass es am Ende zu einem Fehler durch eine Klammer zu viel kam. Aber das Fehlen der Klammern für die if-Statements ist wie gesagt zwar nicht empfohlen, kann technisch allerdings keinen Unterschied machen.

Danke!
Schande auf mein Haupt; ich korrigiere das mal sofort...

Zitat von Mira_Belle

Beide Varianten aus Beitrag #541 funktionieren.
Nur muss es bei mir anstatt persToolBar.addEventListener('popupshowing', onPopupShowing );
persToolBar.addEventListener('popupshown', onPopupShowing ); heißen,
sonst tritt eben genau DAS Problem auf.
Counter wird erst beim zweiten Mal öffnen des Menüs angezeigt.

Danke für's erneute Testen, sorry für den Fehler mit der unteren Klammer.

Aber jetzt gibt es gibt es das Problem plötzlich auch in der Lesezeichenleiste?
Direkt in der Lesezeichenleiste, nicht in der Menüleiste -> Lesezeichenleiste, oder deren Untermenüs?

Mira_Belle

Zitat von Horstmann

Aber jetzt gibt es gibt es das Problem plötzlich auch in der Lesezeichenleiste?

Nicht Plötzlich. Hatte ich aber auch schon in #535 so.
Und in #542 sogar erwähnt.

Hier der derzeitige Code, wie er bei mir, in allen Lesezeichenpopups, funktioniert.

JavaScript

    let bmbMenu = document.getElementById('bookmarks-menu-button');   //--- LesezeichenmenüButton
    let bookMenu = document.getElementById('bookmarksMenu');   //--- Lesezeichenmenü
    let persToolBar = document.getElementById('PersonalToolbar');   //--- Lesezeichenleiste
    let bmbMenuPop = document.getElementById('BMB_bookmarksPopup');   //--- extra für Mac erforderlich

    if(bmbMenu) {
      bmbMenu.addEventListener('popupshown', onPopupShowing );  //--- 'popupshowing' -> 'popupshown'
      }
    if(bmbMenuPop) {
      bmbMenuPop.addEventListener('popupshowing', onPopupShowing );   //--- extra für Mac erforderlich
      }
    if(bookMenu) {  
      bookMenu.addEventListener('popupshown', onPopupShowing ); //--- 'popupshowing' -> 'popupshown'
      }
    if(persToolBar) {
      persToolBar.addEventListener('popupshown', onPopupShowing ); //--- 'popupshowing' -> 'popupshown'
    }

Alles anzeigen

BrokenHeart

Zitat von Horstmann

Die Breite des Haupt-Popups ändert sich jeweils beim ersten Öffnen nach Neustart oder in einem neuen Fenster (falls das Popup noch nicht die erlaubte max-width des Fensters besitzt).
Edit: hat dieses Popup schon die Maximalbreite bevor die Zählereinträge beim Öffnen dazugefügt werden, etwa durch einen sehr langen Link oder Foldernamen (Anhang), gibt es hier kein Problem wie in #2 (s. unten), weil die Zählereinträge die Breite eh nicht mehr ändern können.

Ja, so ist es. Das Problem mit der sich verändernden Breite des Popups resultiert aus dem Umstand, dass die hinzugefügten Zähler das Popup bei Verwendung von popupshown verbreitern. Die Breite des Popups ist aber noch ohne(!) die Zähler vom Firefox berechnet und dargestellt worden. Erst wenn das Popup erscheint, werden die Zähler sichtbar nacheinander hinzugefügt und das Popup wird quasi "live" wieder in der Breite angepasst. ~~Wer es optisch ganz sauber haben möchte, benutzt noch zusätzlich~~ ~~popupshowing~~ ~~für den LesezeichenMenu-Button, wie von Horstmann vorgeschlagen.~~ (<- siehe #557)

Die Veränderung der Ausrichtung des Popups bei meinem ersten Test (zuerst links, dann rechts) hängt genau damit zusammen: weil das Popup durch die hinzugefügten Zähler nicht mehr links darstellbar war, wurde halt "on the fly" nach rechts geswitcht.

Zitat von Horstmann

Aber jetzt gibt es gibt es das Problem plötzlich auch in der Lesezeichenleiste?

Was meinst du genau damit? Werden gar keine Zähler mehr beim ersten Aufruf angezeigt oder passt sich die Breite nicht richtig an?

Zitat von Mitleser

BrokenHeart, ich habe dieses Script angepasst und sehe keine Probleme (Linux), auch nicht mit #bookmarks-menu-button oder #bookmarksMenuPopup (oder erkenne ich das Problem nicht?).

Du hast ja auch alles so in deinem Skript übernommen, wie es vorgeschlagen wurde. Da sollte es dann auch keine Darstellungsprobleme geben...

Horstmann

Zitat von BrokenHeart

Zitat von Horstmann

Aber jetzt gibt es gibt es das Problem plötzlich auch in der Lesezeichenleiste?

Was meinst du genau damit? Werden gar keine Zähler mehr beim ersten Aufruf angezeigt oder passt sich die Breite nicht richtig an?

Nah, Mira hat das erwähnt, ich habe keine Probleme damit.

Ich komm hier so langsam durcheinander, und nicht jede Respons ist nachvollziebar ; was funktioniert für dich von den aktuellen Vorschlägen?
Du hast ja Windows, und wie ich vereinzelte Darstellungsprobleme erfahren, hast du einen Vorschlag für generelle Nutzung?

Ich tendiere momentan zu:

JavaScript

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

        let bmbMenuPop = document.getElementById('BMB_bookmarksPopup');      // neu
        
        if(bmbMenu)
           bmbMenu.addEventListener('popupshown', onPopupShowing );    // popupshowing => popupshown

        if(bmbMenuPop)
           bmbMenuPop.addEventListener('popupshowing', onPopupShowing );   //  neu

        if(bookMenu)
           bookMenu.addEventListener('popupshown', onPopupShowing );    // popupshowing => popupshown

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

Alles anzeigen

BrokenHeart

Zitat von BrokenHeart

Wer es optisch ganz sauber haben möchte, benutzt noch zusätzlich popupshowing für den LesezeichenMenu-Button, wie von Horstmann vorgeschlagen.

Habe es eben noch mal getestet und festgestellt, dass das zusätzliche popupshowing doch nicht den gewünschten Effekt hat. Damit wird eine nachträgliche Verbreiterung nicht verhindert, da hatte ich mich wohl beim ersten Test vertan oder irgendwas anderes falsch gemacht. Da müsste also noch eine andere Lösung her, falls es überhaupt möglich ist.

Ehrlich gesagt stört mich das Verhalten der "Live"-Verbreiterung des Popups auch nicht sonderlich, aber in Kombination mit der Animation könnte das natürlich optisch störend sein.

Zitat von Mira_Belle

Hier der derzeitige Code, wie er bei mir, in allen Lesezeichenpopups, funktioniert.

"Funktionieren" tut das auch, aber wie beschrieben, mit einem optischen "Makel".

Zitat von Horstmann

Du hast ja Windows, und wie ich vereinzelte Darstellungsprobleme erfahren, hast du einen Vorschlag für generelle Nutzung?

Man muss zwei Probleme/Baustellen unterscheiden:

1.) Die Zähler werden beim ersten Klick im Lesezeichenmenü nicht angezeigt:

-> Dieses Problem ist behoben, wenn popupshown eingefügt wird, also so wie ich es in #526 beschrieben hatte oder auch Mira_Belle in #554. Für die Lesezeichenleiste ist es eigentlich nicht notwendig, kann also popupshowing bleiben, schadet aber auch nicht wenn man hier auch popupshown verwendet.

2.) Die "Live-Verbreiterung" des Popups:

-> Dafür gibt es (von meiner Seite) noch keine Lösung.

Horstmann

Zitat von BrokenHeart

Habe es eben noch mal getestet und festgestellt, dass das zusätzliche popupshowing doch nicht den gewünschten Effekt hat. Damit wird eine nachträgliche Verbreiterung nicht verhindert, da hatte ich mich wohl beim ersten Test vertan oder irgendwas anderes falsch gemacht. Da müsste also noch eine andere Lösung her, falls es überhaupt möglich ist.
Ehrlich gesagt stört mich das Verhalten der "Live"-Verbreiterung des Popups auch nicht sonderlich, aber in Kombination mit der Animation könnte das natürlich optisch störend sein.

Die nachträgliche Verbreiterung soll auch nicht verhindert werden, das ist ein ganz normales Firefox Feature auch ohne das Script. Nur Animations Probleme wie von dir beschrieben oder wie von mir beschrieben sind scheinbar das letzte Problem.
In meinem Fall passiert das ohne anderes relevates CSS in einem Testprofil.
Und ist für mich unerträglich.

Könnte man mit CSS evtl. gegensteuern, habe aber den Verdacht dass da uU nicht animierbare Faktoren im Spiel sind.

Bei der Lesezeichenleiste habe ich das Gleiche beobachtet, popupshowing oder popupshown scheint wumpe zu sein.

Da deine, meine und Mira's Varianten keinen zusätzlichen Ärger verursachen, aber das Grundproblem mit der Menüleiste lösen zu scheinen, werd ich das wohl so in der Art einpflegen.
Wenigstens haben wir keinen doppelten addEventListener Dingens (Fachbegriff) mehr für das gleiche Element (#bookmarks-menu-button), das war ja eine deiner Sorgen.

Vor allem aber danke nochmal dafür die Ursache gefunden zu haben!

Mira_Belle

Ich verstehe das mit der "Live-Verbreiterung" nicht!

Mein komplettes Skript in Kombination mit dem Autopopup-Skript.

JavaScript

// BookmarkCount.uc.js

/* ******************************************************************************************************************* */
/* Author BrokenHeart               => https://www.camp-firefox.de/forum/thema/136572/?postID=1229536#post1229536 **** */
/* Correction from BrokenHeart      => https://www.camp-firefox.de/forum/thema/136572/?postID=1229696#post1229696 **** */
/* Extension from Sören Henschel    => https://www.camp-firefox.de/forum/thema/136572/?postID=1229555#post1229555 **** */
/* Customized by Mira inspired by grisu2099     ********************************************************************** */
/* Source                           => https://www.camp-firefox.de/forum/thema/136572/?postID=1229875#post1229883 **** */
/* Customized by Mira inspired by Horstmann     ********************************************************************** */
/* Source                           => https://www.camp-firefox.de/forum/thema/136572/?postID=1229993#post1229993 **** */
/* Adjustments for 139 and customized by Mira inspired by Horstmann     ********************************************** */
/* Source                           => https://www.camp-firefox.de/forum/thema/136572/?postID=1270254#post1270254 **** */
/* Bugfixes for 'popupshowing' to 'popupshown'inspired by BrokenHeart & Horstmann ************************************ */
/* Source                           => https://www.camp-firefox.de/forum/thema/136572/?postID=1274881#post1274881 **** *
/* ******************************************************************************************************************* */
/* Version 1.04a                    => https://www.camp-firefox.de/forum/thema/136572/?postID=1274881#post1274881 **** */
/* ******************************************************************************************************************* */
/* Custom Counter in the Bookmarks for folders and links ************************************************************* */
/* ******************************************************************************************************************* */

(function() {

  if (!window.gBrowser)
      return;

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

  let ProfilePath = PathUtils.toFileURI(PathUtils.join(PathUtils.profileDir, 'chrome', 'icons'));

  // ■■ START UserConfiguration ■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■
    //Custom icons in profile/chrome/icons folder
  let icon1 = "folder-fat.svg";                     //  Custom  Folder Icon
  let icon2 = "bookmark-hollow.svg";                //  Custom  Link Icon              
  let icon3 = "arrow-right.svg";                    //  Custom  Arrow Icon

  // Custom settings
  let bm_font_size = 12; 
  let cs_font_weight = 700;
  let bm_icon_size = 16;

  // NEU: Schriftfarbe getrennt definieren
  //let cs_font_color_1 = "#FFA500";                 // Farbe für Folder-Zähler (links)
  //let cs_font_color_2 = "#00FFFF";                 // Farbe für Link-Zähler (rechts)
  let cs_font_color_1 = "#FFD700";                 // Farbe für Folder-Zähler (links)
  let cs_font_color_2 = "#FFD700";                 // Farbe für Link-Zähler (rechts)

  // NEU: Symbolfarber getrennt definieren 
  //let cs_icon_color_1 = "#5fe575";                 // Farbe für "Ordner", 1.Symbol
  //let cs_icon_color_2 = "#fbf328";                 // Farbe für "Link", 2.Symbol
  //let cs_icon_color_3 = "#bbf700";                 // Farbe für den Pfeil
  let cs_icon_color_1 = "#C0C0C0";                 // Farbe für "Ordner", 1.Symbol
  let cs_icon_color_2 = "#C0C0C0";                 // Farbe für "Link", 2.Symbol
  let cs_icon_color_3 = "#C0C0C0";                 // Farbe für den Pfeil


  // Calculated settings
  let cs_font_size = `${bm_font_size}px`;
  let cs_icon_size = `${bm_icon_size}px`;
  let cs_width_one = `calc(${bm_icon_size}px + 1.4em)`;
  let cs_width_uno = `calc(${bm_icon_size}px)`;
  let cs_width_two = `calc(${bm_icon_size}px + 1.4em)`;
  let cs_width_due = `calc(${bm_icon_size}px)`;

  //let useStyle1 = true; // <- HIER Weiche setzen: true = Variante 1, false = Variante 2
  let useStyle = 1; // <- HIER Weiche setzen:  0 = Symbole hinter den Zählern, 1 = Symbole vor den Zählern
  // ■■ END UserConfiguration ■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■

  // Symbole vor den Zählern
  let css_one = `

    /* Counter #1 Folder */
    #bmContent::before {
      content: attr(data-value1) !important;
      width: ${cs_width_one} !important;
      align-items: center !important;
      display: flex !important;
      justify-content: end !important;
      /*margin-inline: 16px 4px;         */               /* min. Abstand links für enge Popups, rechts zum 2ten Counter */

      margin-left: 16px !important;                       /* Abstand links minimum für schmale Popus  CHANGE  */
      margin-right: -1px !important;                      /* Abstand Icon "Ordner" bis zum "Zähler" */ 

      background-image: url("${ProfilePath}/${icon1}")!important;
      background-position: center right calc(${cs_width_one} - ${cs_icon_size}) !important;
      background-repeat: no-repeat !important;
      background-size: ${cs_icon_size} ${cs_icon_size} !important;     
      color: ${cs_font_color_1} !important;               /* Farbe der 1. Zahl */
      fill: ${cs_icon_color_1} !important;                /* Farbe des 1. Symbols */
    }

    /* Counter #2 Links */
    #bmContent::after {
      content: attr(data-value2) !important;
      width: ${cs_width_two} !important;
      align-items: center !important;
      display: flex !important;
      justify-content: end !important;
      /*margin-inline: -2px;*/

      margin-left: 4px !important;                          /* Abstand "Zähler 1" zum Icon2 */
      margin-right: -1px !important;                        /* Abstand Icon "Links" bis zum "Zähler" */


      background-image: url("${ProfilePath}/${icon2}") !important;
      background-position: center right calc(${cs_width_two} - ${cs_icon_size}) !important;
      background-repeat: no-repeat !important;
      background-size: ${cs_icon_size} ${cs_icon_size} !important;            
      color: ${cs_font_color_2} !important;                 /* Farbe der 2. Zahl */
      fill: ${cs_icon_color_2} !important;                  /* Farbe des 2. Symbols */
    }
  `;

  // Symbole hinter den Zählern
  let css_two = `

    /* Counter #1 Folder */
    #bmContent::before {
      content: attr(data-value1) !important;
      width: ${cs_width_one} !important;
      align-items: center !important;
      display: flex !important;
      justify-content: end !important;
      background-image: url("${ProfilePath}/${icon1}")!important;
      background-position: center right calc(${cs_width_uno} - ${cs_icon_size}) !important;
      background-repeat: no-repeat !important;
      background-size: ${cs_icon_size} ${cs_icon_size} !important;     
      color: ${cs_font_color_1} !important;                 /* Farbe der 1. Zahl */
      fill: ${cs_icon_color_1} !important;                  /* Farbe des 1. Symbols */
      padding: 2px 20px 0px 10px !important;                /* top | right | bottom | left */
      /*margin-inline: 0px -10px; */

      margin-left: 16px !important;                         /* Abstand links minimum für schmale Popus  CHANGE  */
      margin-right: 12px !important;                        /* Abstand Icon "Ordner" bis zum "Zähler" */ 

    }

  	#bmContent::after {
			content: attr(data-value2) !important;
      width: ${cs_width_two} !important;
      align-items: center !important;
      display: flex !important;
      justify-content: end !important;
      /*justify-content: flex-end; */
      background-image: url("${ProfilePath}/${icon2}")!important;
      background-position: center right calc(${cs_width_due} - ${cs_icon_size}) !important;
      background-repeat: no-repeat !important;
      background-size: ${cs_icon_size} ${cs_icon_size} !important;            
      color: ${cs_font_color_2} !important;                 /* Farbe der 2. Zahl */
      fill: ${cs_icon_color_2} !important;                  /* Farbe des 2. Symbols */
			padding: 2px 22px 0 0 !important;                     /* top | right | bottom | left */

      margin-left: -14px !important;                        /* Abstand "Zähler 1" zum Icon2 */
      margin-right: -1px !important;                        /* Abstand Icon "Links" bis zum "Zähler" */

		}
  `;

  function setFunction() {

    const css =`

      /* Counter rechtsbuendig */
      #bmContent {
        display: flex !important;
        margin-inline: auto 0 !important;
        font-family:  Consolas, "Lucida Console", "Courier New", monospace !important;
        font-size: ${cs_font_size} !important;
        font-weight: ${cs_font_weight} !important;
        height: ${cs_icon_size} !important;
      }

      /* Dynamisch gewählter before-Block */
      ${useStyle ? css_one : css_two}

      /* Pfeil */
      menupopup > menu::after {
        content: "" !important;
        background-image: url("${ProfilePath}/${icon3}")!important;
        height: ${cs_icon_size} !important;
        height: ${cs_icon_size} !important;
        fill: ${cs_icon_color_3} !important;
      }

    `;

    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 bmbMenu = document.getElementById('bookmarks-menu-button');   //--- LesezeichenmenüButton
    let bookMenu = document.getElementById('bookmarksMenu');   //--- Lesezeichenmenü
    let persToolBar = document.getElementById('PersonalToolbar');   //--- Lesezeichenleiste
    let bmbMenuPop = document.getElementById('BMB_bookmarksPopup');   //--- extra für Mac erforderlich

    if(bmbMenu) {
      bmbMenu.addEventListener('popupshown', onPopupShowing );  //--- 'popupshowing' -> 'popupshown'
      }
    if(bmbMenuPop) {
      bmbMenuPop.addEventListener('popupshowing', onPopupShowing );   //--- extra für Mac erforderlich
      }
    if(bookMenu) {  
      bookMenu.addEventListener('popupshown', onPopupShowing ); //--- 'popupshowing' -> 'popupshown'
      }
    if(persToolBar) {
      persToolBar.addEventListener('popupshown', onPopupShowing ); //--- 'popupshowing' -> 'popupshown'
    }

/*
    if(bmbMenu)
      bmbMenu.addEventListener('popupshowing', onPopupShowing );
    if(bookMenu)
      bookMenu.addEventListener('popupshowing', onPopupShowing );
    if(persToolBar)
      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.hidden = true;
			    itemPopup.collapsed = true;
          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++;
                }
            }
          }
          itemPopup.hidden = false;
          itemPopup.collapsed = false;

          // Neues Element für Zaehler
          let bmCounta = item.childNodes[1];
            if (!bmCounta) return;                            // Falls undefiniert
          bmCounta.innerHTML = ""
          let bmContent = document.createElement("bmContent");
          bmContent.id = "bmContent";
          bmCounta.appendChild(bmContent);

          //let strCountOut1 = " " + menuCount + " ";         //  Has no brackets
          //let strCountOut1 = "(" + menuCount + ")";         //  Has round brackets
          //let strCountOut1 = "[" + menuCount + "]";         //  Has square brackets
          let strCountOut1 = String(menuCount).padEnd(2, " ");
          bmContent.setAttribute('data-value1', strCountOut1);

          //let strCountOut2 = " " + menuitemCount + " ";      //  Has no brackets
          //let strCountOut2 = "(" + menuitemCount + ")";      //  Has round brackets
          //let strCountOut2 = "[" + menuitemCount + "]";      //  Has square brackets
    //      let strCountOut2 = String(menuitemCount).padEnd(2, " ");
    //      bmContent.setAttribute('data-value2', strCountOut2);

          let strCountOut2 = (useStyle === 0) 
            ? String(menuitemCount).padStart(4, "/  ")
            : String(menuitemCount).padEnd(2, " ");
          bmContent.setAttribute('data-value2', strCountOut2);
          
        }, 100);
    }
  }
})();

Alles anzeigen

Zweites Skript AutoPopup.uc.js gelöscht, weil gemotzt wurde. Ist ok, ist aber eigentlich mein Thread.

Nimm es bitte nicht so Bierernst, Du hast ja recht.

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

Beitrag von Boersenfeger (5. Juli 2025 um 14:46)

Beitrag von Mira_Belle (5. Juli 2025 um 20:50)