On Fri, 9 Oct 2026 23:24:17 GMT, Kevin Rushforth <[email protected]> wrote:

>> Andy Goryachev has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   cleanup
>
> modules/jfx.incubator.richtext/src/main/java/jfx/incubator/scene/control/richtext/model/RichTextModel.java
>  line 77:
> 
>> 75:     public RichTextModel() {
>> 76:         registerDataFormatHandler(RichTextFormatHandler.getInstance(), 
>> true, true, 1000);
>> 77:         registerDataFormatHandler(ImageFormatHandler.getInstance(), 
>> false, true, 500);
> 
> Is it worth a follow-up issue to document the precedence of the various data 
> formats and their handlers?

Hmm, I would rather keep it as implementation detail for now, though it might 
be important for a custom model.

The other alternative is to specify the maximum used by the base model 
(possibly adding a public constant in the `StyledTextModel` )

Yet another possibility, as a temporary measure, is to move these constants to 
Params which is not a part of the documentation.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2224#discussion_r4251296121

Reply via email to