BUG in der Bouquetverwaltung

[PauleFoul]

Hi,

mir ist afgefallen, dass die Logik/Funktion der +/- Tasten in der
Bouquetverwaltung beim Blättern in der Anbieterliste verdreht
ist.

Kann das vielleicht jemand korrigieren!?


Gruß
____Paule

[rasc]

immer noch? das war doch schonmal korrigiert.

[PauleFoul]

immer noch? das war doch schonmal korrigiert.

Ja leider...

[PauleFoul]

@ all Dev es

Kann das mal einer "kurz" drehen? Nervt irgendwie ziemlich finde ich...


Gruß
____Paule

[Gorcon]

Kann das mal einer "kurz" drehen?

Aber bitte nur einmal.

Mir ist es auch erst gestern aufgefallen.

Gruß Gorcon

[Nico 77]

CVS 05.01., kann da irgendwie keine Fehler feststellen!?

[racker]

CVS 31.12 -> Fehler ist nachvollziehbar.

@PauleFoul
ohne es zu wollen gibts da jetzt vlt. eine Diskussion:
definiere einmal Seite hochblättern (page up)
und Seite runterblättern (page down)
im Sinne von nächste Seite und vorhergehende Seite
und welche Tasten (+/-) würdest du dafür verwenden?

Hintergrund: so eine "Verwechslung" passiert - meiner Meinung nach- nur,
wenn die Bedeutung nicht klar festgelegt ist.
Wie man was festlegt kommt auf die Sichtweise an.

Ich würde das so sehen:
page up ->nächste Seite -> Taste +
page down ->vorhergende Seite -> Taste -

[PauleFoul]

@racker

Also ich definiere das ich in einer Liste mit der -Taste eine Seite nach
unten blättere und mit der +Taste eine Seite nach oben. Das ist
überall in Neutrino so und bedarf deshalb eigentlich keine Defintion
meinerseits.
Nur in der Bouquetverwaltung (blättern in der Anbieterliste) ist es eben
anders (verdreht).


Gruß
____Paule

[racker]

... Das ist überall in Neutrino so ...

Das war nicht die Frage.
Ich habe oben extra die Begriffe nächste/vorhergehende verwendet.
Nur wenn alle unter page up (darauf wird im Code abgefragt), das gleiche
(nächste/vorhergehende) verstehen, passieren solche Sachen nicht mehr.
Die +/- Tasten sind ja vom Benutzer einstellbar.
Ob man alles einheitlich "richtig" oder einheitlich "falsch" macht sieht man
ja von außen nicht.

Btw. das Verhalten ist nicht nur in der Anbieterliste so, sondern auch wenn man ein Bouquet ausgewählt hat in der Senderliste. (Bouquetverwaltung)
Vlt. komme ich morgen dazu mir das anzuschauen.

Gruß

[PauleFoul]

... Das ist überall in Neutrino so ...

Das war nicht die Frage.
Ich habe oben extra die Begriffe nächste/vorhergehende verwendet.
Nur wenn alle unter page up (darauf wird im Code abgefragt), das gleiche
(nächste/vorhergehende) verstehen, passieren solche Sachen nicht mehr.
Die +/- Tasten sind ja vom Benutzer einstellbar.
Ob man alles einheitlich "richtig" oder einheitlich "falsch" macht sieht man
ja von außen nicht.

Btw. das Verhalten ist nicht nur in der Anbieterliste so, sondern auch wenn man ein Bouquet ausgewählt hat in der Senderliste. (Bouquetverwaltung)
Vlt. komme ich morgen dazu mir das anzuschauen.

Gruß

Also das was Du hier erzählst muss ich jetzt nicht wirklich verstehen, oder!?

Es ist doch völlig ergal ob ich hoch für runter und du runter für hoch hätst
oder was weiss ich.
Fakt ist, das die Funktion in der Bouquetverwaltung verdreht/spiegelverkehrt zu den übrigen "Blätterfunktionen" in Neutrino
verhält. Wenn ich die "Logik" in den Tasteneinstellungen drehe, ändern
sich alle und somit ist die Funktionalität der Bouquetverwaltung (Anbieterliste)
immer noch verdreht.
Egal ob das im Code cursor_up oder page_up oder was weiss ich heisst!


