On Thu, 30 Jul 2026 15:26:04 GMT, Andy Goryachev <[email protected]> wrote:

>> 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?

I think it is an improvement; if you still don't like it though you could 
perhaps start with "Such properties... " to make it clear you are referencing 
the font properties from the previous paragraph.

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

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

Reply via email to