Beiträge von Sören Hentzschel

  • Mehrzeilige Tableiste für aktuelle Firefox-Versionen

    Sowohl aktueller Code als auch vorgeschlagene Korrektur haben das Problem, dass hier eine Firefox-interne Funktion überschrieben wird und Mozillas aktuelle Änderungen an dieser Funktion nicht berücksichtigt sind. Der Aufruf von _getDragTargetTab() entspricht sogar schon seit 1 1/2 Jahren nicht mehr dem Original-Code und übergibt einen Parameter, den es so lange schon nicht mehr gibt. Gleiches gilt für die _getDropIndex-Funktion selbst auch.

    Wenn man Firefox-interne Funktionen überschreibt, ist es wirklich wichtig, darauf zu achten, dass man möglichst synchron mit dem Original-Code bleibt. In diesem Fall entsteht durch die falsche Funktions-Signatur kein Schaden. Aber das ist die klassische Quelle für Bugs aller Art, die durch Scripts verursacht werden.

    Die Nicht-Berücksichtigung der aktuellen Änderungen wiegt schon schwerer, weil die Nutzer-Einstellung für vertikale Tabs ignoriert wird. Das Script unterstützt ja auch vertikale Tabs. Ich empfehle in jedem Fall, hier die bereits vorhandene Nutzer-Einstellung zu berücksichtigen statt das über eine nicht veränderbare Option im Script zu steuern.

  • MultiRowTabs – Verbesserte und verschönerte Version

    Es gibt noch kein konkretes Problem. Aber es muss sich ja nur jemand auf eine bestimmte Zeile beziehen, dann änderst du in dem Branch etwas und schon stimmt die Zeilennummer unter Umständen nicht mehr. Und die Themen hier sollen ja nach Möglichkeit auch in der Zukunft für andere nachvollziehbar bleiben. ;)

    PS: Apropos Branches, in deinem einen Branch solltest du hoover noch in hover umbenennen. Hoover ist ein Staubsauger. :P

  • MultiRowTabs – Verbesserte und verschönerte Version

    Zitat von Herzmann

    Ich habe nun das Zusammenbauen der CSS mal erheblich kompaktiert. So ist der Code übersichtlicher und dürfte besser performen:
    https://gitlab.com/UlfZibis/multi…?ref_type=heads

    Wenn du den Code extern verlinkst statt hier im Forum zu posten, dann empfehle ich die Verwendung der Funktion „Permanenter Link“ auf GitLab. Denn ansonsten ändert sich der Inhalt mit jeder Änderung und die Zusammenhänge sind später nicht mehr nachvollziehbar, weil Code und Beiträge nicht mehr zusammenpassen.

  • Tele5 - Mediathek, Filme abspielen geht nicht mehr.

    Zitat von madmax25

    Kann ja dann wohl kaum überhaupt irgend ein Add-on sein.

    Zitat von madmax25

    Nur mit deaktiviertem µBlock origin scheint es zu laufen.

    :/

    Zitat von madmax25

    Ich soll also nur mal zu Testzwecken ein neues Profil anlegen, meine Add-ons neuinstallieren und meine Einstellungen rekonstruieren?

    Ja, aber "siiicher doch", ich hab ja auch sonst kein Leben und "so gar nichts anderes" im Leben zu tun.

    Ein Anfang wäre ein neues Profil ohne Anpassungen. Und erst nach Bestätigung, dass das damit funktioniert, wäre es sinnvoll, von da aus das Profil zu erweitern, um die Ursache einzugrenzen.

    Was den zweiten Teil betrifft: Probleme lösen sich in der Regel nicht auf magische Weise von alleine. Man muss auch etwas Zeit investieren. Das kann dir niemand aus diesem Forum abnehmen. Wenn du das nicht möchtest, okay. Das ist deine freie Entscheidung. Aber wenn deine Einstellung ist, dass du Besseres zu tun hast, weiß ich nicht, was du von diesem Forum möchtest. Wir haben auch ein Leben und Besseres zu tun als uns mit Problemen von Nutzern auseinanderzusetzen, die eine Lösung auf dem Silbertablett verlangen - die wir auch überhaupt nicht liefern können. Ein Support-Forum kann grundsätzlich nur Hilfe zur Selbsthilfe anbieten. Wenn du jemanden suchst, der dir die Arbeit abnimmt, musst du jemanden dafür bezahlen, sich das direkt an deinem Computer anzusehen.

    PS: Hilfreich für die Unterstützung durch andere wäre ja auch ein Beispiel-Link …

  • Tableisten CSS anpassen

    Zitat von FuchsFan

    Das kann auch nicht funktionieren, versuche diesen ganz normalen css-Code (user Chrome.css).

    In Beitrag #77 hatte ich erklärt, was das Problem damit ist. Auch damit überschreibst du pauschal jedes slot-Element.

    Zitat von Boersenfeger

    Die Lösung von 2002Andreas aus #94 hingegen funktioniert hier :) und lässt auch den AddonManager unbehelligt.

    Dann lasse ich es zunächst so.

    Würde ich nicht machen, wenn Andreas bestätigt, dass sein Code aus Beitrag #79 so funktioniert. Dann würde ich eher schauen, dass ich die Ursache dafür finde, wieso es bei dir nicht funktioniert. Dein eigener Beitrag #90 hat doch gezeigt, was passieren kann, wenn sich Code an völlig anderen Stellen auswirkt, die mit deiner eigentlichen Absicht nichts zu tun haben. Selbst, wenn du mit dem Code aus Beitrag #94 jetzt kein Problem siehst, dann kommt es vielleicht mit der nächsten Firefox-Änderung an einer komplett anderen Stelle und dann rätselst wir hier wieder, woher das Problem kommt. Vor allem macht Mozilla immer mehr Gebrauch von Custom Elements und Shadow DOM. Die Auswirkungen werden mit der Zeit definitiv eher größer als kleiner, wenn man seine Anpassungen nicht auf ihr eigentliches Ziel limitiert.

  • Tableisten CSS anpassen

    Zitat von Horstmann

    Ich habe mal kurz meinen Franken-Mac mit Nighly 131 angeworfen, da scheint das Prinzip mit dem Variablen Umweg auch für slot zu greifen.

    Ich denke nicht, dass dieser Umweg einen großen Mehrwert bietet. Zur Erklärung möchte auf eine grundsätzliche Gefahr bei diesem Ansatz aufmerksam machen: Eine nicht definierte Variable kann zu unerwarteten Resultaten führen, weil dann eben nicht der Wert greift, der an anderer Stelle, in dem Fall also im originalen CSS der Firefox-Oberfläche, definiert ist, sondern der Standard-Wert, wie er in der CSS-Spezifikation definiert ist. Sofern das slot-Element überall sonst in Firefox seinen Standard-Wert contents für die display-Eigenschaft hat, ist alles gut. Wenn es aber noch ein anderes slot-Element in Firefox gibt, welches beispielsweise display: flex gesetzt hat, würde das bei diesem Ansatz genauso auf display: contents gesetzt, weil du display damit auf undefiniert stellst, was den Standard-Wert laut CSS-Spezifikation zur Folge hat. Gleiches gilt natürlich auch für das Szenario mit der scrollbox und justify-content.

    Letztlich hast du damit also exakt das gleiche Problem wie wenn du direkt scrollbox und slot ohne Variablen in die Datei userChrome.css schreibst, ohne es näher zu spezifizieren. Es sieht zwar auf den ersten Blick so aus, als würdest du das CSS damit nur für die Elemente überschreiben, für welche du die Variable definiert hast. Du übersiehst dabei aber, dass du damit trotzdem das CSS aller Elemente überschreibst, bloß halt nicht für alle Elemente auf den Variablen-Wert, sondern für die anderen Elemente auf ihren Standard-CSS-Wert, der nicht berücksichtigt, ob seitens Firefox ein anderer Wert definiert wurde.

    Die risikoärmste Variante bleibt am Ende, wirklich nur das CSS zu überschreiben, welches man tatsächlich verändern möchte. Dazu braucht es die ::part-Syntax und die wird via userChrome.css nicht unterstützt.

  • Leertaste scrollt zu weit

    Zitat von .DeJaVu

    Irgendwo muss ich anfangen zu suchen ;)

    Und was suchst du? Die Scrolldistanz ist bei Drücken der Leertaste für alle Nutzer gleich (abhängig von den genannten Faktoren).

    Zitat von Mitleser

    Aber das ist bestimmt viel zu umfangreich/veraltet...

    Hast du überhaupt getestet, ob das die Scrolldistanz bei Drücken der Leertaste über die Option hinaus ändert, die bereits genannt wurde und die in deiner Konfiguration überhaupt nicht verändert wird? Vor allem wird dort nur das Physik-Modell von Bezier auf Mass Spring Damper umgestellt und dessen Parameter angepasst. Das verändert, wie sich das sogenannte Smooth Scrolling anfühlt. Ich erkenne aber nicht, was davon die Scrolldistanz verändern soll.

  • Tableisten CSS anpassen

    Ich halte das auch für keine Nebendiskussion, ganz im Gegenteil. Der betroffene Code ist Teil des Shadow DOMs. Und die von mir gezeigte „Lösung“ ist so unspezifisch, dass es ausnahmslos jedes slot-Element (welches wiederum nur im Shadow DOM existiert) betrifft. Es ist nicht absehbar, was für unerwartete Konsequenzen das hat. Deswegen war ja auch meine dringende Empfehlung, meinen Code eben nicht einfach so in die userChrome.css zu schreiben, sondern das näher zu spezifizieren, wofür eine andere Methode erforderlich ist.

  • Leertaste scrollt zu weit

    Zitat von .DeJaVu

    Und damit wir schauen können, was dir das Scrollen verhagelt in Firefox, bitte:

    Wie kommst du zu dieser Annahme? Der Themenersteller hat de facto nach einer Lösung gefragt, die Scroll-Distanz zu reduzieren. Da braucht es weder einen Fehlerbehebungsmodus noch Informationen aus about:support. Das würde ja bedeuten, dass er ein Verhalten hätte, welches andere nicht haben. Darauf deutet bislang nichts hin.

  • Tableisten CSS anpassen

    Zitat von 2002Andreas

    :/

    Offensichtlich funktioniert der Code nur via userChrome.css, nicht über die Entwicklerwerkzeuge. Das ist ungut zum Testen…

    ---

    Mit der Ergänzung funktioniert der Code auch in Firefox Nightly:

    CSS
    slot {
  display: contents !important;
}

    Nur ist das halt sehr unspezifisch und ich weiß nicht, an welchen Stellen sich das ggf. negativ auswirkt. Ich empfehle hier wirklich dringend, den Weg über userChromeShadow.uc.js zu gehen und den Code spezifischer zu schreiben, damit sich das nur in der Tableiste auswirkt.

    Generell sind aktuell im Bereich der Tabs praktisch täglich Änderungen zu erwarten, da mit sowohl vertikalen Tabs als auch Tab-Gruppen, deren Implementierung gestartet ist, gerade zwei große Baustellen bestehen, welche ganz direkt die Tabs betreffen. Und dann kommt noch allgemeines Aufräumen dazu, welches die Implementierung dieser neuen Features begünstigen soll, wozu die Änderung zählt, welche das Problem verursacht hat:

    1913322 - Remove overflow / underflow event usage from arrowscrollbox / tabs.js.
    RESOLVED (emilio) in Toolkit - UI Widgets. Last updated 2024-08-23.
    bugzilla.mozilla.org

    Es ist gerade nicht die Zeit, um Firefox Nightly zu verwenden und sich darauf zu verlassen, dass individuelle Anpassungen des Tab-Codes stabil bleiben.

  • Leertaste scrollt zu weit

    Zitat von .DeJaVu

    Also ist die gescrollte Höhe immer eine Fensterhöhe? Auch bei dynamisch aufgebauten Seiten?

    Ja, ist es - grob gesagt. Die exakte Kalkulation ist Fensterhöhe minus zwei Zeilen oder 10 Prozent, was auch immer davon den kleineren Wert ergibt. Die Zeilenhöhe hängt von der Schriftgröße der Website ab, hat aber einen Minimumwert von 5 Pixeln.

    Zitat von .DeJaVu

    Und die Leertaste ist unveränderlich in Firefox?

    Ziemlich. Den eben erwähnten Minimumwert kannst du marginal beeinflussen, indem mousewheel.min_line_scroll_amount geändert wird. Bei einer Erhöhung von 5 auf 50 wäre die Kalkulation dann Fensterhöhe minus 100 Pixel. Aber da der Wert eben nur greift, solange er geringer als die Zeilenhöhe ist, gibt es de facto darüber kaum Spielraum. Und da diese Einstellung auch das normale Scrollen mit der Maus betrifft, ist das auch nicht unbedingt optimal, wenn es einem nur darum geht.