Gruß
____Paule

PS: Wenn wir über Logik sprechen sollten wir uns fragen ob die
Beschreibung in den Tasteneinstellungen stimmt!?!?

[racker]

Also das was Du hier erzählst muss ich jetzt nicht wirklich verstehen, oder!? ...

Musst du nicht, aber dann hör auf zu quängeln
und sei in der Zwischenzeit zufrieden wie es jetzt funktioniert

[mogway]

Kann das vielleicht jemand korrigieren!?

Sollte nun richtig sein.

Gruß
mogway

[DrStoned]

Kann das vielleicht jemand korrigieren!?

Sollte nun richtig sein.

Gruß
mogway

Vielen Dank dafür.

Greetz von DrStoned

[PauleFoul]

Kann das vielleicht jemand korrigieren!?

Sollte nun richtig sein.

Gruß
mogway

Danke mogway


Gruß
____Paule

[racker]

Habe es mir mal angeschaut.
Die letzten Änderungen haben bewirkt, dass sich page up und page down
im Bouqueteditor überall einheitlich verhalten- nur nicht zum Rest von Neutrino.
Dieses einheitliche Verhalten herzustellen ist nicht schwer -nur viele Dateien zu ändern.

Ab hier wird es für user uninteressant

Die Frage ist wie soll man das Verhalten ändern:
cursor up(down) und page up(down) haben dieselbe Richtung -> wie im Bouqueteditor
cursor up und page up verhalten sich in Bezug auf die Richtung entgegengesetzt -> Rest von Neutrino

Ich persönlich finde die "Kopplung" wie im Bouqueteditor am elegantesten.
So läßt sich jedes Widget gefahrlos um die "paging" Funktion erweitern.
Man läuft dabei nicht Gefahr, dass das Verhalten entgegengesetzt zum Rest von Neutrino ist.
Hier noch Codeschnipsel als Beispiel.
Boqueteditor:

Code: Alles auswählen

