[gnome-shell/wip/carlosg/im-full-reset: 5/5] inputMethod: Manually reset input context state before unfocus
- From: Carlos Garnacho <carlosg src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/wip/carlosg/im-full-reset: 5/5] inputMethod: Manually reset input context state before unfocus
- Date: Mon, 17 Oct 2022 14:20:02 +0000 (UTC)
commit f2f2ed8447ccb664a916f6850afadb85cf27521e
Author: Carlos Garnacho <carlosg gnome org>
Date: Mon Oct 17 13:34:21 2022 +0200
inputMethod: Manually reset input context state before unfocus
The IBusInputContext currently seems to be a bit sloppy wrt propagating
the current state to engines after those are changed, this results in
engines possibly ending with purpose/hint/etc that do not actually
correspond to the current state.
Ensure this state is cleared on unfocus of our IBusInputContext, so
that the next time the input method receives an input focus, the new
state makes it all the way to the currently active engine.
Fixes situations in the GDM/lock screen password entry that IBus
would miss the PASSWORD hint set on the StEntry, and let passwords
be fed to engines.
Closes: https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/5966
js/misc/inputMethod.js | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
---
diff --git a/js/misc/inputMethod.js b/js/misc/inputMethod.js
index ed680ec045..422c144618 100644
--- a/js/misc/inputMethod.js
+++ b/js/misc/inputMethod.js
@@ -185,8 +185,10 @@ var InputMethod = GObject.registerClass({
vfunc_focus_out() {
this._currentFocus = null;
- if (this._context)
+ if (this._context) {
+ this._fullReset();
this._context.focus_out();
+ }
if (this._preeditStr && this._preeditVisible) {
// Unset any preedit text
@@ -344,6 +346,13 @@ var InputMethod = GObject.registerClass({
keyval, 0, IBus.ModifierType.RELEASE_MASK, -1, null, null);
}
+ _fullReset() {
+ this._context.set_content_type(0, 0);
+ this._context.set_cursor_location(0, 0, 0, 0);
+ this._context.set_capabilities(0);
+ this._context.reset();
+ }
+
update() {
if (!this._context)
return;
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]