Re: RFR: 8299756: Minor updates in CSS Reference [v3]

[Marius Hanl]

On Tue, 30 May 2023 15:42:51 GMT, Andy Goryachev <ango...@openjdk.org> wrote:

>> modules/javafx.graphics/src/main/docs/javafx/scene/doc-files/cssref.html 
>> line 367:
>> 
>>> 365:         </ul>
>>> 366:       </li>
>>> 367:       <li><a id="controls">Controls</a>
>> 
>> We should consider leaving this as `href` and adding a `Controls` heading 
>> instead.
>> It should be here: 
>> ![image](https://github.com/openjdk/jfx/assets/66004280/be1134e5-91df-4650-b56d-3f6ecd7cc18a)
>> As it is for all other headers e.g. `Nodes`
>> ![image](https://github.com/openjdk/jfx/assets/66004280/17eba5ee-9804-4452-aaf2-b8448a9d4cb4)
>
> you are right

You need to add a `Controls` headline still.
`<h2><a id="nodes">Nodes</a></h2>`
It should be above the `javafx.scene.control` row.
Then the `href` will work and is consistent with the other headers.

![image](https://github.com/openjdk/jfx/assets/66004280/fd80afd3-5b12-401d-a9f2-d14f123927f6)

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

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