-
Notifications
You must be signed in to change notification settings - Fork 120
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Webkit used in JavaFX 18.0.1 breaks the library #204
Comments
FWIW, It's works for me, here on Windows with an FX bundled JDK from Azul. ...and some exclusions in the pom https://github.com/trixon/mapton/blob/86ae5376e83f74abb53fb8445f057f03843cbc30/modules/google-maps/pom.xml#L65 |
Can you please try cloning this project and running the "MainApp" class? With 18 and then with 18.0.1? |
Also .... do you know which javafx version is bundled with the 18.0.1 JDK? |
@dlemmermann I did some testing and can confirm the problems. Some time ago I ran into that JavaFX will not load nicely from the classpath anymore. So I replaced maven-exec-plugin in your pom with this inspired by the -new project wizzard- in NetBeans.
|
@dlemmermann Is this of any help? I took another look at this and noticed that even the javafx version 17.0.2 did not work for me with Temurine jdk-11.0.18+10. I got it working by reordering the start method like so:
But then I reverted that and used a new version, <javafx.version>19.0.2.1</javafx.version>, and that worked too for me. |
no, that is not helping :-) so you changed the order and it worked and then you changed it back and it still worked? Sounds to me like you didn't actually run with the newer javafx versions. |
That's not what I said, but I'm somewhat confused and had to triple check. The java code change is required for me on window with Temurin 11 if I use javafx 17.0.2 or 18. An alternative to get it working is by using the javafx jmods. I'm unable to reproduce this problem on Fedora 37 Linux. |
No maps are rendered when using JavaFX 18.0.1 with its upgraded WebKit version.
The text was updated successfully, but these errors were encountered: