Beiträge von Horstmann

Zitat von BrokenHeart

Kannst du mal ein GIF von deinem Darstellungsproblem erzeugen? Irgendwie weiß ich im Moment nicht, wie ich mir das vorzustellen habe.

Wer hätte es gedacht, der Mac kann Bildschirmaufnahmen, mit QT Player.

Hier dann gifs: #1 mit dem Problem, #2 ohne, nur als Info.
Wie erwähnt, nur dann ein Problem wenn die Breite ohne Zähler kleiner als die max-width (30em ?) des Elements ist, und nur beim ersten Öffnen in einem Fenster.

#1 wäre dann nur mit:

        if(bmbMenu)
           bmbMenu.addEventListener('popupshown', onPopupShowing );

#2 wäre die alte Version mit popupshowing, oder mit gleicher Wirkung diese erweiterte neue Version:

        if(bmbMenu)
           bmbMenu.addEventListener('popupshown', onPopupShowing );
        if(bmbMenuPop)
           bmbMenuPop.addEventListener('popupshowing', onPopupShowing );

Hoffe das klappt mit den gifs:

Mira_Belle , war nicht ernst gemeint, nur eine kleine Stichelei, sorry!

Zitat von BrokenHeart

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.

Aaaah, das macht Sinn.
Letztlich wäre es uU sinnvoller, die fehlende Darstellung in Menüleiste/Lesezeichen anzugehen, statt das Lesezeichenmenü insgesamt zu ändern mit popupshowing ==> popupshown.

Dein ursprünglicher Ansatz funktioniert ja generell anscheinend immer noch prima, hier am Mac sowieso.

Zitat von Mira_Belle

Mein komplettes Skript in Kombination mit dem Autopopup-Skript.

Du kannst extra für Mac erforderlich reinschreiben bis die Kühe bellen, aber da gibt es keinen Zusammenhang.

Wie kommt das Autopopup Ding in dieses Thema, ohne Zusammenhang, haben wir hier etwa eine Sammelthread?

Zitat von BrokenHeart

Damit meinte ich, dass sich das Popup verbreitert, wenn die Zähler hinzugefügt werden. Da die Addition aller Verzeichnisse und Lesezeichen eine gewisse Zeit in Anspruch nimmt, hat man eventuell einen sichtbaren Effekt, ganz in Abhängigkeit von der "bestehenden" Breite des Popups und seiner Position. Wie ich weiter oben geschrieben habe, stört mich persönlich dieser Effekt überhaupt nicht, bloß hatte ich gedacht, dass Horstmann genau das meint, als er von "Darstellungsproblemen" schrieb, was er aber in seiner letzten Antwort verneint hat.

Horstmann : Kannst du mal ein GIF von deinem Darstellungsproblem erzeugen? Irgendwie weiß ich im Moment nicht, wie ich mir das vorzustellen habe.

Das mit dem Gif aufnehmen müsste ich mir erst noch beibringen.

Der von dir (und mir) beschriebene Effekt stört mich auch nicht, aber mit popupshowing ist er sanft, mit popupshown ruckartig.
Eben weil vermutlich die Zähler zu spät dazugerechnet werden zur Breite mit Letzterem, um noch eine bestehende Animation auslösen zu können.
Was aber wie gesagt bei mir dann gelöst wird durch den/die aktuellen Codevorschläge, also auf meiner Seite no problemo.

Ich mach jetzt aber mal meine Version fertig, bin schon ganz verwirrt.

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!

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:

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

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?

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.

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.

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

Oder so:

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

Zitat von BrokenHeart

Zitat von Mira_Belle

Dieser komische doppelte Aufruf ist bei mir nicht nötig,
aber auch so funktioniert es.

Interessant. Muss mal schauen, ob nur bei mir dieser merkwürdige Effekt auftritt. Vielleicht mal ohne meine 'userChrome.css' testen .

Zitat von Mira_Belle

Das "Event" auszulösen, bevor und nachdem das Popup-Menü vollständig angezeigt wird,
macht meiner Meinung keinen Sinn.

Und BrokenHart hat hier ja recht gut erklärt, warum das "Event" erst nach dem vollständigen Anzeigen
des Menüs ausgelöst werden sollte.

Zitat von BrokenHeart

In beiden FF-Versionen gibt es ein Attribut hasbeenopened des Menü-Popups, welches gesetzt wird, wenn man erstmalig das Popup aufgerufen hat. Deshalb muss hier ein zweites mal der Button betätigt werde, damit das Skript, so wie es jetzt ist, funktioniert.

Da es hier um Millisekunden geht, so denke ich, sieht man das mit der Verzögerung nicht.
Man bekommt es einfach nicht mit.

Hier am Mac, für den #bookmarks-menu-button / Lesezeichen-Menü-Button:

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.

Danach, und für Submenus bleibt die Grösse, vermutlich weil sie mit hasbeenopened festgelegt bleibt; das Attribut sitzt hier am Hauptpopup, ID: #BMB_bookmarksToolbarPopup.

===>

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

//#1 alt: Passt Popup Breite sauber an beim Oeffnen 
if(bmbMenu)
    bmbMenu.addEventListener('popupshowing', onPopupShowing );

//#2 neu: Passt Popup Breite ruckartig an beim/nach Oeffnen 
if(bmbMenu)
    bmbMenu.addEventListener('popupshown', onPopupShowing );  // popupshowing => popupshown

//#3 neu, doppelt: Passt Popup Breite sauber an beim Oeffnen 
if(bmbMenu) {
     bmbMenu.addEventListener('popupshowing', onPopupShowing );   // popupshowing => popupshown
     bmbMenu.addEventListener('popupshown', onPopupShowing );   // popupshowing => popupshown
     }

Hat man noch weitere CSS für Positionierung, Grösse etc. für dieses Hauptpopup, könnte das weitere Anzeigeprobleme mit sich bringen, wie von Brokenheart beschrieben.

Zitat von BrokenHeart

Zitat von Horstmann

Und wenn du beide hinzufügst?

Ich vermute mal du meinst:

PS: Ich glaube zwar, dass es nicht sehr effizient ist, das gleiche Prozedere mehrfach aufzurufen, aber dann muss man halt als "Heimwerker und Bastler" mehr Geld für die entsprechende Hardware hinlegen, dann gleicht sich das wieder aus!

Ooups, sorry, ja, genau das.
Was Schlaueres fällt mir nicht ein - schon gar nicht ohne es testen zu können, sogar wenn ich JS könnte.

PS; Braucht man die Änderung eigentlich unbedingt für den bookmarks-menu-button, weil sie evtl. in der Lesezeichenleiste mitbenutzt wird?

Das hier reicht nicht, mit Anpassung nur für bookmarksMenu?

       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('popupshown', onPopupShowing ); // popupshowing => popupshown
       if(persToolBar)
           persToolBar.addEventListener('popupshowing', onPopupShowing );
   }

Zitat von BrokenHeart

Test-Skript:

Beim Lesezeichen-Menü über die Menüleiste sieht es erst mal ganz gut aus...

Beim Öffnen des Popups über den Lesezeichen-Menü-Button funktioniert es zwar auch, aber das Popup wird erst ganz kurz nach links geöffnet und wieder geschlossen, bevor es dann nach rechts wieder geöffnet wird .

Und wenn du beide hinzufügst?

        if(bmbMenu)
           bmbMenu.addEventListener('popupshowing', onPopupShowing );
           bmbMenu.addEventListener('popupshown', onPopupShowing );

Hauptsache ich hab nicht mal wieder was vergeigt!

Prima Fund wieder mal von BrokenHeart.

Zitat von saobento

Bei mir hat die Änderung nicht geholfen, ist allerdings in einem Linuxsystem, vielleicht macht das dann einen Unterschied.

Hmm, du könntest das hier noch probieren, als Ersatz für den relevanten Codeteil.

panel:not(#PopupSearchAutoComplete) {
    opacity: 0 !important;
    transform: translateY(-100%) !important;
}

panel:not(#PopupSearchAutoComplete)[panelopen] {
    --panelui-subview-transition-duration: 2s !important;
    opacity: 1 !important;
    transform: translateY(0) !important;
    transition: transform 0.9s ease-in-out, opacity 0.9s ease-in-out !important;
}

Zitat von Mira_Belle

Ist auch mit diesem Skript so.

Danke!
Sind ja ziemlich verschiedene Ansätze, mit dem gleichen Problem, die Frage wäre dann halt wann ist das Problem aufgetreten, oder war es immer schon da?
Wobei dann hätte sich bestimmt schon jemand gemeldet.

Also, ich kann ja leider nicht die Menüleiste testen, und für mich am Mac konnte sich auch der Bug nie bemerkbar machen.

Es würde evtl. Sinn machen für Windows User nach einer älteren Version zu suchen, die funktioniert, und dann mit neusten Versionen zu vergleichen, um festzustellen an welchem Punkt der Ärger losging.

