diff options
authorzmike <>2014-04-26 01:41:15 -0400
committerMike Blumenkrantz <>2014-04-26 01:45:46 -0400
commitf6cb180eff39c790fb9956722befc85c6d5f7fc6 (patch)
parentd2538d0aaa14657f5e9a308292d4dc1206b13e4d (diff)
entry no longer completely breaks focus by focusing itself on selection signal
I don't know why this was added, but it breaks the ability to set focus on any widgets at all under some circumstances. it also is nonsensical since you should never need to set focus while selecting because: * you are selecting with the mouse button, thereby having previously triggered focus from the mouse click * you are selecting using api, in which case there's a config option to allow retaining selection which is also conceptually broken by this behavior @fix
1 files changed, 0 insertions, 1 deletions
diff --git a/src/lib/elm_entry.c b/src/lib/elm_entry.c
index de1b20f3e..cecf1322c 100644
--- a/src/lib/elm_entry.c
+++ b/src/lib/elm_entry.c
@@ -1853,7 +1853,6 @@ _entry_selection_start_signal_cb(void *data,
1853 const char *txt = elm_entry_selection_get(data); 1853 const char *txt = elm_entry_selection_get(data);
1854 Evas_Object *top; 1854 Evas_Object *top;
1855 1855
1856 if (!elm_object_focus_get(data)) elm_object_focus_set(data, EINA_TRUE);
1857 EINA_LIST_FOREACH(entries, l, entry) 1856 EINA_LIST_FOREACH(entries, l, entry)
1858 { 1857 {
1859 if (entry != data) elm_entry_select_none(entry); 1858 if (entry != data) elm_entry_select_none(entry);