RFR: 8331214: Doc: update spec for SpinnerFactory classes [v2]

[drmarmac]

On Mon, 6 May 2024 17:59:14 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> I don't think this will help. This is the same issue I mentioned in PR #1389 -- you need to link to the property _method_, so I think changing to the following will do it:
>> 
>> 
>> * If {@link #wrapAroundProperty wrapAround}
>> 
>> 
>> (it shouldn't be necessary to qualify it with the enclosing class, but it wouldn't hurt to do so)
>
> `{@link #wrapAroundProperty wrapAround}` does not resolve **in Eclipse** (it does in javadoc output).
> 
> This might be Eclipse limitation, but it would be awfully nice to have it resolved in Eclipse.

I also fixed the min/max links to refer to minProperty/maxProperty. IntelliJ IDEA has the same problem as eclipse. I guess this has to do with the "javafx" option to the javadoc command (build.gradle:4286 `options.addBooleanOption("javafx").setValue(true);`) that seems to allow to use the private fields' docs for the properties, by naming convention? Maybe we need to file issues in Eclipse/IntelliJ for this so they add the option to their javadoc renderer.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1450#discussion_r1591581530