RFR: 8301121: RichTextArea Control (Incubator)

[Alexander Zuev]

On Tue, 30 Jul 2024 21:01:15 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

> Incubating a new feature - rich text control, **RichTextArea**, intended to bridge the functional gap with Swing and its StyledEditorKit/JEditorPane. The main design goal is to provide a control that is complete enough to be useful out-of-the box, as well as open to extension by the application developers.
> 
> This is a complex feature with a large API surface that would be nearly impossible to get right the first time, even after an extensive review. We are, therefore, introducing this in an incubating module, **jfx.incubator.richtext**. This will allow us to evolve the API in future releases without the strict compatibility constraints that other JavaFX modules have.
> 
> Please check out two manual test applications - one for RichTextArea (**RichTextAreaDemoApp**) and one for the CodeArea (**CodeAreaDemoApp**). Also, a small example provides a standalone rich text editor, see **RichEditorDemoApp**.
> 
> **References**
> 
> - Proposal: https://github.com/andy-goryachev-oracle/Test/blob/main/doc/RichTextArea/RichTextArea.md
> - Discussion points: https://github.com/andy-goryachev-oracle/Test/blob/main/doc/RichTextArea/RichTextAreaDiscussion.md
> - API specification (javadoc): https://cr.openjdk.org/~angorya/RichTextArea/javadoc
> - RichTextArea RFE: https://bugs.openjdk.org/browse/JDK-8301121
> - Behavior doc: https://github.com/andy-goryachev-oracle/jfx/blob/8301121.RichTextArea/doc-files/behavior/RichTextAreaBehavior.md
> - CSS Reference: https://cr.openjdk.org/~angorya/RichTextArea/javadoc/javafx.graphics/javafx/scene/doc-files/cssref.html
> - InputMap (v3): https://github.com/andy-goryachev-oracle/Test/blob/main/doc/InputMap/InputMapV3.md
> - Previous Draft PR: https://github.com/openjdk/jfx/pull/1374

modules/jfx.incubator.richtext/src/main/java/com/sun/jfx/incubator/scene/control/richtext/CellArrangement.java line 263:

> 261:      */
> 262:     private int binarySearch(double localY, int low, int high) {
> 263:         //System.err.println("    binarySearch off=" + off + ", high=" + high + ", low=" + low); // FIX

Looks like remnants of the debugging session. Do you still need it?

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1524#discussion_r1702417057