On Tue, 3 Dec 2024 15:05:54 GMT, Lukasz Kostyra <lkost...@openjdk.org> wrote:
>> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 72 commits: >> >> - Merge remote-tracking branch 'origin/master' into 8301121.RichTextArea >> - review comments >> - Merge remote-tracking branch 'origin/master' into 8301121.RichTextArea >> - whitespace >> - since 24 >> - review comments >> - Merge remote-tracking branch 'origin/master' into 8301121.RichTextArea >> - hide skin input map >> - Merge remote-tracking branch 'origin/master' into 8301121.RichTextArea >> - whitespace >> - ... and 62 more: https://git.openjdk.org/jfx/compare/67eed6d8...4b27b891 > > modules/jfx.incubator.richtext/src/main/java/jfx/incubator/scene/control/richtext/RichTextArea.java > line 1193: > >> 1191: * @see RichTextArea.Tags#CUT >> 1192: */ >> 1193: public void cut() { > > We have `copy(DataFormat)` and `paste(DataFormat)`. Would it make sense to > also have `cut(DataFormat)`? I would say there is no use case for `cut(DataFormat)`. The copy and paste are used for "Copy Special..." and "Paste Special..." items in the context menu - see `RichTextAreaDemoPane:416` ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1524#discussion_r1868060684
