From efbd62c40af11fee12474de1adabf1d46a187d67 Mon Sep 17 00:00:00 2001 From: Shilpa Singh Date: Wed, 14 Jun 2017 13:04:51 +0900 Subject: [PATCH] elm_radio, elc_multibuttonentry, elm_colorselector: "checked" signal sent Summary: Radio, MBE items, color selector items have ELM_ATSPI_ROLE_RADIO_BUTTON role, when one item is selected the other items are automatically unselected, so a notification has to be sent in atspi mode to read the status accordingly Test Plan: In ATSPI mode when radio, mbe item, color selector item is selected status has to notified. Reviewers: kimcinoo, cedric Subscribers: govi, rajeshps, jpeg Differential Revision: https://phab.enlightenment.org/D4960 --- src/lib/elementary/efl_ui_radio.c | 2 -- src/lib/elementary/elc_multibuttonentry.c | 5 +++++ src/lib/elementary/elm_colorselector.c | 7 ++++--- 3 files changed, 9 insertions(+), 5 deletions(-) diff --git a/src/lib/elementary/efl_ui_radio.c b/src/lib/elementary/efl_ui_radio.c index 14ddc9b..17d0cea 100644 --- a/src/lib/elementary/efl_ui_radio.c +++ b/src/lib/elementary/efl_ui_radio.c @@ -72,8 +72,6 @@ _state_set(Evas_Object *obj, Eina_Bool state, Eina_Bool activate) { elm_interface_atspi_accessible_state_changed_signal_emit(obj, ELM_ATSPI_STATE_CHECKED, EINA_TRUE); } - else - elm_interface_atspi_accessible_state_changed_signal_emit(obj, ELM_ATSPI_STATE_CHECKED, EINA_FALSE); } } } diff --git a/src/lib/elementary/elc_multibuttonentry.c b/src/lib/elementary/elc_multibuttonentry.c index 4c8fa38..04c6ea6 100644 --- a/src/lib/elementary/elc_multibuttonentry.c +++ b/src/lib/elementary/elc_multibuttonentry.c @@ -516,6 +516,11 @@ _on_item_clicked(void *data, _item_select(WIDGET(it), it); + if (_elm_config->atspi_mode) + elm_interface_atspi_accessible_state_changed_signal_emit(eo_it, + ELM_ATSPI_STATE_CHECKED, + EINA_TRUE); + if (sd->selected_it) efl_event_callback_legacy_call (WIDGET(it), ELM_MULTIBUTTONENTRY_EVENT_ITEM_CLICKED, eo_it); diff --git a/src/lib/elementary/elm_colorselector.c b/src/lib/elementary/elm_colorselector.c index 10a3c52..a11e08d 100644 --- a/src/lib/elementary/elm_colorselector.c +++ b/src/lib/elementary/elm_colorselector.c @@ -1711,9 +1711,10 @@ _item_action_activate(Evas_Object *obj, const char *params EINA_UNUSED) ELM_COLORSELECTOR_DATA_GET(WIDGET(item), sd); elm_object_signal_emit(VIEW(item), "elm,state,selected", "elm"); - elm_interface_atspi_accessible_state_changed_signal_emit(obj, - ELM_ATSPI_STATE_CHECKED, - EINA_TRUE); + if (_elm_config->atspi_mode) + elm_interface_atspi_accessible_state_changed_signal_emit(obj, + ELM_ATSPI_STATE_CHECKED, + EINA_TRUE); elm_colorselector_color_set(WIDGET(item), item->color->r, item->color->g, item->color->b, item->color->a); evas_object_smart_callback_call(WIDGET(item), SIG_COLOR_ITEM_SELECTED, -- 2.7.4