GNOME Bugzilla – Bug 160723
New Launcher Backend & Tile
Last modified: 2005-01-13 00:53:27 UTC
I mostly brought this back to test the new SQlite code, not sure if it is still wanted. Will attach patch (against CVS) and screenshot.
Created attachment 34608 [details] [review] Beagle Launcher Patch
Created attachment 34609 [details] Beagle Launcher Screenshot
Basic patch looks good. There are a few changes needed before it can go in, though: * Invoking pkg-config from inside the backend is really gross. It'd be much better if it were done at configure-time. Bonus points for also checking where KDE is installed and using those paths. Look at beagled/ExternalStringsHack.cs.in to see the basics on how to do this. * Since you're just doing one string concatenation for each directory, there's no need to create a StringBuilder. You can just do: Dirs.Add (fullpath.ToString () + "applications/"); and then deal with just strings. If there were a lot more using a StringBuilder would be more advantageous.
Created attachment 34670 [details] [review] Updated Launcher Patch This patch contains fix's for the StringBuilder and pkg-config problems that Joe mentioned. KDE support will have to wait till I can figure out how to find the KDE icons. Patch works against CVS.
sorry for the delay, this looks good to commit.