From: Shinwoo Kim Date: Wed, 1 Nov 2017 07:26:43 +0000 (+0900) Subject: atspi: use is_first_rotary_control only if screen-read is enabled X-Git-Tag: submit/tizen/20171107.093719~2^2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7412545366767e0bed6fe8574830d6332b51b9e3;p=platform%2Fcore%2Fuifw%2Fefl-ext.git atspi: use is_first_rotary_control only if screen-read is enabled The is_first_rotary_cotrol is used for ignoring the first bezel event, if the screen-reader is enabled, because the first bezel event should give highlight to the first item of rotary_selector. When the screen-reader is disabled, the first bezel event should select the second item of rotary_selector. The is_first_rotary_control logic worked even though the screen-reader is disabled. So the first bezel event did NOT select the second item of rotary_selector. Change-Id: I00c6fdcdc48b9e78cdb30af247b5c70f80b18c92 --- diff --git a/src/wearable/efl_extension_rotary_selector.c b/src/wearable/efl_extension_rotary_selector.c index b21b0b0..42fce93 100644 --- a/src/wearable/efl_extension_rotary_selector.c +++ b/src/wearable/efl_extension_rotary_selector.c @@ -1406,7 +1406,7 @@ _rotary_selector_rotary_cb(void *data, Evas_Object *obj, Eext_Rotary_Event_Info { if (!rsd->acceleration) { - if (rsd->is_first_rotary_control) + if (elm_atspi_bridge_utils_is_screen_reader_enabled() && rsd->is_first_rotary_control) { rsd->is_first_rotary_control = EINA_FALSE; rsd->selected_index = 0; @@ -2827,9 +2827,9 @@ eext_rotary_selector_add(Evas_Object *parent) { evas_object_smart_callback_add(rotary_selector, "more,option,opened", _rotary_selector_moreoption_opened_cb, rsd); evas_object_smart_callback_add(rotary_selector, "more,option,closed", _rotary_selector_moreoption_closed_cb, rsd); + _rotary_selector_from_moreoption_set(rotary_selector, EINA_FALSE); + rsd->is_first_rotary_control = EINA_TRUE; } - _rotary_selector_from_moreoption_set(rotary_selector, EINA_FALSE); - rsd->is_first_rotary_control = EINA_TRUE; // return rotary_selector; }