#7009 NORM Future : TileMorph set min height

Zarro Boogs per Child bugtracker at laptop.org
Wed Feb 25 21:35:53 EST 2009


#7009: TileMorph set min height
--------------------------------------+-------------------------------------
           Reporter:  karl            |       Owner:  etoys         
               Type:  defect          |      Status:  new           
           Priority:  normal          |   Milestone:  Future Release
          Component:  etoys-activity  |     Version:  Update.1      
         Resolution:                  |    Keywords:                
        Next_action:  review          |    Verified:  0             
Deployment_affected:                  |   Blockedby:                
           Blocking:                  |  
--------------------------------------+-------------------------------------

Comment(by ScottWallace):

 Hmm.. the following code in #attemptCleanupReporting: caught my eye:

 "fix up old watchers left in the world"
     ActiveWorld submorphs
         do: [:each | (each isAlignmentMorph
             and: [(each isKindOf: ScriptEditorMorph) not])
          ifTrue: [each
              allMorphsDo: [:i | i minCellSize: 0]]].

 I can appreciate that it's not that easy to figure out, in an old project,
 what is a "watcher" and what isn't, but this seems like a bit of a heavy-
 handed attempt, likely to break some morphs that happen to be open on the
 desktop at the time is invoked.

 In any case, if we want to incorporate the code of PasteUpMorph.kfr.2.cs,
 it would now need to be integrated manually with the changes to the same
 method that subsequently took place with update 2193scriptNameTileFixup.

-- 
Ticket URL: <http://dev.laptop.org/ticket/7009#comment:7>
One Laptop Per Child <http://laptop.org/>
OLPC bug tracking system


More information about the Bugs mailing list