Hello Sam,

I say commit it, or create an issue if you would like to allow more comment.

Whether or not there is an issue created, having an example in the
examples project to demonstrate the benefits of this change would be
cool.

   michael


On Tue, Sep 30, 2008 at 11:12 AM, woodymac <[EMAIL PROTECTED]> wrote:
>
> Please make this change, so I can dump my subclass of PCanvas.
>
> On Sep 17, 12:36 am, Sam Reid <[EMAIL PROTECTED]> wrote:
>> I'd like to make it possible to make PCanvas partly transparent, but
>> that's impossible with current implementation of PCanvas.  Line 605
>> gives:
>>
>>   g2.setColor( getBackground() );
>>   g2.fillRect( 0, 0, getWidth(), getHeight() );
>>
>> I'd recommend changing this to:
>>
>>         if ( isOpaque() ) {
>>             g2.setColor( getBackground() );
>>             g2.fillRect( 0, 0, getWidth(), getHeight() );
>>         }
>>
>> This is similar to the swing ui implementation.
>>
>> I couldn't find a workaround that involved using a Color with a
>> transparent alpha channel, see discussions 
>> here:http://www.chka.de/swing/components/opacity.html
>>
>> Any comments, questions or objections to this change?
>>
>> Sam Reid

--~--~---------~--~----~------------~-------~--~----~
Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to