clarifications in Export to Application for macOS

benfry · benfry · commit b204d80f1841 · 2022-08-08T17:57:42.000-04:00
diff --git a/java/src/processing/mode/java/ExportPrompt.java b/java/src/processing/mode/java/ExportPrompt.java
@@ -206,7 +206,7 @@ protected boolean trigger() throws IOException, SketchException {
     final String warning1 =
       "<html><div width=\"" + divWidth + "\">";
     final String warning2a =
-      "Embedding Java will make the " + platformName + " application " +
+      "This option will make the " + platformName + " application " +
       "larger, but it will be far more likely to work. " +
       "Users on other platforms will need to ";
     final String warning2b =
@@ -226,7 +226,7 @@ public void mousePressed(MouseEvent event) {
       }
     });
     warningLabel.setBorder(new EmptyBorder(3, 13, 3, 13));
-    warningLabel.putClientProperty("FlatLaf.styleClass", "medium");
+//    warningLabel.putClientProperty("FlatLaf.styleClass", "medium");
 
     final JCheckBox embedJavaButton =
       new JCheckBox(Language.interpolate("export.include_java", Platform.getPrettyName()));
@@ -262,24 +262,25 @@ public void mousePressed(MouseEvent event) {
 
       //if (false && new File("/usr/bin/codesign_allocate").exists()) {
       if (JavaBuild.isXcodeInstalled()) {
-        thePain +=
+        thePain += "<br/>" +
           "This application will be \u201Cself-signed\u201D which means that " +
-          "macOS may complain that is from an unidentified developer. " +
+          "macOS may complain that it comes from an unidentified developer. " +
           "If the application will not run, try right-clicking the app and " +
           "selecting Open from the pop-up menu. " +
           "More details at the <a href=\"\">Exporting Applications</a> wiki page.";
       } else {
         thePain +=
-          "To sign the app, <a href=\"\">click here</a> to begin " +
-          "installing the Command Line Tools from Apple. ";
+          "To create a working macOS application, " +
+          "<a href=\"\">click here</a> to install " +
+          "the Command Line Tools from Apple.";
       }
 
       // Are you f-king serious, Java API developers?
       // (Unless it's an HTML component, even with line wrap turned on,
       // getPreferredSize() will return the size for just a single line.)
 //      JLabel area = new JLabel("<html><div width=\"" + divWidth + "\"><font size=\"2\">" + thePain + "</div></html>");
       JLabel area = new JLabel("<html><div width=\"" + divWidth + "\">" + thePain + "</div></html>");
-      area.putClientProperty("FlatLaf.styleClass", "medium");
+//      area.putClientProperty("FlatLaf.styleClass", "medium");
 
       area.setBorder(new EmptyBorder(3, 13, 3, 13));
       // Using area.setPreferredSize() here doesn't help,
diff --git a/todo.txt b/todo.txt
@@ -43,6 +43,13 @@ X   figure out if we can do anything with signing applications
 X   https://developer.apple.com/forums/thread/128166
 X   https://developer.apple.com/forums/thread/701514#701514021
 X   https://developer.apple.com/forums/thread/701581#701581021
+/ Xcode command line tools must be installed first
+/   show a message before even opening the Export dialog
+X   can't restrict to installation b/c may want to do other platforms
+X   or detection of the CLT may not be correct
+X   or users may be doing their own manual signing anyway
+X   for now, just cleaning up the messages a bit for clarity
+X   and bump the font size back up for legibility
 
 docs
 X update theme instructions
@@ -373,10 +380,8 @@ old things
 + Examples window (Casey will pick 3-4)
 * Reference, Discourse, Github, Wiki
 
-
-
-_ Xcode command line tools must be installed first
-_   show a message before even opening the Export dialog
+_ recent errors from AndroidMode
+_ add a note about underscores in sketch tabs in prefs doc
 
 
 detach sketch names
