On Monday 19 December 2005 13:56, Andrew Williams wrote:
> Well, after further examination of the code engage now uses the e17
> theme if it is preset, otherwise falls back to the installed module
> theme.
>
> basically
>
> if (!e_theme_edje_object_set(obj, category, group)
>   edje_object_file_set(obj, file, group);
>
> which works fine I think.
> but is "theme/modules/engage" the right theme category to be using?
>
> A

All the modules in the apps/e tree use "theme/modules/name_of_module". 
e_theme_object_file_set() will first look to see if a file is set for the 
full category, and then walk up the heirarchy (checking "theme/modules" and 
then "theme"). so, to avoid having to do your fallback to 
edje_object_file_set(), we just need to decide on a convention for 3rd party 
module theme file names / location, and have the module register the file 
with the full category (unless the user has registered a custom one).
This should probably be handled automatically by e_gadget.
--
rephorm


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to