[PATCH] Farbige progressbar, SNR/BER-Anzeige in Infobar

[JtG-Riker]

Jo, hatt ich gemacht, aber der meckerte an den Zeilen und da kam hunk failed.

Naja, egal, wär eh dafür das es so ins cvs kommt - sieht eh besser aus als Matrix

[bosmann]

bei mir auch:

Code: Alles auswählen

bosmann:~/newmake/tuxbox-cvs$ patch -p0 -i /home/bosmann/Desktop/progressbar.cpp.diff
patching file b/tuxbox/neutrino/src/gui/widget/progressbar.cpp
Hunk #1 FAILED at 135.
Hunk #2 FAILED at 186.
2 out of 2 hunks FAILED -- saving rejects to file b/tuxbox/neutrino/src/gui/widget/progressbar.cpp.rej

am beste gleich ins cvs

[rhabarber1848]

Code: Alles auswählen

bosmann:~/newmake/tuxbox-cvs$ patch -p0 -i /home/bosmann/Desktop/progressbar.cpp.diff

ungetestet:

Code: Alles auswählen

bosmann:~/newmake/tuxbox-cvs$ cd apps && patch -p1 -i /home/bosmann/Desktop/progressbar.cpp.diff

[dbt]

Dann schaue man in die rejects und ändert das. Da ist ein Unterschied in der History, wegen eines Fixes, der im CVS nicht drin ist. Das kann nichts bewegendes sein.

[bosmann]

danke rhabarber, es scheint geklappt zu haben:

Code: Alles auswählen

bosmann@bosmann:~/newmake/tuxbox-cvs$ cd apps && patch -p1 -i /home/bosmann/Desktop/progressbar.cpp.diff
patching file tuxbox/neutrino/src/gui/widget/progressbar.cpp

mal sehen was der compiler sagt...

update:
es klappt!

[dbt]

Der Schatten ist noch irgendwie Gaga, wenn ich mir das genauer ansehe

[JtG-Riker]

Bei mir gehts immer noch net, stellt doch ma einer den Patch ein oder am besten direkt einchecklen

Gruß Riker

[bosmann]

anbei

[rhabarber1848]

Nebenbei habe ich bei mir so wie auch bei Neutrino-HD das Matrixdesign rausgeworfen.

committed to CVS

[GetAway]

Hi,

die Aufteilung der Farben finde ich im EPGinfo nicht so prickelnd. Sollte man das nicht einfarbig machen bzw. anders aufteilen?

[Mourice]

Hi,

ich würde das "Gelb" weglassen, und die letzten 10-15% dann "Rot" machen.

In infoviewer.cpp so um Zeile 1642:

CProgressBar pb(true, -1, -1, 85, GREEN_BAR, 0, true);

MfG

[JtG-Riker]

Kann man das noch irgendwie ändern das bei der Nokia Sat nur ~16% Singalstärke angezeigt werden? Müsste man sicher im Treiber
machen, ich meine da wär ma irgendwann was geändert worden im Ves19x3

Ansonsten bin ich mittlerweile zufrieden

Gruß Riker

[Gaucho316]

http://forum.tuxbox.org/forum/viewtopic.php?f=3&t=49039

[bruno71]

Das liegt wohl an dieser änderunung die damals gemacht worden ist:

tda8044h.c

Code: Alles auswählen

-		u8 gain = ~tda8044_readreg(fe->i2c, 0x01);
+		u8 gain = tda8044_readreg(fe->i2c, 0x01);

ves1x93.c

Code: Alles auswählen

-		u8 signal = ~ves1x93_readreg (i2c, 0x0b);
+		u8 signal = ves1x93_readreg (i2c, 0x0b);

[Gaucho316]

Ich bin heute endlich mal dazu gekommen, mir das neuste JtG-Release zu installieren, um mir die Neuerungen auch mal ansehen zu können. Dabei ist mir gleich ein Fehler aufgefallen. Hat man die SNR/BER-Anzeige aktiviert, sieht die Infobar im Movieplayer kaputt aus. Es wird viel zu viel Platz für die Buttonleiste vorgesehen. Dort kann die Infobar ja eigentlich immer im kleinen Format bleiben.

