Re: [Lubuntu-desktop] lxshortcut integration in pcmanfm PATCH
Le Friday 12 August 2011 à 16:00 +0200, Leszek Lesner a écrit : I created a little patch which creates a new menuitem in the new menu called Shortcut. When its called it will execute lxshortcut -i filename_you_entered_for_the_new_desktop.file I originally created this patch for ZevenOS-Neptune and the debian version of pcmanfm (which seems to be an older version). I hope this patch works also for the ubuntu package and the newer version and is useful until a 'real' integration for desktop creation is done in pcmanfm / libfm. Thanks Leszek for the patch. You should post it to pcmanfm bugtracker so upstream can have a look at it :) I'm not sure the use case you try to cover with this patch. Do you have an example ? Also, IMO it should handle the case when lxshortcut is not installed (adding a text box asking to install lxshortcut). Regards, Julien Lavergne ___ Mailing list: https://launchpad.net/~lubuntu-desktop Post to : lubuntu-desktop@lists.launchpad.net Unsubscribe : https://launchpad.net/~lubuntu-desktop More help : https://help.launchpad.net/ListHelp
Re: [Lubuntu-desktop] lxshortcut integration in pcmanfm PATCH
For me, my plan for this part is to make lxshortcut a library, or have this functionality built into libfm directly. A library might be more appropriate since this can also be used by lxsession-edit later. A separate binary is better when we want to make it an optional component and handle cases when it's not installed. Making it a library will let other programs integrate it nicely and easily since many other desktop components might need this functionality, such as lxpanel, pcmanfm, and lxsession-edit. Putting it in libfm has the benefit that we don't have to maintain yet another library. The cons is other programs requiring this feature will depend on libfm. I have not decide how to do this yet, but personally I prefer the library one. (maybe libdesktop-edit?) On Fri, Aug 19, 2011 at 1:10 AM, Julien Lavergne gi...@ubuntu.com wrote: Le Friday 12 August 2011 à 16:00 +0200, Leszek Lesner a écrit : I created a little patch which creates a new menuitem in the new menu called Shortcut. When its called it will execute lxshortcut -i filename_you_entered_for_the_new_desktop.file I originally created this patch for ZevenOS-Neptune and the debian version of pcmanfm (which seems to be an older version). I hope this patch works also for the ubuntu package and the newer version and is useful until a 'real' integration for desktop creation is done in pcmanfm / libfm. Thanks Leszek for the patch. You should post it to pcmanfm bugtracker so upstream can have a look at it :) I'm not sure the use case you try to cover with this patch. Do you have an example ? Also, IMO it should handle the case when lxshortcut is not installed (adding a text box asking to install lxshortcut). Regards, Julien Lavergne ___ Mailing list: https://launchpad.net/~lubuntu-desktop Post to : lubuntu-desktop@lists.launchpad.net Unsubscribe : https://launchpad.net/~lubuntu-desktop More help : https://help.launchpad.net/ListHelp ___ Mailing list: https://launchpad.net/~lubuntu-desktop Post to : lubuntu-desktop@lists.launchpad.net Unsubscribe : https://launchpad.net/~lubuntu-desktop More help : https://help.launchpad.net/ListHelp
Re: [Lubuntu-desktop] lxshortcut integration in pcmanfm PATCH
I have tested this with leszek, Of course I chose an CLI operation - it did not like that. With popping on a 'standard' desktop app (leafpad) it worked fine. For people wanting to add apps to the desktop, this patch is good to go. Asking shortcuts to launch CLI stuff has always been a little more complicated -- I do have the instructions somewhere, but us people wanting CLI should really know how to launch them :) As it is an FAQ for adding a desktop icon and our present work around is, to say the least, involved. I would like this bug fix to be added. regards, Phill. On Fri, Aug 12, 2011 at 3:00 PM, Leszek Lesner leszek.les...@web.de wrote: Hi folks, I created a little patch which creates a new menuitem in the new menu called Shortcut. When its called it will execute lxshortcut -i filename_you_entered_for_the_new_desktop.file I originally created this patch for ZevenOS-Neptune and the debian version of pcmanfm (which seems to be an older version). I hope this patch works also for the ubuntu package and the newer version and is useful until a 'real' integration for desktop creation is done in pcmanfm / libfm. ___ Mailing list: https://launchpad.net/~lubuntu-desktop Post to : lubuntu-desktop@lists.launchpad.net Unsubscribe : https://launchpad.net/~lubuntu-desktop More help : https://help.launchpad.net/ListHelp -- https://wiki.ubuntu.com/phillw ___ Mailing list: https://launchpad.net/~lubuntu-desktop Post to : lubuntu-desktop@lists.launchpad.net Unsubscribe : https://launchpad.net/~lubuntu-desktop More help : https://help.launchpad.net/ListHelp