Im Fx Code sehen die IDs etc. ziemlich ähnlich aus, aber die ersten 3 werden offensichtlich anders geladen als die Lesezeichenordner darunter; könnt ja mal die Trennlinie dazwischen rausschmeissen zum Spass.

Hier ungefähr geht die Diskussion mit den relativ aktuellen Änderungen los, da gibt es Einiges an Code von BrokenHeart , grisu2099 und Mira_Belle , der evtl. die Problematik nicht aufweist.

Zitat von grisu2099

Zitat von Horstmann

Und wenn ihr jetzt noch kurz das Script von Mitleser hier testen könntet

Das gleiche in grün...

Meine Skript-Version:

Vielen Dank!

Hmm; was mir gerade aufgefallen ist, in meinen letzten Scripts gibt's einen Fehler - der aber bei dir, Mitleser und Mira nicht vorhanden ist, also uU nichts ausmacht.
Sitzt aber an genau der relevanten Stelle.

Zeilen 286/287, in meinem Script, und dadurch auch auf Endor's Github:

        if(bookMenu)-
            bookMenu.addEventListener('popupshowing', onPopupShowing );

Das - nach if(bookMenu)- muss raus... ==> if(bookMenu) ...

Zitat von grisu2099

Bei mir wird zuerst immer nur bei der Lesezeichen-Symbolleiste die Daten angezeigt. Erst beim zweiten Mal werden die Daten für die weiteren Zeilen angezeigt. Heißt, ich muß das Popup ein zweites Mal öffnen, bevor bei den Ordnern die Daten angezeigt werden.

Danke; und wir reden nur über die Menüleiste, korrekt?
Könnste mal bitte deinen Code einstellen?

Zitat von Mira_Belle

Zitat von Horstmann

... oder beschreiben welche 3 Einträge zuerst geladen werden?

Und die Counter ab dem zweiten Separator werden erst beim zweiten Mal öffnen angezeigt.
Schon seltsam.

Danke auch; das sind 2 separate Bereiche, das Script scheint den 2ten nicht schnell genug zu erreichen.

Und wenn ihr jetzt noch kurz das Script von Mitleser hier testen könntet - Icons sind erstmal egal - gibt es da auch das Problem?
Damit könnte man das eingrenzen; wie gesagt, ich kann's am Mac nicht testen.

Gab's das Problem schon immer, oder schwer zu sagen?

Zitat von grisu2099

Zitat von Horstmann

junger Mann,

Das werte ich jetzt mal als Kompliment...

Absolut!

Aber sag mal, hast du das Problem auch, du hast ja glaub ich deine eigene Codeversion?
Also teilweise starke Verzögerung/Fehlfunktion beim Laden dieser Einträge in Menüleiste -> Lesezeichen Schaltfläche?

Zitat von grisu2099

Zitat von Horstmann

welche 3 Einträge zuerst geladen werden?

Immer die obersten...

Mac hier, junger Mann, Mac!
Das Popup sieht in Windows anders aus, andere Inhalte.

Zitat von Endor

Ja so funktioniert es hier einwandfrei.

Danke !
Keine saubere Lösung, aber die einfachste.
Nur nochmal: speziell die panel Codeteile sind schwer zu kontrollieren und offensichtlich nicht durchgetestet, und das Ganze ist nur ein Test!
Als nächstes kommt noch Einer und will die Suchergebnisse Popups auch umanimiert haben, etc.., ...

Zitat von Endor

Es tritt hier nur bei Schaltfläche Lesezeichen in der Menüleiste auf.
Wenn ich warte werden die fehlenden später nachgeladen.
Mfg.
Endor

Danke; Menüleiste ist hier am Mac nicht testbar, da fällt mir leider nichts zu ein.

Das um teils Sekunden verspätete Auftauchen hatte ich in einigen wenigen Fällen in alten Versionen, aber dann waren alle Einträge betroffen.

Erstmal bin ich aber beruhigt; wenn es in der Lesezeichenleiste und der Schaltfläche Lesezeichenmenü normal funktioniert, ist wohl kein generelles Problem da.

Falls du das aktuellste Script benutzt, und die Verblassenfunktion aktiv hast, probier mal let bm_opacity_null = 0;, und/oder let trenner = 1; oder let trenner = 3; zu setzen, und schauen ob es besser ist.

Und könntest du bitte einen Screenshot vom Menüleisten Popup machen, und zeigen oder beschreiben welche 3 Einträge zuerst geladen werden? Ich hätte da einen Verdacht.