Love tasks for desktop lovers

[Alain Perry]

>  * I think you should also use the environment variable in
>    shortcuts_append_documents()

Of course, silly me. Done.

>  * I don't remember where Documents is placed in the Places menu of the
>    panel, but I think it should be at the same place. If it's the first
>    item, then great. Else, you'll need to change the position in your
>    patch.

Done.

>  * "const gchar* envvar;" => const gchar *envvar;

Done.

> Overall, the patch looks okay.
>
> A next step would be to patch gnome-panel to make it use the environment
> variable too (note that the Documents item is added by an Ubuntu patch),
> and also to patch nautilus to have this Documents item where needed (in
> the shortcuts sidebar, eg).

Well, your remarks have made me see some other things that needs to be
done for this patch to be complete:

* The Desktop entry in the left pane is localized, but it is not
localized in the pathbar at the top. Though they have the same icon,
we lack consistency here.

* The "home" entry uses the user name in the filechooser, but uses
"Personal folder" in the gnome-panel menu, so the same remark is also
true here. However, I'm wondering if it is a good idea to use
"Personal folder" in the pathbar, since it is quite long. Sebastien,
Vincent, any thoughts on this ?

* The Documents entry should definitely have it's own icon too (in the
shortcuts pane as well as in the pathbar) and be named "Documents" in
the pathbar for consistency as well. However, for the first of those
two problems, I'd need an artist, because believe me, you don't want
me to draw an icon.

So the big question is: should I use my time trying to solve all these
issues, or do we not care ?

Anyway, just in case Sebastien wants the patch in its current state (I
guess localization of the pathbar would be a different patch anyway),
I'm attaching it.

Thanks for your help,

--
Alain Perry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 001_fs_documents.patch
Type: text/x-patch
Size: 5565 bytes
Desc: not available
Url : http://lists.ubuntu.com/archives/ubuntu-desktop/attachments/20051229/b10c035d/001_fs_documents.bin