[icedtea-web] RFC: PR918: java applet windows uses a low resulution black/white icon

[Omair Majid]

Hi Jiri,

On 04/18/2012 11:46 AM, Jiri Vanek wrote:
> On 04/18/2012 04:57 PM, Omair Majid wrote:
> Walking through the patch, but I will reply tomorrow after full
> exploration.

Thanks in advance!

>>
>> The attached patch fixes PR918 [1] by removing the code the set the
>> icons for all top-level windows. Any windows created by an applet/jnlp
>> application should now get the default java icon if they dont set an
>> icon explicitly. The patch also sets the icons for all the top-level
>> windows we create explicitly to the default application icon (currently
>> netx's icon).
>>
>> We probably want to change the application icon from netx-icon.png to
>> something else too, but I plan to tackle that as a separate issue (do we
>> even have an icedtea-web icon?)
>>
>> Any thoughts or comments?
>>
> Yap  O:) - after first look the usage of Nx icon have terrified me :)
> [nothing personal, it is just small and nasty;)
> 

As I said, I am completely open to replacing it. The patch just makes
sure that the icon is only used for icedtea-web windows as opposed to
all windows by default.

> Just hint for icon - IMHO it should be similar to splash screen.

Yes, some sort of consistency would be great.

> On
> one side i can re factor splash screen design to match new icon. On
> second new icon can be inspired by splash screen.
> so 16x16 I and W letters, some color matches
>    32x32 IcedTea-web as my small splashscreen is designed
>    64x64 Just shrunken splashscreen - full colours and leafs
> 
> This sizes are just idea

I am not a graphics expert, so I will be okay with whatever new
size/shape the new icon takes (and whoever the new designer happens to
be ;) ).

> and your ImageResources can provide small x
> medium x large icon as desired by moment and situation.
> 

Yes, I can certainly modify ImageResources to take care of more sizes.

Cheers,
Omair