[squeak-dev] The Inbox: Morphic-hjh.1453.mcz

David T. Lewis lewis at mail.msen.com
Sat Jun 16 15:38:52 UTC 2018


On Sat, Jun 16, 2018 at 07:55:32PM +0530, K K Subbu wrote:
> On Saturday 16 June 2018 10:27 AM, H. Hirzel wrote:
> >After unloading Etoys 'Projects' -> 'New project' -> 'New
> >MorphicProject' no longer works.
> >
> >Etoys implements #isTileScriptingElement.  (see screen shot). After
> >unloading only one sender it left - in method
> >PasteUpMorph>>allTileScriptingElements
> 
> and in Morph>>isTileScriptingElement. Both seem to be results of 
> monkey-patching (understandable in research experiments).
> 
> Wouldn't a better idea be to subclass EtoysMorph (Morph) and EtoysWorld 
> (PasteUp) and move such methods into them? Like in EtoysPresenter?
> 
> This will require a one-time effort in changing base Etoys class defs 
> and methods. But this one-time change will payoff in terms of cleaner 
> separation between Morphic and Etoys.
> 
> Would this be too risky a change for 5.2 release?

Changing class names is likely to cause issues for project loading. It
would be a good idea to put the proposed change into the inbox, but it
would probably be best not to merge it until after the 5.2 release.

Meanwhile, the proposal of Hannes seems to be an improvement over the
current state, should that go to trunk?

Dave



More information about the Squeak-dev mailing list