[libdazzle] suggestion-entry: fix usage when on-screen-keyboard
- From: Christian Hergert <chergert src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [libdazzle] suggestion-entry: fix usage when on-screen-keyboard
- Date: Mon, 13 Aug 2018 20:10:03 +0000 (UTC)
commit b7587db1d496dcdb647e31e894d62202e1bc43f6
Author: Christian Hergert <chergert redhat com>
Date: Mon Aug 13 13:08:11 2018 -0700
suggestion-entry: fix usage when on-screen-keyboard
This fixes some poor workarounds we had try to allow editing text
automatically. Instead, consumers should just block signals directly rather
than letting us try to guess the right thing.
src/suggestions/dzl-suggestion-entry.c | 16 +---------------
1 file changed, 1 insertion(+), 15 deletions(-)
---
diff --git a/src/suggestions/dzl-suggestion-entry.c b/src/suggestions/dzl-suggestion-entry.c
index cbec0ae..20817d7 100644
--- a/src/suggestions/dzl-suggestion-entry.c
+++ b/src/suggestions/dzl-suggestion-entry.c
@@ -260,19 +260,6 @@ dzl_suggestion_entry_changed (GtkEditable *editable)
g_assert (DZL_IS_SUGGESTION_ENTRY (self));
- /*
- * If we aren't focused, just ignore everything. Additionally, if the text is
- * being changed outside of a key-press-event, we want to ignore completion
- * there too (such as calling set_text() while focused).
- *
- * One such example might be updating an URI in a webbrowser.
- *
- * We should be okay on the a11y/IM front here too, since that should happen
- * via synthesized events.
- */
- if (!priv->in_key_press || !gtk_widget_has_focus (GTK_WIDGET (editable)))
- goto update;
-
g_signal_handler_block (self, priv->changed_handler);
text = dzl_suggestion_entry_buffer_get_typed_text (priv->buffer);
@@ -300,7 +287,6 @@ finish:
g_signal_handler_unblock (self, priv->changed_handler);
-update:
dzl_suggestion_entry_update_attrs (self);
DZL_EXIT;
@@ -379,7 +365,7 @@ dzl_suggestion_entry_notify_selected_cb (DzlSuggestionEntry *self,
g_assert (DZL_IS_SUGGESTION_ENTRY (self));
g_assert (DZL_IS_SUGGESTION_POPOVER (popover));
- if (priv->in_key_press == 0 || priv->in_move_by > 0)
+ if (priv->in_move_by > 0)
{
DzlSuggestion *suggestion = dzl_suggestion_popover_get_selected (priv->popover);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]