Lesezeichensymbolleiste leer

Horstmann

Zitat von nierewa

Hallo Leute.
Danke für die vielen Rückmeldungen.

Zitat

Teste doch bitte mal dieses geänderte Skript:(oder auch ein anderer user)

Funktioniert

Zitat

Und auch dieses Script funktioniert hier.

UND sogar das Tastenkürzel!

Das Script funktioniert auch.
Ich habe nur die XXX.style.height auf 28px erhöht.
Bei 24px wurden die Ordner in der Lesezeichenleiste etwas zu klein dargestellt.

Alles anzeigen

Nur eine Frage noch abschliessend, welches Script hast Du letztendlich in Benutzung ?

Aus Deinen Antworten ist es leider wieder mal nicht herauszulesen - z.B. das zweite Zitat hier bezieht sich auf das Script von Visoer - mit einer Leistenhöhe von gewaltigen 36px .

Deine Antwort darunter bezieht sich aber auf irgendeines der anderen Scripts, in dem anscheinend eine Leistenhöhe von weniger als 28px eingetragen ist . Was nicht ganz zusammenpasst .

Mira_Belle

Ich vermute, es ist "mein" Script gemeint.

Denn ich habe den Eintrag persToolbar.style.height == '24px'

auf den er/sie sich bezieht.

Kann mich aber auch irren.

Es wäre dann dieses hier:

JavaScript

// Button_PersonalToolbar.uc.js
// Source file https://www.camp-firefox.de/forum/thema/134920-schaltfl%C3%A4che-um-lesezeichen-leiste-ein-auszublenden/?postID=1213680#post1213680
// Source file https://www.camp-firefox.de/forum/thema/134920-schaltfl%C3%A4che-um-lesezeichen-leiste-ein-auszublenden/?postID=1215809#post1215809
// PersonalToolbar_ausblenden
// basierend auf den Ideen von BrokenHeart und Verbesserungen von Horstmann

// let startVisibility = 'collapse' //'visible' oder 'collapse' Der Startzustand wird definiert:"visible" startet mit sichtbarer Leiste 

let startHeight = '0px'; //'0px' oder '24px' Hoehe Leiste
let autoCloseTime = 5000; // 0 = kein Auto-Close / >0 Zeit in Millisekunden /5000 => 5sec 
let intervalID = 0;
let currentProfileDirectory = Services.dirsvc.get("ProfD", Ci.nsIFile).path.replace(/\\/g, "/"); // Die Pfadangabe zu dem Profilordner
let buttonicon = "bookmark-hollow.svg" // Name & Dateiendung des anzuzeigenden Symbols
let persToolbar = document.getElementById('PersonalToolbar');

(function ptbut() {              
    if (location.href !== 'chrome://browser/content/browser.xul' && location.href !== 'chrome://browser/content/browser.xhtml') return;
    try {
        CustomizableUI.createWidget({
            id: 'PersonalToolbar-button',
            type: 'custom',
            defaultArea: CustomizableUI.AREA_NAVBAR,
                onBuild: function(aDocument) {
                    let toolbaritem = aDocument.createXULElement('toolbarbutton');                  
                    let props = {
                        id: 'PersonalToolbar-button',
                        class: 'toolbarbutton-1 chromeclass-toolbar-additional',
                        removable: true,
                        label: 'Toggle',
                        tooltiptext: 'Lesezeichenleiste ein-/ausblenden',
                        style: 'list-style-image: url("' + ("file:" + currentProfileDirectory + "/chrome/icons/" + buttonicon) +'");',
                    };                   
                    for(var p in props)
                        toolbaritem.setAttribute(p, props[p]);
                    return toolbaritem;                 
                }
        });
        CustomizableUI.registerToolbarNode(tb);
    } catch(e) { };

    document.getElementById('PersonalToolbar').style.minHeight = '0';

    if( startHeight === '24px' || startHeight === '0px' ) {
        document.getElementById('PersonalToolbar').style.height = startHeight;
   }
    else {
        document.getElementById('PersonalToolbar').style.height = '24px';
    }

    document.getElementById('PersonalToolbar-button').addEventListener( "click", onClick );
  
    function onClick(aEvent) {   
        if (persToolbar.style.height == '') {
            persToolbar.style.height = '0px';    }
        else if (persToolbar.style.height == '24px') {
            persToolbar.style.height = '0px'; }
        else if(persToolbar.style.height == '0px') {
            persToolbar.style.height = '24px';
            if(!intervalID && autoCloseTime) {
                intervalID =  aEvent.target.ownerGlobal.setInterval( autoClose, autoCloseTime ); 
            }
        }
      }
      
    function autoClose() {
        if( intervalID ) {
            if( persToolbar.style.height == '24px') {
                persToolbar.style.height = '0px';
                clearInterval(intervalID); 
                intervalID=0;
            }
        }
    }
})();

