[gnome-shell/wip/carlosg/osk-updates: 11/26] keyboard: Unify OSK key commit handling in single-state signal

[Carlos Garnacho <carlosg src gnome org>]

commit 8e25e1170df8fc5aee8a1353e90c77d794abd9bf
Author: Carlos Garnacho <carlosg gnome org>
Date:   Thu Apr 14 20:42:30 2022 +0200

    keyboard: Unify OSK key commit handling in single-state signal
    
    Instead of having callers handle pressed+released, emit string
    commits on a distinct signal that is emitted all at once during
    release.
    
    This also unifies the behavior of keys that have an extended keys
    popup and not wrt press/release behavior and key repeat.

 js/ui/keyboard.js | 37 +++++++++++++------------------------
 1 file changed, 13 insertions(+), 24 deletions(-)
---
diff --git a/js/ui/keyboard.js b/js/ui/keyboard.js
index 142f2574c3..b91efc4f3d 100644
--- a/js/ui/keyboard.js
+++ b/js/ui/keyboard.js
@@ -292,8 +292,9 @@ var Key = GObject.registerClass({
     Signals: {
         'activated': {},
         'long-press': {},
-        'pressed': { param_types: [GObject.TYPE_UINT, GObject.TYPE_STRING] },
-        'released': { param_types: [GObject.TYPE_UINT, GObject.TYPE_STRING] },
+        'pressed': {},
+        'released': {},
+        'commit': { param_types: [GObject.TYPE_UINT, GObject.TYPE_STRING] },
     },
 }, class Key extends St.BoxLayout {
     _init(params, extendedKeys) {
@@ -355,10 +356,7 @@ var Key = GObject.registerClass({
     _press(button, commitString) {
         this.emit('activated');
 
-        let keyval;
-
         if (button == this.keyButton) {
-            keyval = this._keyval;
             this._pressTimeoutId = GLib.timeout_add(GLib.PRIORITY_DEFAULT,
                 KEY_LONG_PRESS_TIME,
                 () => {
@@ -378,11 +376,8 @@ var Key = GObject.registerClass({
                 });
         }
 
-        if (!keyval)
-            keyval = this._getKeyval(commitString);
-
-        if (this._extendedKeys.length === 0)
-            this.emit('pressed', keyval, commitString);
+        this.emit('pressed');
+        this._pressed = true;
     }
 
     _release(button, commitString) {
@@ -397,11 +392,12 @@ var Key = GObject.registerClass({
         if (!keyval && commitString)
             keyval = this._getKeyval(commitString);
 
-        if (this._extendedKeys.length > 0)
-            this.emit('pressed', keyval, commitString);
+        if (this._pressed && (commitString || keyval))
+            this.emit('commit', keyval, commitString || "");
 
-        this.emit('released', keyval, commitString);
+        this.emit('released');
         this._hideSubkeys();
+        this._pressed = false;
     }
 
     cancel() {
@@ -923,7 +919,7 @@ var EmojiPager = GObject.registerClass({
             key.connect('long-press', () => {
                 this._panAction.cancel();
             });
-            key.connect('released', (actor, keyval, str) => {
+            key.connect('commit', (actor, keyval, str) => {
                 if (this._currentKey != key)
                     return;
                 this._currentKey = null;
@@ -1542,20 +1538,13 @@ var Keyboard = GObject.registerClass({
             if (key.width !== null)
                 button.setWidth(key.width);
 
-            button.connect('pressed', (actor, keyval, str) => {
-                if (!Main.inputMethod.currentFocus ||
+            button.connect('commit', (actor, keyval, str) => {
+                if (str === '' || !Main.inputMethod.currentFocus ||
                     !this._keyboardController.commitString(str, true)) {
                     if (keyval != 0) {
                         this._keyboardController.keyvalPress(keyval);
-                        button._keyvalPress = true;
-                    }
-                }
-            });
-            button.connect('released', (actor, keyval, _str) => {
-                if (keyval != 0) {
-                    if (button._keyvalPress)
                         this._keyboardController.keyvalRelease(keyval);
-                    button._keyvalPress = false;
+                    }
                 }
 
                 if (!this._latched)