[squeak-dev] ToolBuilder / Pharo 1.0 changes

Andreas Raab andreas.raab at gmx.de
Fri Apr 23 04:50:10 UTC 2010


Hi -

Here are the changes between the latest ToolBuilder from 
http://www.squeaksource.com/ToolBuilder vs. Pharo 1.0 (again there are 
additional but fairly minor edits involved):

Package ToolBuilder-Kernel
--------------------------
- ToolBuilderTests
Removed.

- UIManager
   * fontFromUser:
   --
   * chooseFrom:lines:message:
   * chooseFrom:lines:message:title:
   * chooseFrom:message:
   * chooseFrom:message:title:
   * chooseFrom:values:lines:message:
   * chooseFrom:values:lines:message:title:
   * chooseFrom:values:message:
   * chooseFrom:values:message:title:
   --
   * createPageTestWorkspace
   * checkForNewDisplaySize
   * composeFormFor:
   * grafPort
   * interactiveParserFor:
   * newDisplayDepthNoRestore:
   * onDebug:context:title:full:
   * onPrimitiveError:
   * onEventSensorStartup:
   * onSnapshot
   * restoreDisplay
   * restoreDisplayAfter:

Two sets of issues that we should discuss: If you care about unloading 
the (very small) set of tests we should move those into a separate 
package, which is easy enough.

Secondly, UIManager. It seems there have been three types of additions:

1) 'Genuine' user requests like fontFromUser:. Since we added a similar 
protocol in Squeak we should work out which protocol is preferable and 
use it (or even both).

2) Polymorph extensions. Assuming Gary agrees, we should move those into 
UIManager but I'll point out that a quick look at senders didn't reveal 
any users? All the references were between these methods no apparent 
external callers. Are these used in other packages?

3) Methods that are not about user requests (everything from 
creatPageTestWorkspace through restoreDisplayAfter:). I don't really 
like those very much (after all UIManager is all about asking users for 
information); is there really no better place in Pharo to put these? I 
can kind of see why one would stick some of those into UIManager if 
there is absolutely no other place but ... is there?

Package ToolBuilder-Morphic
---------------------------
- MorphicToolBuilderTests
Removed.

- MorphicUIManager
   * isActiveManager - presumes Morphic-only
   * (all the code for the new methods in UIManager)

Not much new stuff here.

Package Polymorph-ToolBuilder-Morphic
-------------------------------------
* PSPluggableListSpec
Adds icon and double click selector to the list spec.

This should be folded all the way up into PluggableListSpec. The lack of 
these is a real oversight.

That's about it for the Pharo changes. Except from the methods in 
UIManager that don't really belong there, there's little to worry about.

The next thing I'll do is post some recommendations of how I think we 
can move this forward. I'll have to think about this a little though.

Cheers,
   - Andreas



More information about the Squeak-dev mailing list