RFR: 8314147: Updated the PhongMaterial documentation [v6]

Nir Lisker nlisker at openjdk.org
Wed Feb 28 14:09:55 UTC 2024


On Tue, 27 Feb 2024 11:53:35 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:

>> Nir Lisker has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fixed typo
>
> modules/javafx.graphics/src/main/java/javafx/scene/paint/PhongMaterial.java line 115:
> 
>> 113:  * <p>
>> 114:  * <b>Important:</b> there is currently a bug that causes objects with 0 opacity to not render at all (despite having a
>> 115:  * specular or a self-illumination component). Setting the opacity to 1/255 instead will give the desirable result.
> 
> Is there is JBS issue to track this? The JBS issue should have a pointer for this comment to be removed when fixed.

No, not yet. There a few oddities in the shader that I intend to look at for jfx23, this will be one of them.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1378#discussion_r1506020787


More information about the openjfx-dev mailing list