Yes, I want to get the effect you're talking about - the 9999 won't do
if it visually horizontally stretches the item morph. As Andreas says,
this patch doesn't suffice because the selection is made harder, but
more ideas are welcome.

In the mean time, I'm focusing on simplifying the code, with minimal 
functional changes, and those focused on generalizing to the point 
that we can use other display morphs, at which point we can run 
into the width issue (and such) more concretely.

See new version.

Daniel

Eddie Cottongim <cottonsqueak at earthlink.net> wrote:
> This is a multi-part message in MIME format.
> 
> --Boundary_(ID_O2Xh7p4BevN+BYPDWGqIfw)
> Content-type: text/plain; charset=iso-8859-1
> Content-transfer-encoding: 7BIT
> 
> Changeset looks good so far.
> 
> While we're looking at this area, here's a suggestion. Can we get rid of the
> resizing of every morph to be 9999 wide? Other than being ugly, there could
> be problems if we actually start using list items other than StringMorphs -
> they may not appreciate being resized grossly. It will also make list
> initializing a touch faster.
> 
> Attached is a first shot at doing so - its meant to go on top of Daniel's
> changeset, not the in-image version, though it could be done there, too.
> 
> Eddie
> 
> ----- Original Message -----
> From: "Daniel Vainsencher" <danielv at netvision.net.il>
> To: <squeak-dev at lists.squeakfoundation.org>
> Sent: Saturday, August 09, 2003 10:28 AM
> Subject: [ENH][Refactor] PLMCleanUpStart
> 
> 
> > "Change Set: PLMCleanUpStart
> > Date: 9 August 2003
> > Author: Daniel Vainsencher
> >
> > This changeset adds comments for all PLM subclasses.
> >
> > Also removes the assumption of uniform height in PluggableListMorph (not
> > yet in subclasses), and refactors the morph generation. Might slow list
> > update down a bit.
> >
> > Factors out the StringMorph creation and initialization logic to
> > simplify the methods and reduce duplication.
> >
> > The goal is to introduce the ability to accept items other than
> > String/Text/MethodReference, and display them as morphs other than
> > StringMorphs.
> >
> > Patches/advice about how to handle PLM variants welcome."!
> >
> 
> 
> ----------------------------------------------------------------------------
> ----
> 
> 
> 
> --Boundary_(ID_O2Xh7p4BevN+BYPDWGqIfw)
> Content-type: application/octet-stream; name=PLMNoResizing.1.cs.gz
> Content-transfer-encoding: base64
> Content-disposition: attachment; filename=PLMNoResizing.1.cs.gz
> 
> H4sIAAAAAAAAAK1VS2/bRhA+88D/MFIOtIBCsaOoiYn04CpV0cIvRAJyEBRjyR2KCy93lX3IUZEf
> 39mlZYl+9FD4JGln5vu+eSqbGt3A7LtHdjsa/lqgY6AryLL38LeXW3h3fDzKMlhI5tA68GtOX3J4
> Mx6djpegFZzCmV95MgVPYA7G+buTfDyCdZP10v6kZmqFMEOXJ8n1+cWl/oJW/CPUKv0ckJIk6SCk
> Z97V2oT3PzgXCBPtnFYr0aTp2UYLDuY+HqSgGOGwscC9CS+NNusaSoPMCa2G/V6apr1r6VcrVkg8
> J/+L6NEgcXA7JR7IuGF3FJyBdaxZ0wNWJXx8e/o2JnTyIT/5QIkEryuVA5swtWE2Tfqf6QVcja0O
> qkT4XkbrL8AUh7W2VhRUQyathgAAlWENQoXIC1be9tM0+QmlltrM9VRI+VW4GhjJb3+Cwh8OLEos
> Qz5nZICfFGP9Gg08UjRME7JEX+RtmqK61O5SyDxNksVjmpuWmMKSSHMTiKrhc67Czg1Tdk3KlAup
> 5bA4ug+KH/pOoRmAimzLJdzVQuLceAzMz1HHqPi8DIwvaDt4kb4RiqkS4RMcD8cBVlQPDE8pGl/W
> 52JVOzTL1nlKXXjJmzNzi62U/mxX7tgIyj32dVfYdsQyC4X2ilOjKRFUnAxOQxV61mzhTnBX9wms
> 27obOOr2p8UYkL+rv4aYPHpUIBTV8/dobcEGQdrBYBDUkS2NlpIGwYXmVNo0U1rlPLZxAFKXTLYQ
> c/2n1AWTeXeUBrEWVBpFRbL0TvYdZKsskrbTFXP7Qk60zZL2/0BLLGb+uGNUzDiOFdSMFq30FvYN
> Wzyghn04gH0kYId9bbBCGr4SLdzittDM8Ig5iSPUg96r7Pl72vOahkaGwXmYgxe3ChZdw73YqImu
> FB/GVkAZLyB/VZVevZbOQtIh+l9KcROuAYVw+d+CRyQ4rg3F/UXnOgdkZQ3x9nkTn1p41uZCqsJV
> J51qg4aGk45eInZeUxY8tofpfXtifEoXlnsPmP/WZrxh0uNVdW00XVS3pf+1ADWJfpZqehET7oQS
> byTtgL1hdubCX9CVmdNFCGQxBTIF8oOf8ZPIwr7m+xQDRacWO4U2wl7c5xPAPZNtWgGK4o72ZXs5
> LVp/qbUK8Ttvgpv5IiLTDUrDjXyoaFfLM8H5vlkh2W+dgDg//wIWTvc3WQgAAA==
> 
> --Boundary_(ID_O2Xh7p4BevN+BYPDWGqIfw)
> MIME-version: 1.0
> Content-type: text/plain; charset=us-ascii
> Content-transfer-encoding: 7BIT
> 
> 
> 
> --Boundary_(ID_O2Xh7p4BevN+BYPDWGqIfw)--