Xfce Wiki

Sub domains
 

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revisionPrevious revision
Next revision
Previous revision
dev:garcon [2009/08/17 19:45] 82.217.229.66dev:garcon [2013/04/11 14:57] (current) hptt
Line 4: Line 4:
  
 **Specifications:** [[http://standards.freedesktop.org/menu-spec/latest/|Desktop Menu]], [[http://standards.freedesktop.org/desktop-entry-spec/latest/|Desktop Entry]].\\ **Specifications:** [[http://standards.freedesktop.org/menu-spec/latest/|Desktop Menu]], [[http://standards.freedesktop.org/desktop-entry-spec/latest/|Desktop Entry]].\\
-**Sources:** [[http://git.xfce.org/libs/garcon/|Garcon]], [[http://git.gnome.org/cgit/alacarte/|Alacarte]].+**Sources:** [[http://git.xfce.org/xfce/garcon/|Garcon]], [[http://git.gnome.org/cgit/alacarte/|Alacarte]].
  
 ===== File Monitoring ===== ===== File Monitoring =====
  
-We'd like the give the library build-in support for monitoring menu changes. This means we monitor the .menu file(s) for changes (for example made by Alacarte) and the .desktop file directories for added/removed/changed desktop files. When something changed we don't want to rebuild everything, but rather update and trigger some signals and notify properties.+We'd like the give the library built-in support for monitoring menu changes. This means we monitor the .menu file(s) for changes (for example made by Alacarte) and the .desktop file directories for added/removed/changed desktop files. When something changed we don't want to rebuild everything, but rather update and trigger some signals and notify properties.
  
  
Line 43: Line 43:
 This is where we will do all the monitoring work. We will setup monitors for the menu file itself, as well for the merged files and the desktop entry directories. The object will probably get some signals when an item is added/removed/moved, but we need to do this in such a way it is efficient to update menu or tree implementations. This is where we will do all the monitoring work. We will setup monitors for the menu file itself, as well for the merged files and the desktop entry directories. The object will probably get some signals when an item is added/removed/moved, but we need to do this in such a way it is efficient to update menu or tree implementations.
  
-===== Misc  ===== +===== TODO ===== 
- +  * <del>Fix handling of the Hidden key in .desktop files</del>. 
-A couple of tasks that need to be done: +    * <del>Right now we return NULL when loading a desktop files with Hidden=True, this is wrong, we should instead return a GarconMenuItem, add _{get.set}_hidden functions and property. There is a possibility the hidden attribute changes during runtime, then the menu item should be visible again</del>.
   * Always return a reffed object in the _get_ functions and mention this in the API docs.   * Always return a reffed object in the _get_ functions and mention this in the API docs.
 +    * Because we always ref and because of the implementation of the singletons: we also return reffed items in the GList's. This means that developers should also call ''g_list_foreach (list, (GFunc) g_object_unref, NULL);'' or unref in their own loop. This is not a bad thing tho, because of the file monitoring it could be that the menu implementation releases an item (because the .desktop file was destroyed), so apps _should_ have their own references.
 +    * We could add a helper: ''#define garcon_free_list(l) G_STMT_START { g_list_foreach ((l), (GFunc) g_object_unref, NULL); g_list_free (l); } G_STMT_END''
   * A couple of structs need to move to the header.   * A couple of structs need to move to the header.
 +    * We need to reserve space if we do this.
   * Personally I find the gio helpers a bit ugly in the public api, maybe we can make this private (preferred) or rename them to start with garcon_.   * Personally I find the gio helpers a bit ugly in the public api, maybe we can make this private (preferred) or rename them to start with garcon_.
   * <del>Possibly drop ''garcon_init()'' and ''garcon_shutdown()''. Still needs some thoughts, but it would be nice if we can simplify this as much as possible</del>.   * <del>Possibly drop ''garcon_init()'' and ''garcon_shutdown()''. Still needs some thoughts, but it would be nice if we can simplify this as much as possible</del>.
Line 57: Line 59:
   * <del>GIO file loading in ''GarconMenuDirectory''</del>.   * <del>GIO file loading in ''GarconMenuDirectory''</del>.
   * <del>Get rid of the ''g_strv_length()'' calls in loops</del>.   * <del>Get rid of the ''g_strv_length()'' calls in loops</del>.
-  * Use the G_KEY_FILE_DESKTOP_* defines where possible.+  * <del>Use the G_KEY_FILE_DESKTOP_* defines where possible</del>.
  
-Possible things we want to implement:+===== Future=====
  
   * We currently support desktop files with the type Application and Directory, but not Link.   * We currently support desktop files with the type Application and Directory, but not Link.