RFR: 8090547: Allow for transparent backgrounds in WebView [v6]

Jose Pereda jpereda at openjdk.java.net
Mon Sep 6 15:08:42 UTC 2021


On Mon, 6 Sep 2021 13:30:46 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Jose Pereda has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use color to int32 converter instead of hash
>
>> The other option, as you mention, is finding out if the 32-bit int has alpha 0 or 1, which can be done storing only the int value, not the color, so this looks cleaner, we don't really need to hold a reference to the Color after all
> 
> Yes, I like this better, too.
> 
> I also agree with the point about including `rgba` in the name. It is also meaningful to keep `int` in the name. We would typically call this format `IntRgba`.

@kevinrushforth In case you didn't see it, I replied to your comment https://github.com/openjdk/jfx/pull/563#discussion_r702164954 here: https://github.com/openjdk/jfx/pull/563/files#r702814752 (but there was not email notification).

-------------

PR: https://git.openjdk.java.net/jfx/pull/563


More information about the openjfx-dev mailing list