From 799a870bd212e7635b02fa7955161c4f7f819218 Mon Sep 17 00:00:00 2001
From: benfry <fry@processing.org>
Date: Fri, 3 Oct 2003 01:00:54 +0000
Subject: [PATCH] fixes to font builder for selecting items and making 'data'
 dir if not exist

---
 processing/app/PdeEditor.java      |  2 ++
 processing/app/PdeFontBuilder.java | 38 +++++++++++++++++++-----------
 processing/todo.txt                | 25 +++++++++++++++++---
 3 files changed, 48 insertions(+), 17 deletions(-)

diff --git a/processing/app/PdeEditor.java b/processing/app/PdeEditor.java
index 861dd2b12..32e425f3f 100644
--- a/processing/app/PdeEditor.java
+++ b/processing/app/PdeEditor.java
@@ -711,6 +711,8 @@ public class PdeEditor extends JPanel {
     //System.out.println("RUNNING");
     buttons.run();
 
+    for (int i = 0; i < 10; i++) System.out.println();
+
     if (externalEditor) {
       // history gets screwed by the open..
       String historySaved = historyLast;
diff --git a/processing/app/PdeFontBuilder.java b/processing/app/PdeFontBuilder.java
index 64455fb83..3d7aaf24b 100644
--- a/processing/app/PdeFontBuilder.java
+++ b/processing/app/PdeFontBuilder.java
@@ -127,23 +127,29 @@ public class PdeFontBuilder extends JFrame {
     fontSelector = new JList(list); //families); 
     fontSelector.addListSelectionListener(new ListSelectionListener() {
         public void valueChanged(ListSelectionEvent e) {
-          //System.out.println(e.getFirstIndex());
-          selection = e.getFirstIndex();
-          okButton.setEnabled(true);
+          if (e.getValueIsAdjusting() == false) {
+            //System.out.println(e);
+            //System.out.println(e.getFirstIndex());
+            //selection = e.getFirstIndex();
+            selection = fontSelector.getSelectedIndex();
+            okButton.setEnabled(true);
 
-          int fontsize = 0;
-          try {
-            fontsize = Integer.parseInt(sizeSelector.getText().trim());
-            //System.out.println("'" + sizeSelector.getText() + "'");
-          } catch (NumberFormatException e2) { }
+            int fontsize = 0;
+            try {
+              fontsize = Integer.parseInt(sizeSelector.getText().trim());
+              //System.out.println("'" + sizeSelector.getText() + "'");
+            } catch (NumberFormatException e2) { }
 
-          if (fontsize != 0) {
-            font = new Font(list[selection], Font.PLAIN, fontsize);
-            //System.out.println("setting font to " + font);
-            sample.setFont(font);
+            if (fontsize != 0) {
+              font = new Font(list[selection], Font.PLAIN, fontsize);
+              //System.out.println("setting font to " + font);
+              sample.setFont(font);
 
-            String filenameSuggestion = list[selection].replace(' ', '_');
-            filenameField.setText(filenameSuggestion);
+              String filenameSuggestion = list[selection].replace(' ', '_');
+              filenameField.setText(filenameSuggestion);
+            }
+            //filenameField.paintComponent(filenameField.getGraphics());
+            //getContentPane().repaint();
           }
         }
       });
@@ -280,6 +286,10 @@ public class PdeFontBuilder extends JFrame {
     try {
       font = new Font(list[selection], Font.PLAIN, fontsize);
       BFont f = new BFont(font, true);
+
+      // make sure the 'data' folder exists
+      if (!targetFolder.exists()) targetFolder.mkdirs();
+
       f.write(new FileOutputStream(new File(targetFolder, filename)));
 
     } catch (IOException e) {
diff --git a/processing/todo.txt b/processing/todo.txt
index 97ab7f3ca..3ecf84ee5 100644
--- a/processing/todo.txt
+++ b/processing/todo.txt
@@ -2,14 +2,29 @@
 X BImage.replicate for straight 1:1 copy, blend() for blended version
 X   remove the blendMode function because it's confusing 
 X big changes to image code from toxi
+X repaired smoothing so that it doesn't crush the last line
+X bresenham ellipses/circle ignore alpha
+X   strokeWeight != 1 then shouldn't use flat_rect or flat_ellipse
+X   alpha didn't work either b/c some pixels seem to draw more than once
+X fixed bug where font wouldn't create if data dir exists
+X font builder: changes to font selection don't always update properly
 
-_ bresenham ellipses/circle ignore alpha
-_ strokeWeight != 1 then shouldn't use flat_rect or flat_ellipse
+_ problems for beautify:
+_   //for (int i = 0; i < 10; i++) {
+_   } catch (IOException e) { }
+_   need to ignore comments.. but not those inside quotes.. oog
+_ add ESC to font builder, and find
+_ font builder should open back up with the same selection
+_   maybe just hide it, just like find
+_ font builder should update when the size for the font is changed
+_  (so long as the size itself is valid)
+_ why aren't cursors working on the mac
+_ clearing the text area on the mac
 
 
 DEVELOPERS
 _ try ariel's Thread.yield() suggestion
-_ set default framerate of 24? 30?
+_ set default framerate of 24? 30? 2x that?
 
 
 MEDIUM
@@ -18,6 +33,8 @@ _ jikes errors are missing newline/linefeed ?
 _ using run-expert.bat on 62 causes NullPointerException weirdness (?)
 _ network/shared_canvas, network/value has old code
 _ write script to remove .DS_Store and CVS folders from dist
+_   # remove DS_Store files
+_   sudo find . -name ".DS_Store" -depth -exec rm {} \;
 _ change run.bat, run-expert.bat to use 128m
 _ ability to include other .java and .pde code from sketchbook folder
 _   'add files' for .java or .pde pulls into the folder
@@ -35,6 +52,8 @@ _ multi-line errors a mess in jikes
 _   maybe a dropdown list thing, with the first just shown?
 _ processing.exe: problem if expert version is run, and no java installed
 _   call the person a genius and tell them to install java
+_ update illustrator code to use bagel api
+_   even if not actually working properly.. just in naming of things
 
 
 application export