Alles anzeigen

2002Andreas

Zitat von Mira_Belle

Ich vermute, es ist "mein" Script gemeint.

Und ich denke eher nicht, denn da fehlt die Funktion mit der Tastenkombination wie im Beitrag Nr. 1 im Skript.

Mira_Belle

Zitat von 2002Andreas

Zitat von Mira_Belle

Ich vermute, es ist "mein" Script gemeint.

Und ich denke eher nicht, denn da fehlt die Funktion mit der Tastenkombination wie im Beitrag Nr. 1 im Skript.

Das stimmt, aber ich glaube doch, dass nierewa imstande ist, zu kombinieren.

visoer hat ja an vielen Stellen in diesem Thread und in dem Anderen aufgezeigt, wie der Code für die

Tastenkombination auszusehen hat und wo er hingehört.

Z.B:

Beitrag

RE: Lesezeichensymbolleiste leer

@Mira_Belle

[…]

Bei deinem Script geht das bei mir so:

(Quelltext, 16 Zeilen)

visoer

22. Dezember 2022 um 15:23

Aber auch hier gilt wieder, Aufklären musste es nierewa

2002Andreas

Zitat von Mira_Belle

zu kombinieren.

Muss er ja nicht, es gibt ja 2 Skripte hier mit der Tastenkombination.

Mira_Belle

Zitat von 2002Andreas

Zitat von Mira_Belle

zu kombinieren.

Muss er ja nicht, es gibt ja 2 Skripte hier mit der Tastenkombination.

Das stimmt.

Habe jetzt min. viermal den Thread durchgelesen, aber Horstmann hat recht!

Es wird einem einfach nicht klar, welches Skript nierewa letzten Endes benutzt.

Für den Fall, dass nierewa das Rätsel nicht mehr auflöst,

biete ich hier das von mir überarbeitete und funktionierende JavaScript an.

Es wurde die Animation integriert, sowie eine Tastenkombination!

Bei dem Aktivieren der Leiste mittels Tastenkombination ist zu beachten, dass das automatische Schließen nicht funktioniert

Jedoch beim Benutzen des Buttons.

JavaScript

// Button_PersonalToolbar.uc.js
// Source file https://www.camp-firefox.de/forum/thema/134920-schaltfl%C3%A4che-um-lesezeichen-leiste-ein-auszublenden/?postID=1213680#post1213680
// Source file https://www.camp-firefox.de/forum/thema/134920-schaltfl%C3%A4che-um-lesezeichen-leiste-ein-auszublenden/?postID=1215809#post1215809
// Source file https://www.camp-firefox.de/forum/thema/135748-lesezeichensymbolleiste-leer/?postID=1216410#post1216410
// PersonalToolbar_ausblenden
// basierend auf den Ideen von BrokenHeart und visoer, Verbesserungen von Horstmann und Anpassungen von Mira_Belle

let startHeight = '0px';                // '24px' oder '0px' Der Startzustand wird definiert: '24px' startet mit sichtbarer Leiste
let autoCloseTime = 5000;               // 0 = kein Auto-Close / >0 Zeit in Millisekunden /5000 => 5sec 
let collapse = '0px';                   // Leiste nicht sichtbar
let visible = '24px';                   // Leiste sichtbar
let buttonicon = "bookmark-hollow.svg"  // Name & Dateiendung des anzuzeigenden Symbols
let intervalID = 0;
let currentProfileDirectory = Services.dirsvc.get("ProfD", Ci.nsIFile).path.replace(/\\/g, "/"); // Pfadangabe zum Profilordner
let persToolbar = document.getElementById('PersonalToolbar');

