On Thu, 1 Jun 2023 15:42:26 GMT, Andy Goryachev <ango...@openjdk.org> wrote:

>> I think this link should point to the (currently missing) Controls header 
>> and not the Control class. The "details" on focusibility that this note 
>> refers to are in the package header for `javafx.scene.control`, which will 
>> be right after the Controls header once you add it.
>
> there is a duplicate entry for -fx-focus-traversable in Node and Control 
> sections.  I think this particular link should point to Control, where it 
> says 
> 
> "Control sets the default value of the focusTraversable property to true. The 
> default value of the focusTraversable property for the following controls is 
> false: Accordion, Cell, Label, MenuBar, ProgressBar, ProgressIndicator, 
> ScrollBar, ScrollPane, Separator, SplitPane, ToolBar."

Although... the information is also repeated in the Control class, so I won't 
object if you want to keep your change to point there.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1144#discussion_r1213355006

Reply via email to