[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [RFC] [PATCH]: Icon themability patch for -GUI images/icons

From: Fred Kiefer
Subject: Re: [RFC] [PATCH]: Icon themability patch for -GUI images/icons
Date: Sat, 30 Oct 2004 17:16:59 +0200
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.6) Gecko/20040114

Alex Perez wrote:
This patch builds upon a previous non-behavior-changing patch which I wrote and Alexander Malmberg comitted this morning on my behalf. For some background:

NSWorkspace had a private convenience class called _getImageWithName: which, with the permission of Alexander Malmberg, we moved to NSImage and renamed to -standardImageWithName: and modified slightly so the alternate: argument was not necessary (it now intelligently looks for "ImageName" and then for "common_ImageName" (and anything else for that matter, like "camaelon_ImageName", if you override the method.) In any event, What this patch does is enable icon themability. You can have sets of icons which you can use by simply having a very small bundle override the -getImageWithName: method.

This patch builds upon the previous one, by making nearly every single image which was previously loaded directly via NSImage -imageNamed: "common_ImageName" load with the new convenience NSImage method.

Unless you want your own icon set, you will notice zero behavioral change.

Everyone, please comment on this...If I don't hear anything negative from anyone by tuesday or so of next week, I will commit it as-is. It works on my machine.

Sorry to repeat myself, but NSImage has been themable all the time. There is the file nsmapping.strings in gui/Images, which defines which file name will be used instead of a given image name. What else would you need for themability? Yes, the problem is that this feature is currently not widely used in GNUstep GUI and in the GNUstep applications. Hard coded image names are often used in them. Do you expect the introduction of an additional API, which you even expect to be overwritten in theme bundles to resolve this?

Sorry, I am actually getting bored of this and most other pseudo-discussions going on in the GNUstep mailing lists. Perhaps I should think about dropping out of them.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]