(function ptbut() {              
    if (location.href !== 'chrome://browser/content/browser.xul' && location.href !== 'chrome://browser/content/browser.xhtml') return;
    try {
        CustomizableUI.createWidget({
            id: 'PersonalToolbar-button',
            type: 'custom',
            defaultArea: CustomizableUI.AREA_NAVBAR,
                onBuild: function(aDocument) {
                    let toolbaritem = aDocument.createXULElement('toolbarbutton');                  
                    let props = {
                        id: 'PersonalToolbar-button',
                        class: 'toolbarbutton-1 chromeclass-toolbar-additional',
                        removable: true,
                        label: 'Toggle',
                        tooltiptext: 'Lesezeichenleiste ein-/ausblenden',
                        style: 'list-style-image: url("' + ("file:" + currentProfileDirectory + "/chrome/icons/" + buttonicon) +'");',
                    };                   
                    for(var p in props)
                        toolbaritem.setAttribute(p, props[p]);
                    return toolbaritem;                 
                }
        });
        CustomizableUI.registerToolbarNode(tb);
    } catch(e) { };

//----------BEGINN----------------------
// Funktion zum Abfragen der Tastatur ("belauscht" Eingaben)
// Hinweis: Tasten, die anderweitig bereits "abgefangen" werden, ergeben keinen Event
    document.addEventListener('keydown', (event) => {
        let keyName = event.key;
// Funktion wird ausgeführt, wenn "AltGr & q gedrückt werden
        if (event.altKey && keyName == 'q') {
            onClick();
        }
    }, false
    );
//----------ENDE------------------------ 
// Anmerkungen und beispiele am Ende des Scriptes

document.getElementById('PersonalToolbar').style.minHeight = '0';
document.getElementById('PersonalToolbar').style.transition = 'all 0.3s steps(6, end)';

    if( startHeight === visible || startHeight === collapse ) {
        document.getElementById('PersonalToolbar').style.height = startHeight;
   }
    else {
        document.getElementById('PersonalToolbar').style.height = visible;
    }

    document.getElementById('PersonalToolbar-button').addEventListener( "click", onClick );
  
    function onClick(aEvent) {   
        if (persToolbar.style.height == '') {
            persToolbar.style.height = collapse;
            }
        else if (persToolbar.style.height == visible) {
            persToolbar.style.height = collapse;
            }
        else if(persToolbar.style.height == collapse) {
            persToolbar.style.height = visible;
            if(!intervalID && autoCloseTime) {
                intervalID =  aEvent.target.ownerGlobal.setInterval( autoClose, autoCloseTime ); 
            }
        }
      }
      
    function autoClose() {
        if( intervalID ) {
            if( persToolbar.style.height == visible) {
                persToolbar.style.height = collapse;
                clearInterval(intervalID); 
                intervalID=0;
            }
        }
    }
})();

// Funktion wird ausgeführt, wenn STRG + Alt + l gedrückt wird
//  if (event.ctrlKey && event.altKey && keyName == 'l') {       

// Funktion wird ausgeführt, wenn STRG + w gedrückt werden
//if (event.ctrlKey && keyName == 'w') {

Alles anzeigen

Horstmann

// Button_PersonalToolbar.uc.js

Zitat von Mira_Belle

Für den Fall, dass nierewa das Rätsel nicht mehr auflöst,
biete ich hier das von mir überarbeitete und funktionierende JavaScript an.
Es wurde die Animation integriert, sowie eine Tastenkombination!

Super, dankeschön !

Hier lernt man wirklich was dazu als Javascript Analphabet wie ich ; eine Menge guter Tips in den verschiedenen Varianten .

Mira_Belle

Zitat von Horstmann

Super, dankeschön !
Hier lernt man wirklich was dazu als Javascript Analphabet wie ich ; eine Menge guter Tips in den verschiedenen Varianten .

Gerne doch,

und mir geht es da genau wie Dir.

Und ganz ehrlich, Du hast ja auch einiges dazu beigetragen, dass es dieses Script gibt.

