spinner: Fix entry not inactive even gets an enter key cb.

Summary:
 The entry inactive signal emitted when finish spinner value set.
but the active signal emitted after that. so it makes weird view state.

 The function 'key_action_toggle()' called twice. (for entry, spinner)
event propagate called this func twice and its make this issue.

This patch makes to ignore toggle event when entry is not visible.
T2262

Test Plan:
Run elementary_test, execute spinner widget sample.

Click the first spinner.
Input something on entry.
Press enter key.
Check the issue.

Reviewers: raster, Hermet

Subscribers: kuuko

Differential Revision: https://phab.enlightenment.org/D2411

Change-Id: I0caa58a4751938319d0a0d8739dc9f5edd4b2deb
Origin: upstream

diff --git a/src/lib/elm_spinner.c b/src/lib/elm_spinner.c
index 413a092..64c6a8c 100644 (file)
--- a/src/lib/elm_spinner.c
+++ b/src/lib/elm_spinner.c
@@ -457,7 +457,7 @@ _key_action_toggle(Evas_Object *obj, const char *params EINA_UNUSED)
 {
    ELM_SPINNER_DATA_GET(obj, sd);
    if (sd->spin_timer) _spin_stop(obj);
-   else _entry_toggle_cb(NULL, obj, NULL, NULL);
+   else if (sd->entry_visible) _entry_toggle_cb(NULL, obj, NULL, NULL);
    return EINA_FALSE;
 }