Double-clicking a word selects the word plus the leading or trailing space

[benedictleejh]

In the InlineStyleTextArea, when I double-click on a word, it selects the word and its leading or trailing space. I'd like for the double-click to only select the word itself, without the leading or trailing space (similar to the JavaFX text area). Is this possible, and if so, how should I do so?

[JordanMartinez]

Here's a quick workaround:

area.addEventHandler(MouseEvent.MOUSE_CLICKED, new EventHandler<MouseEvent>() {
    @Override
    public void handle(MouseEvent event) {
        if (event.getClickCount() == 2) {
            IndexRange range = area.getSelection();
            String selectedText = area.getSelectedText();
            if (selectedText.startsWith(" ")) {
                area.selectRange(range.getStart() + 1, range.getEnd());
            } else if (selectedText.endsWith(" ")) {
                area.selectRange(range.getStart(), range.getEnd() - 1);
            }
        }
    }
});

[TomasMikula]

Thanks for reporting and thanks for the work-around. We should fix this at the core, though.

[JordanMartinez]

Although I don't fully understand BreakIterator, I found a way to make it work.

• Remove NavigationActions.previousWord()'s line: "wordBreakIterator.previous()"
• Remove NavigationActions.nextWord()'s line: "wordBreakIterator.next()"

[TomasMikula]

That will fix this issue, but previousWord and nextWord are used elsewhere, so it will likely break other usages.

[JordanMartinez]

Something similar to this issue is when the user presses the combination SHIFT+Shortcut+LEFT/RIGHT. The selection will also include an extra space.

For example, let's say "|" represents the caret position in the following sentence
"This sim|ple sentence gives an illustration,"
Pressing the above combination to the right will select "ple " instead of just "ple" (shown by angle brackets)
"This sim<ple >sentence gives an illustration"
Every additional selection, if doing the same key press, will still include the extra space char.

"This sim<ple sentence >gives an illustration"
"This sim<ple sentence gives >an illustration"

If going to the left, the selection will be " sim" instead of "sim"
"This< sim>ple sentence gives an illustration."

[JordanMartinez]

How about changing previousWord() and nextWord() to the following (not sure how the javadoc would need to be updated):

default void previousWord(SelectionPolicy selectionPolicy, boolean applyPrevious) {
    if(getLength() == 0) {
        return;
    }

    BreakIterator wordBreakIterator = BreakIterator.getWordInstance();
    wordBreakIterator.setText(getText());
    wordBreakIterator.preceding(getCaretPosition());
    if (applyPrevious) { wordBreakIterator.previous(); }

    moveTo(wordBreakIterator.current(), selectionPolicy);
}

[TomasMikula]

I was actually thinking of something similar, but instead of a boolean, use an int n and then call wordBreakIterator.previous() n-1 times. A behavior-preserving refactoring would be to change all previousWord() calls to previousWord(2). Then to address this issue would be to change it to previousWord(1) somewhere. Maybe changing the method name would also make sense.

[JordanMartinez]

That makes sense, so something like this:

/**
* Skips n number of word boundaries backwards.
* Based on the given selection policy, anchor either moves with
* the caret, stays put, or moves to the former caret position.
*/
default void skipToPreviousWordBoundary(int n, SelectionPolicy selectionPolicy) {
    if(getLength() == 0) {
        return;
    }

    BreakIterator wordBreakIterator = BreakIterator.getWordInstance();
    wordBreakIterator.setText(getText());
    wordBreakIterator.preceding(getCaretPosition());
    if (n < 1) n = 1;
    // Do I need a maximum limit based on getLength() here ?
    for (int i = 0; i < n - 1; i++) {
        wordBreakIterator.previous();
    }

    moveTo(wordBreakIterator.current(), selectionPolicy);
}

Renaming it would make sense, but I'm not sure if my above suggestion is the best naming scheme.

[JordanMartinez]

If there's no need for a maximum limit, I guess a better way to write that for-loop code would be:

if (n >= 2) {
    for (int i = 0; i < n-1; i++) {
        wordBreakIterator.previous();
    }
}

[TomasMikula]

I don't really see a difference. What about just

    wordBreakIterator.preceding(getCaretPosition());
    for (int i = 1; i < n; i++) {
        wordBreakIterator.previous();
    }

[JordanMartinez]

Yeah, that's simpler.

What about the method's name?

[TomasMikula]

What about wordBreaksBackwards(int n), wordBreaksForwards(int n)?

[JordanMartinez]

I think that's good.

[TomasMikula]

OK, if you want to go ahead and submit a PR, go ahead ;)

[effad]

FYI: Just stumbled across this problem too and looking forward for a fix.