-
Notifications
You must be signed in to change notification settings - Fork 12
Conversation
@Turmio
Turmio
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR also includes some of https://github.com/eficode/JavaFXLibrary/tree/java-11-support branch solutions and has solutions to unresolved issues in that branch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had a mouse with back and forward buttons. Test failed since message had BACK and FORWARD values.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jdk-11+26 version did not work. Had some missing methods. Upgrading resolved issues.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Relates to previous comment. Maybe these are not necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed public API in JavaFX.
Java 11 support. Currently tested on Windows machine so may need some extra work on other machines. CI is not fixed yet.
Implements #23