Skip to content

Commit f685a1b

Browse files
committed
clean up/clarify message about missing JavaFX
1 parent 9832a9a commit f685a1b

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

core/src/processing/core/PApplet.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2092,12 +2092,12 @@ protected PGraphics makeGraphics(int w, int h,
20922092
}
20932093

20942094
} catch (ClassNotFoundException cnfe) {
2095-
// if (cnfe.getMessage().indexOf("processing.opengl.PGraphicsOpenGL") != -1) {
2096-
// throw new RuntimeException(openglError +
2097-
// " (The library .jar file is missing.)");
2098-
// } else {
2095+
// Clarify the error mesage for less confusion on 4.x
2096+
if (renderer.equals(FX2D)) {
2097+
renderer = "JavaFX";
2098+
}
20992099
if (external) {
2100-
throw new RuntimeException("You need to use \"Import Library\" " +
2100+
throw new RuntimeException("Please use Sketch \u2192 Import Library " +
21012101
"to add " + renderer + " to your sketch.");
21022102
} else {
21032103
throw new RuntimeException("The " + renderer +

todo.txt

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,9 @@ X fix modules path warning for Tools in the PDE
7171
X "WARNING: Unsupported JavaFX configuration" when running Tools that use JavaFX
7272
X we were probably spared the warnings because the older JARs were hanging around
7373
X update modules path when exporting application
74-
_ lots more cleanup to do in javafx/build.xml
74+
X You need to use "Import Library" to add processing.javafx.PGraphicsJavaFX
75+
o automatically import JavaFX if FX2D is in sketch? or tell user?
76+
X concerned about auto-import being too unreliable
7577

7678
major font cleanup
7779
X JDK fonts have been removed; fonts folder too? (not seeing it in the JDK)
@@ -95,8 +97,6 @@ X i.e. all the things that rely on preferences can be inited separately (later
9597

9698

9799
for next release
98-
_ You need to use "Import Library" to add processing.javafx.PGraphicsJavaFX
99-
_ automatically import JavaFX if FX2D is in sketch? or tell user?
100100
_ Code completion not working
101101
_ https://github.com/processing/processing4/issues/177
102102
_ confirmed not working from Sam's repo either
@@ -105,6 +105,8 @@ _ change application signature from Pde3 to Pde4 (and package name as well)
105105

106106
_ do a test to see if PApplet extends PSketch breaks libraries
107107

108+
_ lots more cleanup to do in javafx/build.xml
109+
108110
_ remove the PDE classpath from sketches
109111
_ causing it to include incomplete JFX, other misc; too confusing
110112

0 commit comments

Comments
 (0)