Beiträge von omar1979

Zitat von Horstmann

Das ist für den Lesezeichen Menü Button und sein Popupmenü, den man in Symbolleiste anpassen in die Werkzeugleiste einfügen kann.

Danke für die Erklärung.

Dieser Script im Beitrag: #116 funkt mit diesem Button nur, wenn ich den Button im Menu-Leiste ganz oben oder auf dem Lesezeichen-Symbolleiste habe. Sonst funkts nirgends.

So stehe ich wieder am Anfang.

Kann man das Script nicht so ändern, dass es auch dann funkt, wenn man zum Beispiel die Lesezeichen-Ordner oder den Button auch auf eine Zusatzleiste verschiebt?

Dann könnte ich diesen Script auch für immer benützen.

Also ich hab jetzt das Script aus #108 genommen so wie es dort ist. Zuerst mit Restart-button dann mit fehlermodus neugestartet. Beide male das gleiche. das script funkt nicht.

Dann hab ich im gleichen script wie 2002andreas beschrieben hat die 3 hinteren leerzeichen gelöscht und wieder mit restartbutton und dann mit fehlermodus gestartet. wieder beide male das gleiche. das script funkt nicht.

aber ohne dies 2 zeilen mit dem bmbmenu funkt das script im lesezeichen-symbolleiste und im lesezeichen-menü.

Was bewirken die 2 Zeilen mit dem bmbMenu?

Kann mir wer das erklären?

Zitat von BrokenHeart

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:

Dieser Script funkt jetzt. Ordner im Lesezeichen-Symbolleiste.

Ordner im Lesezeichen-Menü.

Zitat von omar1979

Was bewirken die 2 Zeilen mit dem bmbMenu?

Kann mir wer das erklären?

Kann mir das jemand erklären?

also ohne diese 2 Zeilen mit dem bmbMenu funkt das Script bei mir im Lesezeichen-symbolleiste und im Lesezeichen-Menu oben.

Also ich hab jetzt so wie 2002Andreas das Leerzeichen vor diesen 3 hinteren Klammern gelöscht

let bmbMenu = document.getElementById('bookmarks-menu-button');

let bookMenu = document.getElementById('bookmarksMenu');

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

bmbMenu.addEventListener('popupshowing', onPopupShowing );

bookMenu.addEventListener('popupshowing', onPopupShowing );

persToolBar.addEventListener('popupshowing', onPopupShowing );

neu gestartet. funkt nicht.

dann hab ich die Leerzeichen wieder gemacht, diesmal mit fehlermodus neugestartet funkt auch nicht.

keine ahnung warum es nicht funkt.

meine Frage noch, was machen diese 2 neue Zeilen?

das bmbMenu?

so zur info: ich benütze das normale editor von win11 nicht notepad++

Zitat von 2002Andreas

Hier auch nicht

Danke für bestätigen. Jetzt weiss ich, dass es nicht an mir liegt.

Zitat von BrokenHeart

Gleich mal hier als neue Skript-Version eingebaut: (Dank an Horstmann )

Das ganze Script geladen und es funkt nicht.

Zitat von Horstmann

Könnte was u.a. für omar1979 sein.

Scheint im ersten Versuch zu klappen, 2 Zusätze für Zeilen 23 - 27 in deinem Code:

Hab das so gemacht,

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

Jetzt funktioniert das Script wieder nicht.

Zitat von FuchsFan

Und zusätzlich habe ich nach Zeile 15 das noch hinzugefügt, damit sieht es dann so aus.

Dankeschön.. ich hab das

padding-right: 15px;

übernommen..

Weiss jemand gerade wie ich vor den Zahlen noch ein Text hinzufügen kann, damit es inetwa so aussieht?

Blabla-Ordner (Links: 5 / Ordner: 3) <<<-inetwa so. Also vor den Zahlen Links oder Ordner hinzufügen?

Obige Frage selbst erledigt.

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

Zitat von 2002Andreas

sorry dafür.

Nichts zu entschuldigen. Ich schreibe ja immer wirres Zeug.

Zitat von 2002Andreas

Evtl. das

Nein, nicht das. Das von BrokenHeart

Das hier:

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

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

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

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.

Zitat von BrokenHeart

Nachdem Firefox gestartet wurde [STRG+SHIFT+J] drücken.

Zitat von BrokenHeart

Erhöhe in meinem Skript mal die Werte in Zeile 52 und Zeile 59. Zum Testen ruhig mal auf '1000'!

habe das 100 und 50 beide auf 1000 aber funkt auch nicht

Hab jetzt das userChromeShadow.css von chrome-ordner entfernt, nützt aber auch nichts.

Und ich markiere das Script immer blau dann mit rechtsklick kopieren und dann im editor einfügen.

Zitat von BrokenHeart

Steht bei euch eine Fehlermeldung in der Konsole bzgl. dieses Skripts?

Keine Ahnung wo ich da schauen muss. Sorry.

Zitat von BrokenHeart

Habt ihr mal alle sonstigen Skripte entfernt? Vielleicht kommt ja auch ein spezielles AddOn als Störfaktor in Frage?

Läuft bei euch dieses (Original-)Skript? Wenn ja, dann bin ich wirklich überfragt und ratlos...

Nein habe kein Scripte entfernt und keine Ahnung ob da ein AddOn dazwischen funkt.

Ja das Original-Script das du oben gepostet hast funkt.

Also ich hab es auch auf Win11 getestet und bei mir funkt es auch nicht. Habe den FF mit dem Rechtsklick auf das Restartbutton neugestartet.

Zitat von 2002Andreas

Für diese Funktion brauchst du ein weiteres Skript:

Dankeschön.