Mouse-Related events aren't overrideable like KeyEvents are

[JordanMartinez]

In StyledTextAreaBehavior, MouseEvents are handled using EventStreams which don't take into consideration if the MouseEvent is already consumed (unlike EventHandlerTemplate and KeyEvents):

EventHandlerHelper.installAfter(area.onKeyPressedProperty(), keyPressedHandler);
EventHandlerHelper.installAfter(area.onKeyTypedProperty(), keyTypedHandler);

subscription = Subscription.multi(
        eventsOf(area, MouseEvent.MOUSE_PRESSED).subscribe(this::mousePressed),
        eventsOf(area, MouseEvent.MOUSE_DRAGGED).subscribe(this::mouseDragged),
        eventsOf(area, MouseEvent.DRAG_DETECTED).subscribe(this::dragDetected),
        eventsOf(area, MouseEvent.MOUSE_RELEASED).subscribe(this::mouseReleased),
        () -> {
            EventHandlerHelper.remove(area.onKeyPressedProperty(), keyPressedHandler);
            EventHandlerHelper.remove(area.onKeyTypedProperty(), keyTypedHandler);
        });

The following summarized code doesn't work:

EventHandler<MouseEvent> mouseRelease = EventHandlerTemplate
            .on(MouseEvent.MOUSE_RELEASED).act {area, event -> event.consume() }
            .create()
            .bind(this)
EventHandlerHelper.install(onMouseReleasedProperty(), mouseRelease)

If I want to customize the MouseEvent handling, it can only be done by adding an EventFilter

[TomasMikula]

Good point. We should investigate how to port the mouse event handling to the same approach that is used for key handling.

[TomasMikula]

Oh, I see you already did that :)

[JordanMartinez]

😄 Yup!

[JordanMartinez]

So...... this issue and others I've opened (#218, #223, #152) have been resolved in the current master branch but haven't yet been included in an official release. Am I supposed to close them now, or wait for you to do so when the next version that includes the code that resolves these issues is released?

[TomasMikula]

Thanks for checking up on them. I think we can close them now :)

[JordanMartinez]

Resolved via ca3b771