visoer

Zitat

ich vermute, es ist "mein" Script gemeint.

Ich habe noch Spekulatius von Weihnachten und würde die Kekse hier einsetzten. Mein Tipp ist: 80:20 für Andreas vs. Mira.

Steigt jemand ein?

Aber in jedem Fall zählt: The winner is nierewa.

2002Andreas

Zitat von visoer

The winner is nierewa.

so sollte es ja auch sein.

nierewa

Zitat

Ich vermute, es ist "mein" Script gemeint.

Denn ich habe den Eintrag persToolbar.style.height == '24px'
Das stimmt, aber ich glaube doch, dass nierewa imstande ist, zu kombinieren.

visoer hat ja an vielen Stellen in diesem Thread und in dem Anderen aufgezeigt, wie der Code für die

Tastenkombination auszusehen hat und wo er hingehört.

Stimmt. Das habe ich hinbekommen.

Das Script was ich jetzt nutze ist das von.... Andreas
Wußte nicht das es so wichtig ist welches im Einsatz ist. Sonst hätte ich es gleich mit geschrieben.
In beiden Sripten stckt ja eine Menge arbeit und ich wollte beide honorieren

Vielen dank nochmal an alle Beteiligten.

Mira_Belle

Zitat von nierewa

Das Script was ich jetzt nutze ist das von.... Andreas

Nee, Spaß, die Hauptsache ist ja, dass Du Dein Problem gelöst (bekommen) hast.

Horstmann

Zitat von nierewa

Zitat

Ich vermute, es ist "mein" Script gemeint.

Denn ich habe den Eintrag persToolbar.style.height == '24px'
Das stimmt, aber ich glaube doch, dass nierewa imstande ist, zu kombinieren.

visoer hat ja an vielen Stellen in diesem Thread und in dem Anderen aufgezeigt, wie der Code für die

Tastenkombination auszusehen hat und wo er hingehört.

Stimmt. Das habe ich hinbekommen.

Das Script was ich jetzt nutze ist das von.... Andreas

Alles anzeigen

Heheheee, das macht schon wieder überhaupt keinen Sinn .....

Andreas' Script hat das Tastenkürzel ja schon, und arbeitet nicht mit style.height , sondern der alten style.visibility .

Ist nicht schlimm die konstante Konfusion, nur witzig ; für mein liebstes Script gab's jetzt wieder eine Auffrischung und neue Varianten zur Umsetzung, feine Sache .

Allen einen Guten Rutsch und Frohes Neues !

2002Andreas

Zitat von Horstmann

Andreas' Script hat das Tastenkürzel ja schon

Es ging dem User doch gar nicht um die Tastenkombination.

Die war doch in dem Skript aus Beitrag Nr. 1 schon enthalten.

Es ging nur darum, dass die Lesezeichensymbolleiste mit dem Skript leer war.

Und durch meinen Zusatz in dem Skript ist das jetzt eben nicht mehr so.

Alle anderen Skripte haben also nichts mit seiner eigentlichen Frage zu tun, sondern sind eine "Zugabe".

Mira_Belle

Zitat von 2002Andreas

Alle anderen Skripte haben also nichts mit seiner eigentlichen Frage zu tun, sondern sind eine "Zugabe".

Äh, nö.

style.visibility verursachte das Problem und entweder konnte man es lösen, indem man Deinen "Zusatz",

(was genau ist/war dieser denn?) nutzte oder mit dem Umstellen des Scriptes auf style.height und Pixelwerten.

Zwar hat nierewa geschrieben, dass er/sie 2002Andreas Script nutzt, aber der Einwand mit ...

Zitat von nierewa

Stimmt. Das habe ich hinbekommen.

... steht dazu aber im Wiederspruch.

So ist mir auch weiterhin nicht klar, welches Script nierewa jetzt genau nutzt!

Aber egal, es gibt nun mehre Varianten, die funktionieren und die auf verschiedene Weisen anpassbar sind.

Wünsche allen einen Guten Rutsch und Frohes neues Jahr !

Mira_Belle

Zitat von Mira_Belle

