[Patch] softhddevice und Umlaute in Eingabefeldern

mini73 · 5. Dezember 2013

Moin!

Ausgehend von diesem Thread hab ich mich gefragt, warum man eigentlich keine Umlaute in Eingabefeldern mit einer Tastatur eingeben kann. Nach einigem Wühlen in der Klasse cKbdRemote des vdr bin ich dann irgendwie darauf gestoßen, dass die Ausgabeplugins auch ihren Teil dazu beitragen. Das "remote"-Konzept im vdr war mir bis heute noch ziemlich unbekannt...

Jedenfalls hab ich mich auf die Suche nach den Umlauten gemacht. Dabei hab ich festgestellt, dass von X für ein "ä" der Name "adiaeresis" geliefert wird. Das ist natürlich ein Tastenname, mit dem der vdr so gar nichts anfangen kann. Nach etwas weiterem Wühlen (kenne mich mit X überhaupt nicht aus) habe ich festgestellt, dass softhddevice "XLookupString" aufruft, das schon einen String mit dem gedrückten Buchstaben zurückliefert. Dieser wird bloß nirgendwo benutzt...

Da ich nichts vorhandenes kaputt machen wollte, übergebe ich diesen String nun extra an "FeedKeyPress", so dass diese den Buchstaben nutzen kann, wenn alle Stricke reißen. Ich hab aber noch keine Ahnung, in welchem Encoding dieser String ist, ich vermute aber, dass es der eingestellten locale entspricht. Ich wandle ihn nach Utf8, dann mit den vdr-Routinen in etwas, das den internen Tasten entspricht und reiche das Ergebnis einfach mal an den vdr weiter... Und siehe da, plötzlich erscheinen Umlaute in den Eingabefelder.
Buchstaben mit Akzenten hab ich allerdings noch nicht hinbekommen.

Es darf gerne mal jemand mit mehr Wissen über den Patch schauen. Wer möchte, darf es natürlich auch für xine(liboutput) portieren, da scheint es das Problem auch noch zu geben.

Lars.

seahawk1986 · 5. Dezember 2013

Nach vier Jahren mit Software-Frontends für den VDR kann ich endlich die Tastatur sinnvoll zur Texteingabe nutzen

johns · 5. Dezember 2013

XLookupString sollte eigentlich das machen was man will, aber es war irgendein Problem.

Sowie du es gemacht hast, habe ich dann einfach den String an FeedKeyPress übergeben.
XLookupString sollte Buchstaben usw. als String zurückgeben, wenn es einfache Zeichen sind.
Sondertasten (Cursor...) sollten dann leeren String ergeben.

Es kann sein, daß das Leerzeichen ein Problem war. Das Mapping vom Vdr erwartet den Text "Space" und XLookupString gab das Zeichen 'Space' zurück.

Johns

mini73 · 5. Dezember 2013

Ja, XLookupString gibt zwei Dinge zurück, den Namen der Taste und eine String-Repräsentation, die bei Sondertasten leer ist.
Bisher hast du nur den Namen verwendet und das passiert auch immer noch so. Wenn dieser Name in der remote.conf gefunden wird, wird er auf die entsprechende vdr-Taste gemappt. Kommt der Name aber nicht vor (wie z.B. bei "adiaeresis", das ist ja keine vdr-Taste), dann liefert cRemote::Put ein false zurück und nur dann verfüttere ich einfach den String an den vdr als KBDKEY. Und das auch nur, wenn der Utf8-Code kleiner als 256 ist. Intern arbeitet der vdr ja nicht mit Unicode (und wird er auch nie). Ist also nur eine Erweiterung des bisherigen Verhaltens. Die Logik könnte man sicherlich verfeinern, aber das kann ja jemand machen, der noch komischere Zeichen als Umlaute braucht.

Prima wäre es, wenn das noch jemand mit einer anderen locale-Einstellung probieren könnte, ob die Zeichen dann immer noch korrekt ankommen.

Lars.

johns · 5. Dezember 2013

Ich hatte es versucht alles im X11 Teil zulösen, was aber schief ging.
Aber sorum könnte es klappen. Man kann dann Tasten wie Return mappen.

Was passiert wenn ich die Taste a auf Audio gemappt habe? Kannst du dann noch a in Textfeldern eingeben?

(Habe den Patch noch nicht angewandt)

