[haiku-development] Re: tracker patch
- From: "Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx>
- To: haiku-development@xxxxxxxxxxxxx
- Date: Fri, 25 May 2007 19:58:28 +0200 CEST
mike@xxxxxxxxxxx wrote:
> The attached patch adds an option to tracker to turn off the icon
> text on
> the desktop. I hope it will be accepted.
While the patch itself looks good (only a minor coding style issue in
line 151/152), and you've found the right places for your changes
(which isn't always that simple in Tracker), I don't quite understand
the purpose of this patch, to be honest.
Why would anyone ever want to remove the text from all icons on his
desktop (or you in this case :-))?
> if this is the wrong place for these types of patches please let me
> know.
> I have some more but wanted to send a "test" one first.
No, it's fine to post them here.
Bye,
Axel.
Other related posts:
