On Wed, 19 Aug 2026 12:32:51 GMT, Marius Hanl <[email protected]> wrote:

>> Michael Strauß has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   remove empty tag
>
> modules/javafx.graphics/src/main/java/javafx/scene/layout/package-info.java 
> line 245:
> 
>> 243:  *
>> 244:  * At a render scale of {@code 1.0}, one logical unit is one pixel. At 
>> a render scale of {@code 1.5}, one pixel is
>> 245:  * {@code 1 / 1.5} ≈ {@code 0.6667}, logical units. A correctly snapped 
>> value is therefore not necessarily an integer,
> 
> Is it just me or does the comma feels off here: `one pixel is ... 0.6667, 
> logical units`

yep, comma is not needed here.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2260#discussion_r3815687005

Reply via email to