[Patch] Menüelemente mit der Rechts-Taste auswählen

[dbt]

Ja ich weiß. ich bin grad dabei einiges zu fixen, das schau ich mir auch noch an.

[mb405]

klasse. du bist mein held

[GetAway]

Schau ich mir an. Obwohl, bei Schriftgrößenänderungen gibt es viele Probleme, das ist ein Fass ohne Boden.

[Gaucho316]

Nach dem Schließen des Menüs bleiben übrigens Reste der Icons auf dem Bildschirm übrig.
[...]
Könnte es sein, dass folgende einfache Änderung in der Zeile 106 der menue.cpp hilft? Ich kann's leider erst heute Abend ausprobieren.

Alt:

Code: Alles auswählen

if (selected)

Neu:

Code: Alles auswählen

if (selected && offx > 0)

Wie sieht's aus? Ist das nichts fürs CVS? Die Menüs kann man ja trotzdem weiter anpassen und die noch nicht veränderten Menüs sehen wieder gut aus.

[dbt]

Wie sieht's aus? Ist das nichts fürs CVS? Die Menüs kann man ja trotzdem weiter anpassen und die noch nicht veränderten Menüs sehen wieder gut aus.

Hatte das bewusst noch nicht eingecheckt, damit man wirklich drüber stolpert und es auffällt wo es ein Problem ist...

[dbt]

Ja, das muss noch so umgebogen werden, dass man das, was zu viel ist, ausblendet, aber heute nicht mehr, falls du schon was hast, wie immer Patch ablegen, dann schieb ich das nach...

[Gaucho316]

Folgende Idee habe ich eben gehabt. Es gibt in "menue.cpp" ja das Objekt "GenericMenuBack". Wie wäre es denn auch "GenericMenuCancel" anzulegen? So etwas wird ja auch an mehreren Stellen gebraucht.

[Gaucho316]

Ich habe einen Fehler entdeckt. Durch die Änderungen an "audio_select.cpp" vom 20.06.2010 ist im Audioauswahlmenü die aktuelle Audiospur nicht mehr vorausgewählt. Oder ist das nur bei mir so?

[dbt]

Eigentlich beabsichtigt. Nur die erste Option ist Standard zur Auswahl beim öffnen.

[Gaucho316]

Finde ich nicht so toll. Woher weiß ich nun, welche Audiospur ich gewählt habe?

[dbt]

Ok, nachbessern...

[Gaucho316]

Der folgende Patch sorgt nun dafür, dass im Audioauswahlmenü die aktuelle Audiospur vorausgewählt ist.

Link entfernt, da Patch im CVS

Ich frage mich allerdings, warum der aktuelle CVS-Code überhaupt kompiliert. Die Zeile 101 in "audio_select.cpp" verstehe ich nicht. Warum folgt nach der schließenden Klammer von new CMenuForwarderNonLocalized noch ein Komma und (count == g_RemoteControl->current_PIDs.PIDs.selected_apid)?

Code: Alles auswählen

fw[count] = new CMenuForwarderNonLocalized(g_RemoteControl->current_PIDs.APIDs[count].desc, true, NULL, APIDChanger, apid, CRCInput::convertDigitToKey(count + 1)), (count == g_RemoteControl->current_PIDs.PIDs.selected_apid);

[seife]

Der Compiler bemeckert das ja auch brav:

Code: Alles auswählen