Edit: Außerdem wäre ich dafür, den deutschsprachigen Eintrag in den Einstellungen von "Progressbar LED" in "Fortschrittsbalken LED" umzubenennen.

[Gaucho316]

Hat man die SNR/BER-Anzeige aktiviert, sieht die Infobar im Movieplayer kaputt aus. Es wird viel zu viel Platz für die Buttonleiste vorgesehen. Dort kann die Infobar ja eigentlich immer im kleinen Format bleiben.

Ich habe mal einen Patch dafür gebaut. Damit sieht die Infobar im Movieplayer wieder gut aus, auch wenn man die Satellitenanzeige aktiviert hat. Außerdem habe ich die Icons für 16:9 und AC3 im Movieplayer nach rechts gerückt. Mich hat schon länger gestört, dass da so eine unnötige Lücke ist.

Link entfernt, da Patch im CVS

Und beim Einchecken bitte nicht vergessen, den deutschsprachigen Eintrag in den Einstellungen von "Progressbar LED" in "Fortschrittsbalken LED" umzubenennen, falls niemand etwas dagegen hat.

[Gaucho316]

Ich habe noch einen Fehler entdeckt. Wenn man die SNR/BER-Anzeige aktiviert hat und umschaltet, hat der für die SNR/BER-Anzeige vorgesehene Platz die Hintergrundfarbe der Buttonleiste, solange die Werte noch nicht angezeigt werden. Es sieht also aus, als wäre die Buttonleiste riesig. Der folgende Patch fixt das.

Link entfernt, da Patch im CVS

[dbt]

Ich habe noch einen Fehler entdeckt. Wenn man die SNR/BER-Anzeige aktiviert hat und umschaltet, hat der für die SNR/BER-Anzeige vorgesehene Platz die Hintergrundfarbe der Buttonleiste, solange die Werte noch nicht angezeigt werden. Es sieht also aus, als wäre die Buttonleiste riesig. Der folgende Patch fixt das.

Link entfernt, da Patch im CVS

Jo, hat mich auch gestört!

[Gaucho316]

Und noch ein Patch für den farbigen Fortschrittsbalken. Ich hoffe, es ist der letzte. Die Berechnung der Anzahl der anzuzeigenden Balken und die tatsächliche Anzahl der gezeichneten Balken stimmten nicht überein. Außerdem habe ich die Zeichenreihenfolge von Schatten und Hintergrund umgedreht, da zuerst der Schatten gezeichnet werden muss. Weiterhin habe ich noch überflüssigen Code entfernt, da ein- und derselbe passive Balken unnötigerweise mehrmals gezeichnet wurde. Das war noch ein Überbleibsel aus der ersten Variante der Anzeige mit Punkten.

Link entfernt, da Patch im CVS

[Gaucho316]

Einen hab ich noch. In den Detailinfos zur aktuellen Sendung (EPGView) ist der Maximalwert für den Fortschrittsbalken auf 104 gesetzt. An anderer Stelle wird aber mit einem Maximalwert von 100 gerechnet. Der folgende Patch korrigiert das.

Link entfernt, da Patch im CVS

[dbt]

Danke! Sollte jetzt alles drin sein.

[Gaucho316]

Super! Ich danke dir fürs Einchecken.

[Gaucho316]

Einen hab ich noch. In den Detailinfos zur aktuellen Sendung (EPGView) ist der Maximalwert für den Fortschrittsbalken auf 104 gesetzt. An anderer Stelle wird aber mit einem Maximalwert von 100 gerechnet. Der folgende Patch korrigiert das.

Dieser Fehler existiert auch in Neutrino HD.

Link entfernt, da Patch im CVS

[Mourice]

Hi,

mir ist aufgefallen, dass die farbigen Progressbars bei den einzelnen Farben verschiedene Höhen haben.

Ich habe schon gesucht, aber ich finde die Stelle nicht.

Kann mir jemand auf die Sprünge helfen ?

MfG

[Gaucho316]

Bist du dir sicher, dass das nicht eher an deinem Fernseher liegt? Denn laut Code ist die Höhe bei allen Farben gleich (Zeile 202, 208 und 217 in progressbar.cpp), wie man auch auf den Screenshots aus den Postings weiter oben sehen kann.