Bug 413076 - add bookmarks file support to the file chooser
add bookmarks file support to the file chooser
Status: RESOLVED OBSOLETE
Product: gtk+
Classification: Platform
Component: Widget: GtkFileChooser
unspecified
Other Linux
: Normal enhancement
: ---
Assigned To: gtk-bugs
Federico Mena Quintero
:
Depends on:
Blocks:
  Show dependency tree
 
Reported: 2007-02-28 14:48 UTC by Emmanuele Bassi (:ebassi)
Modified: 2013-05-29 11:18 UTC (History)
1 user (show)

See Also:
GNOME target: ---
GNOME version: ---


Attachments
initial, proof of concept patch (34.21 KB, patch)
2007-02-28 14:49 UTC, Emmanuele Bassi (:ebassi)
needs-work Details | Diff | Review

Description Emmanuele Bassi (:ebassi) 2007-02-28 14:48:57 UTC
subject says all.  the following patch:

 * ports the gtk-bookmarks file format to GBookmarkFile, while importing the old
   semi-private format (without synchronisation: it can be added if it's deemed
   important);
 * adds a function for adding per-application shortcut folders using a single
   bookmark file:

     void
     gtk_file_chooser_add_shortcuts_from_file (GtkFileChooser *chooser,
                                               const gchar    *filename,
                                               const gchar    *translation_domain)

   the translation domain is used to localise the shortcuts, so that we can patch
   intltool to extract the bookmark title and merge it into the application .pot
   file for translators to localise;
 * adds internal API for adding a shortcut folder with a label; the public API
   has not been changed.

this patch is still a proof-of-concept: it should just show how to add per-application shortcuts from files.

future improvements: adding default shortcuts from a default location ($XDG_DATA_DIRS/places/*.xbel), using the application name/group as a filter, possibly with a mmap()able cache file which gets created using a small program like gtk-update-icon-cache.
Comment 1 Emmanuele Bassi (:ebassi) 2007-02-28 14:49:39 UTC
Created attachment 83546 [details] [review]
initial, proof of concept patch
Comment 2 Emmanuele Bassi (:ebassi) 2013-05-29 11:18:19 UTC
I think this can be closed as obsolete.

Note You need to log in before you can comment on or make changes to this bug.