> Yes, I realise it's some work to set up the styles. But it'll be a lot 
> smaller amount of work to consider now than later. And I don't see much of a 
> maintenance issue. It was only a request to consider it in the discussions.

Noted! I think this really becomes up to whoever decides to continue work on 
drawing the icons. Right now we have Peter Cheung working on a first batch, 
delegating to his graphic designers, who are experienced with Adobe 
Illustrator. Perhaps let's see where that goes first.

> This has nothing to do with single accent colours, changing the default 
> palette or platform applications
(from https://github.com/apache/netbeans/pull/3445 :)
> at no point have I suggested switching to duotone icons in order to integrate 
> with this change

Sorry, to clarify: The point about the color palette was my own. My argument is 
that the existing NetBeans icon style is not very well-suited for assigning 
logical colors. In order to designate a meaningful, configurable, "accent 
color", for example, we would have to change the overall color palette of the 
entire icon set. And that's something that's easier to do after-the-fact than 
up-front (see my earlier explanation for why).

-- Eirik

-----Original Message-----
From: Neil C Smith <[email protected]> 
Sent: Sunday, January 2, 2022 10:06 AM
To: dev <[email protected]>
Subject: Re: New theme for netbeans, team member wanted

On Sun, 2 Jan 2022, 14:57 Eirik Bakke, <[email protected]> wrote:

> > Relying on colour filtering of the rasterized icons for eg. dark 
> > themes
> makes no sense when we can style the SVG data before rasterization 
> with semantic information intact.
>
> The motivation is just that it's less work. Even things like picking 
> names for the color classes is work. And then we're stuck with 
> maintaining and supporting those classes, like maintaining an API.
>

Yes, I realise it's some work to set up the styles. But it'll be a lot smaller 
amount of work to consider now than later. And I don't see much of a 
maintenance issue. It was only a request to consider it in the discussions. But 
if they're added, I'm happy to look at the loading code support.

Best wishes,

Neil

Reply via email to