On Mon, 6 May 2024 17:59:14 GMT, Andy Goryachev <ango...@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

Reply via email to