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

diff --git a/src/lib/elementary/efl_ui_radio.c b/src/lib/elementary/efl_ui_radio.c
index 14ddc9b..17d0cea 100644 (file)
--- 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 (file)
--- 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 (file)
--- 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,