audio_select.cpp: In member function `int CAudioSelectMenuHandler::doMenu()':
audio_select.cpp:101: warning: right-hand operand of comma has no effect

Aber Compilerwarnings sind ja nicht dazu da, dass sie sich jemand ansieht...

Das Scansetup war auch ziemlich kaputt, da hatte sich allerdings nicht der Complier beschwert, sondern nur valgrind (und es standen unmögliche Sachen im menuforwarder )

[dbt]

Danke für die Nachlese. Audioselect ist auch korrigiert... und wenn hier mal Valgrind laufen würde, wär' das auch auch mal was.

[GetAway]

Das Scansetup war auch ziemlich kaputt, da hatte sich allerdings nicht der Complier beschwert, sondern nur valgrind (und es standen unmögliche Sachen im menuforwarder )

Danke fürs fixen, 2 Sachen habe ich dazu noch.
Für Kabelnutzer bitte von Alle Sats auf Komplett ändern.

Durch den Checkin ist wohl folgendes kaputt gegangen:
Beim vergeben eines Namens für Benutzermenü, wird der String nicht mehr sofort übernommen
und die Anzeige bleibt so wie voirher. Gleiches gilt fürs löschen des Namens.

[seife]

Durch den Checkin ist wohl folgendes kaputt gegangen:
Beim vergeben eines Namens für Benutzermenü, wird der String nicht mehr sofort übernommen
und die Anzeige bleibt so wie voirher. Gleiches gilt fürs löschen des Namens.

Ich habe keine Ahnung, was das Benutzermenü ist oder wie man dem einen Namen gibt. Beschreib das mal bitte.

edit: ok, habs gefunden.

[seife]

Das hat immer nur "aus versehen" funktioniert - genauso wie das, was ich gestern gefixt habe. Ich hab grad was in der Mache, was das richtig fixt.

Edit: sollte jetzt gehn.

[seife]

Danke für die Nachlese. Audioselect ist auch korrigiert... und wenn hier mal Valgrind laufen würde, wär' das auch auch mal was.

Über den Audioselect hatte Valgrind nichts zu meckern.

[dbt]

Meinte das auch mehr allgemein, weils doch hilfreich wäre.

[seife]

Ich hab mir das mal nachdem meine Kundschaft heute recht verwirrt war nochmal angeschaut.

Beispiel: Tuxcom ist auf "blau"-"blau".

Drücke ich jetzt einmal blau, dann sehe ich nicht mehr, was der shortcut für Tuxcom ist, weil ja da das "Pfeil nach Rechts"-Icon erscheint.

Das erscheint mir nicht wirklich intuitiv.

Dass man Menüelemente mit "rechts" und mit "OK" auswählen kann, das kapieren zumindest meine Nutzer auch so, aber dass Tuxcom mit "blau"-"blau" gestartet wird, das haben sie erst nach rumprobieren rausgefunden

Ich habe das für mich jetzt so gelöst:

Code: Alles auswählen

--- a/tuxbox/neutrino/src/gui/widget/menue.cpp
+++ b/tuxbox/neutrino/src/gui/widget/menue.cpp
@@ -122,7 +122,7 @@ void CMenuItem::paintItemButton(const int startX, const int frame_height, const
                }
                else
                {
-               if ((CRCInput::isNumeric(directKey)) || (directKey >= CRCInput::RC_red && directKey <= CRCInput::RC_blue) || icon_name.empty())
+               if (icon_name.empty())
                        icon_name = icon_Name;
                }
        }
--- a/tuxbox/neutrino/src/gui/audio_select.cpp
+++ b/tuxbox/neutrino/src/gui/audio_select.cpp
@@ -99,7 +99,7 @@ int CAudioSelectMenuHandler::doMenu ()
                        char apid[5];
                        sprintf(apid, "%d", count);
                        fw[count] = new CMenuForwarderNonLocalized(g_RemoteControl->current_PIDs.APIDs[count].desc, true, NULL, APIDChanger, apid, CRCInput::convertDigitToKey(count + 1));
-                       fw[count]->setItemButton(NEUTRINO_ICON_BUTTON_OKAY, true);
+                       // fw[count]->setItemButton(NEUTRINO_ICON_BUTTON_OKAY, true);
                        AudioSelector.addItem(fw[count], (count == g_RemoteControl->current_PIDs.PIDs.selected_apid));
                }
        }

[rhabarber1848]

Drücke ich jetzt einmal blau, dann sehe ich nicht mehr, was der shortcut für Tuxcom ist, weil ja da das "Pfeil nach Rechts"-Icon erscheint.

Das erscheint mir nicht wirklich intuitiv.

+1. Ich teste den Patch in den nächsten Tagen.

[dbt]

anderer Vorschlag: sinngemäß einen transparenten Rightbutton nehmen damit der Hintergrund erhalten bleibt, dann sollte das erkennbar bleiben, der Pfeil sollte reichen

[rhabarber1848]

anderer Vorschlag: sinngemäß einen transparenten Rightbutton

Auch eine gute Idee. Dann wäre also ein blauer Knopf zu sehen, wo der Pfeil transparent drüber gelegt ist, richtig?

[dbt]

Ich probier mal einige Möglichkeiten aus, mal schauen, mal sehen was am besten aussieht.

[GetAway]

Man könnte in den Menü's auch 2-spaltig fahren. Pfeil und Ok weiter nach links verlagern,
so das die Col-Buttons und Ziffern gar nicht bedeckt werden. Dazu müßte die Spalte nur
etwas breiter werden. Ein anderer Vorschlag wäre alles nach rechts zu verlagern.