[jfx18] RFR: 8279345: Realign class docs of LightBase and subclasses
Ambarish Rapte
arapte at openjdk.java.net
Tue Jan 25 09:19:45 UTC 2022
On Sun, 16 Jan 2022 22:54:22 GMT, Nir Lisker <nlisker at openjdk.org> wrote:
> Now that the standard concrete light types were added, there is an opportunity to rearrange and rewrite some of the class docs. Here is a summary of the changes:
>
> * Moved the explanations of attenuation and direction up to `LightBase` since different light types share characteristics. `LightBase` now contains a summary of its subtypes and all the explanations of the properties/characteristics of the lights divided into sections: Color, Scope, Direction, Attenuation.
> * Each light type links to the relevant section in `LightBase` when it mentioned the properties it has.
> * Added examples of real-world applications for each light type.
> * Consolidated the writing style for all the subclasses.
modules/javafx.graphics/src/main/java/javafx/scene/DirectionalLight.java line 46:
> 44: * {@code DirectionalLight}s can represent strong light sources that are far enough from the objects they illuminate
> 45: * that their light rays appear to be parallel. Because these light sources are considered to be infinitely far, they
> 46: * cannot be attenuated. The sun's light on Earth is a common light source that can be simulated with this light type.
The previous statement seems more suitable here:
`The Sun is a common light source that can be simulated with this light type.`
or little modified version of this new statement:
`The Sun light on Earth is a common type of light that can be simulated with this light type.`
modules/javafx.graphics/src/main/java/javafx/scene/LightBase.java line 133:
> 131: *
> 132: * <h2><a id="Direction">Direction</a></h2>
> 133: * The direction the light is facing, defined by the {@code direction} vector property of the light. The light's
The direction of the light is defined by the {@code direction} vector property of the light
-------------
PR: https://git.openjdk.java.net/jfx/pull/717
More information about the openjfx-dev
mailing list