On Thu, 30 Jul 2026 11:22:01 GMT, John Hendrikx <[email protected]> wrote:

>> Andy Goryachev has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   font property
>
> modules/javafx.graphics/src/main/docs/javafx/scene/doc-files/cssref.html line 
> 1427:
> 
>> 1425:     <h4><a id="fontprops">Font Properties</a></h4>
>> 1426:     <p>Most classes that use text will support the following font 
>> properties.</p>
>> 1427:     <p>Additionally, any property whose suffix matches an entry in the 
>> table below will be treated
> 
> Although the header does capture that this is about font properties, this 
> still somehow reads to me as "any random property whose suffix ..." so 
> suggestions:
> 
> 
> Suggestion:
> 
>     <p>Additionally, any font property whose suffix matches an entry in the 
> table below will be treated

hmmm, still not the best - it creates font...font tautology, and also in all 
these properties only the "font" suffix corresponds to the font property, the 
other control certain aspects of the font (font-weight).

is the new version better?

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2230#discussion_r3683991312

Reply via email to