[Patch] Überarbeitung der Buttonbeschriftung im Moviebrowser

[Gaucho316]

Seit einiger Zeit stört mich schon die Beschriftung der Buttons in der Fußleiste des Moviebrowsers. Erstens fehlt neben dem roten und dem grünen Button hinter dem Doppelpunkt jeweils ein Leerzeichen. Und zweitens finde ich, dass es dämlich aussieht, dass einfach nur "Filter:" neben dem grünen Button steht, wenn kein Filter gesetzt ist. Ich habe für diesen Fall einfach "Filter: -" daraus gemacht. Getestet habe ich meinen Patch und kann sagen, dass er funktioniert.

Link zum Patch entfernt

[GetAway]

@Gaucho316

Es gibt noch ein Problem mit der Fußleiste, wenn der Filter so benutzt wird wie abgebildet.
Das hat zwar jetzt nichts mit deinem Patch zu tun, aber du bist ja gerade da dran.

[Gaucho316]

Ich kann mir das Bild zwar nicht im Großformat ansehen, meine aber trotzdem erkennen zu können, was du meinst. Bei mir läuft die Schrift allerdings nicht aus dem Fenster. Hast du sie größer als standardmäßig eingestellt?

[GetAway]

Die Funktion ist wohl i.A. Ich habe es mal woanders gehostet.
Sortieren nach Qualität, Filter auf Inof1, Titel markieren. OK. Tritt nur bei langen Titeln auf.

[Gaucho316]

Also, wie gesagt, bei mir passt's auch bei langen Titeln. Deshalb ja die Vermutung mit der Schriftgröße. Auf deinem Screenshot kommt mir die Schrift nämlich wirklich größer als standardmäßig vor. Die Beschreibung neben dem grünen Button wird übrigens nach 24 Zeichen abgeschnitten und dann werden 3 Punkte rangehängt.

[GetAway]

Die Schriftgröße ist unverändert. Das Problem liegt an der proportionalen Schrift die benutzt wird.
Da kannst du bei 24 Buchstaben abschneiden wie du willst, wenn der Titel aus überwiegend breiten Buchstaben
besteht, hängt es hinten drüber.

[Gaucho316]

Ok, hast gewonnen. Ich habe es jetzt nachvollzogen. Ich hatte "Sortieren" auf "Datum" und nicht auf "* (Qualität)" stehen. Deshalb hat es bei mir immer gepasst. Am einfachsten wäre es wohl, den Abschneidewert von 24 auf 22 oder 20 zu verkleinern. Dazu müsste die Zeile 1662 von moviebrowser.cpp (vom aktuellen CVS-Stand ausgehend) wie folgt geändert werden:

vorher:

Code: Alles auswählen

filter_text = filter_text.length() < 24 ? filter_text : filter_text.substr( 0, 24 ) + "...";

nachher:

Code: Alles auswählen

filter_text = filter_text.length() < 20 ? filter_text : filter_text.substr( 0, 20 ) + "...";

[GetAway]

Die Schrift ist eigentlich nicht zu lang, sondern der Abstand.
Man könnte auch den Ok-Button Fix machen, dann würde es mit 24 immer
noch passen, eventuell sogar auf 26 erhöhbar.

[Gaucho316]

Ich hab's mir mal angeschaut. Bei dem entsprechenden Code sehe ich nicht wirklich durch. Das müsste sich dann jemand anders ansehen. Der Einfachheit halber würde ich aber immer noch die Kürzung des Abschneidewertes vorschlagen.

[GetAway]

@Gaucho316

Ich habe deinen Patch genommen und noch eine Begrenzung reingemacht.

moviebrowser_rework-button-captions-in-footer_2010-12-29_2237.diff

[Gaucho316]

Wo hast du denn den Wert 115 her?

[GetAway]

Augenmaß? Getestet.

[Gaucho316]

Da wäre ich aber vorsichtig. Der Text kann doch in verschiedenen Sprachen verschieden lang sein. Kann man vielleicht die Berechnung von width3 nach oben ziehen und die dann nutzen? Dabei fällt mir auf, dass in refreshFoot() oben ein String ok_text definiert wird, der nie benutzt wird, und weiter unten dann ok_text2. Und eine unnützes if ist auch noch drin. Irgendwie ist das alles ein klein wenig überarbeitungswürdig. Ich setze mich morgen mal dran.

[dbt]

Probier das mal mit einer exotischeren Schriftart wie russisch oder griechisch. Da passt überhaupt nichts mehr. Die Brechnung der Stringlänge ist dann völlig daneben. Das Problem hatte ich mal mit der buttons-Klasse, womit ich bisher immer noch nicht wirklich zufrieden bin.

[Gaucho316]

Ich hab's jetzt nicht mit 'ner exotischen Schriftart probiert, aber trotzdem überarbeitet. Was haltet ihr davon?

Link entfernt, da Patch im CVS

[GetAway]

Sieht gut aus.