Äh, nö.
style.visibility verursachte das Problem und entweder konnte man es lösen, indem man Deinen "Zusatz",
(was genau ist/war dieser denn?) nutzte oder mit dem Umstellen des Scriptes auf style.height und Pixelwerten.

Ich löse mal selbst das Rätsel.

Du hast CSS-Code hinein gebastelt.

JavaScript

     var css =`
   #PlacesToolbarItems {
  min-width: 200px !important;
  }    
   `;

Und schreibst gleich im Anschluss, dass auch 10px reichen würden.

Beitrag

RE: Lesezeichensymbolleiste leer

[…]

Teste doch bitte mal dieses geänderte Skript:(oder auch ein anderer user)

(Quelltext, 73 Zeilen)

camp-firefox.de/attachment/40085/

2002Andreas

28. Dezember 2022 um 16:57

Horstmann

Zitat von 2002Andreas

Zitat von Horstmann

Andreas' Script hat das Tastenkürzel ja schon

Es ging dem User doch gar nicht um die Tastenkombination.
Die war doch in dem Skript aus Beitrag Nr. 1 schon enthalten.
Es ging nur darum, dass die Lesezeichensymbolleiste mit dem Skript leer war.
Und durch meinen Zusatz in dem Skript ist das jetzt eben nicht mehr so.
Alle anderen Skripte haben also nichts mit seiner eigentlichen Frage zu tun, sondern sind eine "Zugabe".

Alles anzeigen

Alle anderen Scripte basieren auf dem gleichen Problem mit der leeren Leiste, das ja auch Mira und Visoer hatten und so lösen konnten, und wenn ich's recht verstanden hattest Du das Problem auch an einem Punkt ( mit 109 ? ) .

Keine Zugabe - alternative Lösungsansätze ; und meiner Vermutung nach die langfristig besseren, weil der Konflikt mit der integrierten FF Schaltlösung eine potentielle Schwachstelle sein könnte .

Der OP hat in seiner speziellen Weise in der jüngsten Antwort suggeriert dass er Tasten Code eingefügt hat in seiner benutzten Version - was bei Deiner Version unnötig ist - und gleichzeitig darin und davor dass evtl. Pixelwerte der Höhe geändert wurden .

Das macht nun mal keinen Sinn .

Horstmann

Zitat von Mira_Belle
Ich löse mal selbst das Rätsel.
Du hast CSS-Code hinein gebastelt.
JavaScript
     var css =`
   #PlacesToolbarItems {
  min-width: 200px !important;
  }    
   `;
Und schreibst gleich im Anschluss, dass auch 10px reichen würden.

Beitrag

RE: Lesezeichensymbolleiste leer

[…]

Teste doch bitte mal dieses geänderte Skript:(oder auch ein anderer user)

(Quelltext, 73 Zeilen)

camp-firefox.de/attachment/40085/

2002Andreas

28. Dezember 2022 um 16:57

Ja genau, das war der Ansatz in Andreas Script, abgespeichert für die Zukunft .

Meine Vermutung wäre dass die Minimumbreite eher höher als 10px liegt, nach anderweitigen Experimenten in der Richtung , aber kann das Problem nicht zuverlässig reproduzieren, also nicht testen .

Wenn man umgekehrt die Lesezeichensybole in der Leiste mutwillig unsichtbar machen will, so dass sie auf den Overflowbutton reduziert wird, geht das bei mir ab ca. max-width von 20 -30px, je nach sonstigen CSS Einstellungen .

2002Andreas

Zitat von Mira_Belle

So ist mir auch weiterhin nicht klar, welches Script nierewa jetzt genau nutzt!

Och nö, das hat er doch nun längst geklärt

Zitat von nierewa

Das Script was ich jetzt nutze ist das von.... Andreas

Zitat von Horstmann

hattest Du das Problem auch an einem Punkt ( mit 109 ? ) .

Hatte ich auch, und das sollte eine Bestätigung sein, dass das Skript aus Beitrag Nr.1 nicht mehr richtig funktioniert.

Nachdem ich mich dann damit beschäftigt hatte, kam ich auf die Lösung mit dem CSS Code im Skript.

Ich denke, damit sollte das Thema nun wirklich beendet sein.