else if (msg==CRCInput::RC_up || msg==(neutrino_msg_t)g_settings.key_channelList_pageup)
		{
			if (!(Bouquets.empty()))
			{
				int step = 0;
				int prev_selected = selected;

				step = (msg == (neutrino_msg_t)g_settings.key_channelList_pageup) ? listmaxshow : 1;  // browse or step 1
				selected -= step;

andere:

Code: Alles auswählen

else if ( msg == (neutrino_msg_t)g_settings.key_channelList_pageup )
		{
			selected+=listmaxshow;

Da es mogway vorgeschlagen hat stelle ich hier die Frage:
Wie soll es jetzt laufen?


PS: Für user läuft das transparent ab, da wird kein Unterschied gesehen.

[Nico 77]

Kann das vielleicht jemand korrigieren!?

Sollte nun richtig sein.

Gruß
mogway

Das Tastenmenü zeigt an Seite hochblättern: vol: inc, wie auch richtig. In der Bouquetverwaltung ist nun aber genau falsch rum, mit Minus muss ich nun vorwärts und mit Plus rückwärts gehen.
Vorher war es hier richtig, jetzt wurde wieder ein Bug eingebaut der schonmal behoben wurde.

[racker]

Die Übersetzung "Seite hochblättern" ist etwas unglücklich gewählt.
Die Funktion bedeutet page up ->analog Bild(Seite) oben, wie auf der PC-Tastatur ->vorhergehende Seite
entsprechend dann page down ->Bild unten ->nächste Seite
Wenn du den aktuellen CVS-Stand hast, kannst du im Menü experimentelle Einstellungen
mit enable_new_pageupdown = 1 das verhalten vereinheitlichen.

Um die korrekte Bezeichnung sollten sich bitte die GUI-Spezis kümmern - ich werde daran nichts ändern.

Von der Planung her werde ich den fehlerhaften Code in ca. 4 Wochen entfernen.

[Nico 77]

Bis vor kurzem war das meiner Meinung nach richtig und so ist es auch bei jedem anderen Reciever(z.b. Pace, Philips, Technisat). Mit der + Taste scrollt man runter und mit der - Taste zurück.

Ps: enable_new_pageupdown = 1 zeigt hier keine Wirkung und das Menü passt irgendwie nicht zum Rest, bißchen unglücklich aufgebaut.

[PauleFoul]

Hallo LEUTE!!!

Das Verhalten kann man in den Tasteneinstellungen doch anpassen.

Wichtig is nur das es dann ÜBERALL gleich ist. Jetzt ist nämlich die
gesamte Bouquetverwaltung zerhauen!


Gruß
____Paule

[Nico 77]

Ja genau, nur jetzt ist es nicht gleich.

Bei den Bouquets ist es noch richtig mit + wird nach unten gescrollt doch bei der Bouquetverwaltung scrollt man nun mit - runter oder man ändert es in den Tasteneinstellungen wie du schon sagst dann scrollt man bei den Bouquets mit - nach unten und in der Bouquetverwaltung mit + nach unten.

Komische logik.

[mogway]

So, ich hoffe ich habe nun das durcheinander gerichtet. Falls ich noch etwas vergessen habe, bitte kurz melden.

Gruß
mogway

[racker]

Mit meinen letzten Änderungen war das Verhalten in allen Listen gleich.
Vorrausgesetzt man hat diese auch aktiviert. Bei meinen Tests ist mir auch
ein Segfault im EPGplus aufgefallen, den ich auch behoben habe.

@PauleFoul
Es wäre schön gewesen, wenn du "Bouquetverwaltung zerhauen" näher beschrieben hättest.

So, ich hoffe ich habe nun das durcheinander gerichtet. Falls ich noch etwas vergessen habe, bitte kurz melden.

Gruß
mogway

test the stuff next time plz

Dieser Kommentar ist eine Frechheit zumal man sich hier mit keiner
Meinung geäußert hat.

[mogway]

Mit meinen letzten Änderungen war das Verhalten in allen Listen gleich.
Vorrausgesetzt man hat diese auch aktiviert.

Das ist richtig. Nur wenn man die Option deaktiviert hat, war es noch mehr durcheinander als vorher (das meine PauleFoul bestimmt mit "Bouquetverwaltung zerhauen"). Um hier Irritationen zu verhindern habe ich es gleich richtig eingebaut.

Nun sollte das Verhalten in allen Menus gleich sein. Da ja nun Up/Down & PageUp/PageDown in einem Block gehandelt wird, habe ich den Overhead code mit entsorgt.

test the stuff next time plz

Dieser Kommentar ist eine Frechheit zumal man sich hier mit keiner
Meinung geäußert hat.

Das bezog sich auf das Durcheinander bei deaktivierter Option und war nicht böse gemeint.

.oO( Das die Deutschen immer erst alles negativ sehen/ verstehen...)

Edit: Das Zitat lautet übrigends richtig: to prevent confusion, test the stuff next time plz

Gruß
mogway

[racker]

Das ist richtig. Nur wenn man die Option deaktiviert hat, war es noch mehr durcheinander als vorher (das meine PauleFoul bestimmt mit "Bouquetverwaltung zerhauen").

Auch das Ein- Ausschalten habe ich getestet und es hat sich so verhalten wie es sollte.
Sonst hätte ich es auch nicht in dieser Form committet
Um für die Zukunft zu lernen wäre es nicht schlecht zu wissen was da "durcheinander/zerhauen" gewesen sein soll.

Edit: Das Zitat lautet übrigends richtig: to prevent confusion, test the stuff next time plz

Nach einem commit checke ich auf einem 2.System neu aus und teste die
neuen Funktionen. Du kannst da davor schreiben was du willst, du unterstellst
mir hier den Code nicht zu testen- öffentlich, das ist eine Frechheit!
Manchmal liegt es auch an der Art und Weise wie man etwas macht.

Ich hoffe wir haben beide daraus für die Zukunft etwas gelernt.

PS: Ist zwar OT aber bei keinem sind die Diffs in der Mailingliste so schwer zu lesen wie bei dir - mit was führst du den commit durch?

[mb405]

och menno. ihr dürft euch doch wegen solcher lappalien nicht verkrachen.