layout: Move edje_object_can_access to legacy
authorJean-Philippe Andre <jp.andre@samsung.com>
Wed, 2 Aug 2017 06:17:54 +0000 (15:17 +0900)
committerJean-Philippe Andre <jp.andre@samsung.com>
Fri, 4 Aug 2017 02:52:16 +0000 (11:52 +0900)
This is an API enabling accessibility on text(block) parts
in a layout. But it is said to have many issues. I can already
see that it only changes a flag but doesn't trigger any code
to create the appropriate objects, so definitely not fully
working.

According to @kimcinoo this may remain in legacy land for now.

src/lib/elementary/elm_layout.c
src/lib/elementary/elm_layout.eo
src/lib/elementary/elm_layout_legacy.h

index c4a3356..66922ce 100644 (file)
@@ -1870,16 +1870,20 @@ _elm_layout_part_cursor_engine_only_get(const Eo *obj EINA_UNUSED, Elm_Layout_Sm
    return !elm_object_cursor_theme_search_enabled_get(pc->obj);
 }
 
-EOLIAN static Eina_Bool
-_elm_layout_edje_object_can_access_set(Eo *obj EINA_UNUSED, Elm_Layout_Smart_Data *sd, Eina_Bool can_access)
+EAPI Eina_Bool
+elm_layout_edje_object_can_access_set(Eo *obj, Eina_Bool can_access)
 {
+   Elm_Layout_Smart_Data *sd = efl_data_scope_safe_get(obj, MY_CLASS);
+   EINA_SAFETY_ON_NULL_RETURN_VAL(sd, EINA_FALSE);
    sd->can_access = !!can_access;
    return EINA_TRUE;
 }
 
-EOLIAN static Eina_Bool
-_elm_layout_edje_object_can_access_get(Eo *obj EINA_UNUSED, Elm_Layout_Smart_Data *sd)
+EAPI Eina_Bool
+elm_layout_edje_object_can_access_get(const Eo *obj)
 {
+   Elm_Layout_Smart_Data *sd = efl_data_scope_safe_get(obj, MY_CLASS);
+   EINA_SAFETY_ON_NULL_RETURN_VAL(sd, EINA_FALSE);
    return sd->can_access;
 }
 
index b03518d..0e11461 100644 (file)
@@ -26,29 +26,6 @@ class Elm.Layout (Elm.Widget, Efl.Part, Efl.Container, Efl.File,
    event_prefix: elm_layout;
    data: Elm_Layout_Smart_Data;
    methods {
-      @property edje_object_can_access {
-         set {
-            [[Set accessibility to all texblock(text) parts in the layout
-              object.
-
-              @since 1.7
-            ]]
-            return: bool; [[$true on success or $false on failure. If $obj
-                            is not a proper layout object, $false is returned.]]
-         }
-         get {
-            [[Get accessibility state of texblock(text) parts in the layout
-              object
-
-              @since 1.7
-            ]]
-         }
-         values {
-            can_access: bool; [[Makes all textblock(text) parts in the layout
-                                $obj possible to have accessibility. $true
-                                means textblock(text) parts can be accessible.]]
-         }
-      }
       @property theme {
          set {
             [[Set the edje group from the elementary theme that will be used
index dc245f8..24228ed 100644 (file)
@@ -391,4 +391,32 @@ EAPI Eina_Bool elm_layout_text_set(Evas_Object *obj, const char * part, const ch
  */
 EAPI const char *elm_layout_text_get(const Evas_Object *obj, const char * part);
 
+/**
+ * @brief Set accessibility to all texblock(text) parts in the layout object.
+ *
+ * @param[in] can_access Makes all textblock(text) parts in the layout @c obj
+ * possible to have accessibility. @c true means textblock(text) parts can be
+ * accessible.
+ *
+ * @return @c true on success or @c false on failure. If @c obj is not a proper
+ * layout object, @c false is returned.
+ *
+ * @since 1.7
+ *
+ * @ingroup Elm_Layout
+ */
+EAPI Eina_Bool elm_layout_edje_object_can_access_set(Evas_Object *obj, Eina_Bool can_access);
+
+/**
+ * @brief Get accessibility state of texblock(text) parts in the layout object
+ *
+ * @return Makes all textblock(text) parts in the layout @c obj possible to
+ * have accessibility. @c true means textblock(text) parts can be accessible.
+ *
+ * @since 1.7
+ *
+ * @ingroup Elm_Layout
+ */
+EAPI Eina_Bool elm_layout_edje_object_can_access_get(const Evas_Object *obj);
+
 #include "elm_layout.eo.legacy.h"