From a61786bd0e3a5508945d57d5aa5f55b68182e0b0 Mon Sep 17 00:00:00 2001 From: Yeongjong Lee Date: Mon, 16 Jul 2018 06:09:29 -0400 Subject: [PATCH] elm_spinner: Do not focus to text button on non-editable mode Summary: text button don't need to be focused if spinner is non-editable mode. ref T6798 Test Plan: 1. elementary_test -to spinner 2. Focus second spinner 3. move focus using key. 4. turn on access mode and test 1-3 again. Reviewers: CHAN Reviewed By: CHAN Subscribers: cedric, #committers, zmike Tags: #efl Maniphest Tasks: T6798 Differential Revision: https://phab.enlightenment.org/D6581 --- src/lib/elementary/elm_spinner.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/lib/elementary/elm_spinner.c b/src/lib/elementary/elm_spinner.c index fc0b4250e9..3c2c08c727 100644 --- a/src/lib/elementary/elm_spinner.c +++ b/src/lib/elementary/elm_spinner.c @@ -1220,6 +1220,7 @@ _elm_spinner_efl_canvas_group_group_add(Eo *obj, Elm_Spinner_Data *priv) priv->text_button = elm_button_add(obj); elm_object_style_set(priv->text_button, "spinner/default"); + elm_widget_can_focus_set(priv->text_button, _elm_config->access_mode); efl_event_callback_add (priv->text_button, EFL_UI_EVENT_CLICKED, _text_button_clicked_cb, obj); @@ -1595,6 +1596,7 @@ EOLIAN static void _elm_spinner_editable_set(Eo *obj EINA_UNUSED, Elm_Spinner_Data *sd, Eina_Bool editable) { sd->editable = editable; + elm_widget_can_focus_set(sd->text_button, editable | _elm_config->access_mode); } EOLIAN static Eina_Bool