efl/legacy/elementary
RAJEEV RANJAN 46acbce93e From: RAJEEV RANJAN <rajeev.r@samsung.com>
Subject: Patch to address some focus and disabling issue in elementary
widgets

We have observed some generic issues in Elementary widgets.

1. Widgets not getting disabled if we call elm_object_disabled_set(ob,
EINA_TRUE) followed by elm_object_style_set(obj, <style_name>);

This works fine if we disable the widget after changing the style(call
to theme_hook happens here).


It happens because we are not sending the edje signal in theme_hook of
the widgets, if the widget is already in disabled state.

I have added this code in theme_hook of elm_button as an example and
we will need to apply the similar stuff in all other widgets if needed.

 
 
 2. The widget gets focused in a particular scenario even though we
call elm_object_focus_allow_set(obj, EINA_FALSE).

The code snippet to explain the scenario which we observed was like
this:

Evas_Object *button = NULL, *icon = NULL;

button = elm_button_add(parent);

elm_object_focus_allow_set(button, EINA_FALSE);

icon = elm_icon_add(button);

elm_icon_file_set(icon, EDJE_PATH, "test.icon");

elm_button_icon_set(button, icon);

 
  
  An icon object does not take focus by default. But still whenever
the focus returned to the parent(layout in this case) after closing
some popup, the button gets focused though we had disabled it
explicitly.
 
 The reason is in this case, when we call elm_icon_add(button), icon
gets added to button as a sub-object and in this call, we set
sd->child_can_focus for button(the parent) as true because till now,
the focus property for icon has not been set to false which by default
is true. 
 
 This was set later in the function elm_icon_add() as follows:
 elm_widget_can_focus_set(obj, EINA_FALSE);
  
  Actually this should be set before adding icon as sub-object of the
parent which in this case is a button.
 
 I have done the required modification in the function elm_icon_add().
  
  Attached to this mail is the patch file for these two issues.The
revision from which I have created this patch file is 53129.
We may need to do similar changes in other widgets if required.



SVN revision: 53137
2010-10-07 07:31:24 +00:00
..
config Focus highlight kinda configurable. More to come when discomfitor feels like implementing it 2010-10-04 16:55:32 +00:00
data event catcher should be the lowest object! 2010-10-06 22:45:46 +00:00
debian Some fixes for debian. 2009-05-23 13:57:54 +00:00
doc elm AUTHORS += nash 2010-09-17 05:38:57 +00:00
m4 Fix common misspellings 2010-09-09 00:35:43 +00:00
src From: RAJEEV RANJAN <rajeev.r@samsung.com> 2010-10-07 07:31:24 +00:00
.gitignore Merge branch 'elmdnd' 2010-09-07 06:56:16 +00:00
AUTHORS Update email address across the board. 2010-09-04 18:32:59 +00:00
COPYING namefix. 2010-08-16 04:44:45 +00:00
ChangeLog elementary becomes its own tree/lib/test binary 2008-09-30 06:58:56 +00:00
INSTALL fix INSTALL files. 2008-10-01 07:37:28 +00:00
Makefile.am also conform to the includes in name-vmaj dir policy 2010-07-11 23:34:49 +00:00
NEWS elementary becomes its own tree/lib/test binary 2008-09-30 06:58:56 +00:00
README right readme. 2008-09-30 08:18:41 +00:00
autogen.sh Stop trying to make the m4 directory. It already exists in the source tree. 2009-08-25 21:38:33 +00:00
configure.ac Adding debug option in elementary. 2010-10-01 19:03:23 +00:00
elementary.pc.in actually ONLy need 1 -I 2010-07-11 23:39:54 +00:00
elementary.spec.in Apply the same fix MEJ did ( http://marc.info/?l=enlightenment-svn&m=127157420508183&w=3 ) to more specs 2010-04-21 20:55:23 +00:00

README

Elementary - a basic widget set that is easy to use based on EFL for mobile
touch-screen devices.