Johns

seahawk1986 · 5. Dezember 2013

Zitat von johns

Was passiert wenn ich die Taste a auf Audio gemappt habe? Kannst du dann noch a in Textfeldern eingeben?

In der remote.conf gemappte Tasten werden nicht mehr für die Texteingabe verwendet (das war davor auch schon so). Ich bin gerade dabei meine Belegung umzustellen, damit die Texteingabe nicht beeinträchtigt wird:
[Blockierte Grafik: https://dl.dropboxusercontent.com/u/960809/yaVDR-Keyboard.png]

Spoiler anzeigen

mini73 · 5. Dezember 2013

Häufig ist ja auch m für Menü gemappt, das konnte man früher auch nie in den Eingabefeldern eingeben. War immer überraschend, wenn dann plötzlich das Menü zuging.

Das ist aber ein grundsätzliches Problem im vdr (z.B. auch auf der Konsole) und nur mit einer anderen Belegung wie der von seahawk zu lösen.

Lars.

johns · 5. Dezember 2013

Okay, dann spielt es keine Rolle wie man die Änderung im Plugin einbaut.

Kann man nicht über cOsd::IsOpen() die Prioritöten ändern?

Wenn im Menu, werden 1 Zeichencodes direkt als Eingabe geschickt.
Und wenn kein Menu so wie im Patch.

Johns

seahawk1986 · 5. Dezember 2013

Zitat von johns

Wenn im Menu, werden 1 Zeichencodes direkt als Eingabe geschickt.
Und wenn kein Menu so wie im Patch.

Klingt eigentlich gut - aber hätte man dann nicht auch unterschiedliche Belegungen je nachdem ob gerade die Kanalinfo oder eine OSD-Message gerade angezeigt wird oder nicht?

mini73 · 5. Dezember 2013

Dann bekommen wir aber doch das "Space" wieder nicht richtig in den vdr, oder?
Schön wäre es, wenn man den vdr fragen könnte, ob gerade ein Eingabefeld im Editiermodus ist, aber das geht dann spätestens nicht mehr, wenn ein Plugin sich ein eigenes Menuitem ausdenkt und implementiert.

Die ganze Menügeschichte ist noch nicht so richtig rund und ich denke, es wird auch nicht einfach, da was Rundes draus zu machen.

Lars.

kls · 5. Dezember 2013

Zitat von mini73

Dann bekommen wir aber doch das "Space" wieder nicht richtig in den vdr, oder?
Schön wäre es, wenn man den vdr fragen könnte, ob gerade ein Eingabefeld im Editiermodus ist, aber das geht dann spätestens nicht mehr, wenn ein Plugin sich ein eigenes Menuitem ausdenkt und implementiert.

Die ganze Menügeschichte ist noch nicht so richtig rund und ich denke, es wird auch nicht einfach, da was Rundes draus zu machen.

Es ist eigentlich ganz einfach: beim Anlernen der Fernbedienung nur die Tasten verwenden, die man nicht in Eingabefeldern benutzen möchte.
Einzige Ausnahme: die Zifferntasten. Diese kann man auch in Eingabefeldern benutzen, wenn man sie beim Anlernen für die Ziffern verwendet hat.

Klaus

mini73 · 5. Dezember 2013

Ja, Menü hab ich schon auf Pos1, Info auf Ende usw.
So ähnlich, wie seahawk es oben gepostet hat.

Lars.

johns · 6. Dezember 2013

Zitat von mini73

Dann bekommen wir aber doch das "Space" wieder nicht richtig in den vdr, oder?
Schön wäre es, wenn man den vdr fragen könnte, ob gerade ein Eingabefeld im Editiermodus ist, aber das geht dann spätestens nicht mehr, wenn ein Plugin sich ein eigenes Menuitem ausdenkt und implementiert.

Kann durchaus sein, werde es einfach mal ausprobieren.

Besser wäre wenn das Eingabemapping vom VDR, "States" kennen würde. Dann bräuchten die Plugins nur den Eingabe "State" setzen und man hätte die Eingabebelegung. Und dann Funktionen, wie lange drücken oder zweimal kurz usw.
Ist aber nicht DAU tauglich.

Johns

FrankJepsen · 6. Dezember 2013

Zitat von seahawk1986

Ich bin gerade dabei meine Belegung umzustellen, damit die Texteingabe nicht beeinträchtigt wird:
[Blockierte Grafik: https://dl.dropboxusercontent.com/u/960809/yaVDR-Keyboard.png]

Die Belegung gefällt mir. Aber ich würde noch die ESC-Taste mit Back belegen. Wenn ich den VDR mit einer Tastatur bediene drücke immer wieder mal auf die ESC-Taste, bis mir wieder einfällt, dass ich beim VDR mit Backspace zurückkomme.

Tschüß Frank

seahawk1986 · 6. Dezember 2013

Zitat von FrankJepsen

Die Belegung gefällt mir. Aber ich würde noch die ESC-Taste mit Back belegen. Wenn ich den VDR mit einer Tastatur bediene drücke immer wieder mal auf die ESC-Taste, bis mir wieder einfällt, dass ich beim VDR mit Backspace zurückkomme.

Das darf ja jeder so handhaben wie er will - mit vdr-sxfe ist ESC halt beispielsweise schon belegt...

Code

XKeySym.Back   Escape

johns · 8. Januar 2014

Der Patch ist nun im GIT, hoffe er funktioniert so wie gewünscht.

wobei ich mich frage ob man nicht immer csoft->Put aufrufen soll ohne die Entscheidung ob der Tastenname 1 Zeichen oder mehrere ist:

Diff

diff --git a/softhddevice.cpp b/softhddevice.cpp
index d5b8f0d..0ce4d24 100644
--- a/softhddevice.cpp
+++ b/softhddevice.cpp
@@ -239,18 +239,14 @@ extern "C" void FeedKeyPress(const char *keymap, const cha
     }
 
     //dsyslog("[softhddev]%s %s, %s, %s\n", __FUNCTION__, keymap, key, letter);
-    if (key[1]) {                      // no single character
-       if (!csoft->Put(key, repeat, release) && letter) {
-           cCharSetConv conv;
-           unsigned code;
-
-           code = Utf8CharGet(conv.Convert(letter));
-           if (code <= 0xFF) {
-               cRemote::Put(KBDKEY(code));     // feed it for edit mode
-           }
+    if (!csoft->Put(key, repeat, release) && letter) {
+       cCharSetConv conv;
+       unsigned code;
+
+       code = Utf8CharGet(conv.Convert(letter));
+       if (code <= 0xFF) {
+           cRemote::Put(KBDKEY(code)); // feed it for edit mode
        }
-    } else if (!csoft->Put(key, repeat, release)) {
-       cRemote::Put(KBDKEY(key[0]));   // feed it for edit mode
     }
 }

Alles anzeigen

Johns

mini73 · 8. Januar 2014

Wenn es nur ein Zeichen ist, dann braucht man die Konvertierung nicht.
Ich meine, ich hab das auch ausprobiert und dann gab es noch ein anderes Problem. Ich erinnere mich nur nicht mehr...

Lars.

johns · 23. Januar 2014

So nun geht im Moment kein Anlernen der Fernbedienung mehr, kann dies mit dieser Änderung zusammen hängen?

Code

vdr: video/vdpau: osd upload 668x241+38+94 1ms 643952
vdr: [3547] [softhddev]FeedKeyPress XKeySym, Up,

Kommt, aber der VDR reagiert nicht auf das Up.

Johns

mini73 · 23. Januar 2014

Probiere ich heute Abend mal aus.

Lars.

mini73 · 23. Januar 2014

War gar nicht so schlimm:

Diff

diff --git a/softhddevice.cpp b/softhddevice.cpp
index d5b8f0d..af7072f 100644
--- a/softhddevice.cpp
+++ b/softhddevice.cpp
@@ -240,7 +240,7 @@ extern "C" void FeedKeyPress(const char *keymap, const char *key, int repeat,


     //dsyslog("[softhddev]%s %s, %s, %s\n", __FUNCTION__, keymap, key, letter);
     if (key[1]) {                      // no single character
-       if (!csoft->Put(key, repeat, release) && letter) {
+       if (!csoft->Put(key, repeat, release) && !cRemote::IsLearning() && letter) {
            cCharSetConv conv;
            unsigned code;

Alles anzeigen

Im Lernmodus kann man sich wohl nicht so richtig auf den Returnwert verlassen.

Lars.

[Patch] softhddevice und Umlaute in Eingabefeldern

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema