RFR: 8234920: Add SpotLight to the selection of 3D light types [v15]
Nir Lisker
nlisker at openjdk.java.net
Wed May 26 09:43:18 UTC 2021
On Tue, 25 May 2021 11:30:25 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:
>> Nir Lisker has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Update documentation
>
> modules/javafx.graphics/src/main/java/javafx/scene/SpotLight.java line 49:
>
>> 47: * <p>
>> 48: * In addition to the attenuation factors defined in {@code PointLight}, the {@code Spotlight}'s intensity is also
>> 49: * affected by 3 factors:
>
> I think instead of phrasing it as "**affected** by 3 factors", it should be phrased as something like,
> `the SpotLight is defined by 3 more properties` Or
> `the SpotLight possesses 3 more properties` ?
I think that a big part of the difficulty in explaining what is happening is that we are dancing around the mathematical formulation instead of giving it directly :)
I will update the `SpotLight`'s docs to be more mathematical, but it requires to also update `PointLight`'s docs to be entirely clear. My intention is to update the docs for all the lights and material after we finish adding this one and `DirectionalLight` since it will be possible to do similarities and differences comparisons between them.
-------------
PR: https://git.openjdk.java.net/jfx/pull/334
More information about the openjfx-dev
mailing list