[Cuis-dev] BoxedMorph padding

ken.dickey at whidbey.com ken.dickey at whidbey.com
Sat Sep 7 16:30:32 PDT 2024


On 2024-09-07 13:49, Juan Vuletich wrote:
..
> I think this is useful. But I'm not really happy with adding the ivar
> to BoxedMorph.
..
> So, I think it would be better to add PaddedMorph as subclass of
> BoxedMorph, with ivar 'padding'. Then we could (pehaps gradually)
> change the superclass of classes where we see the advantage, starting
> with your code to enable the feature.

I think this a good idea.  I also noted many cases where padding makes 
no sense.

Perhaps PaddedMorph subclass of BoxedMorph?

Subclasses of PaddedMorph might start with:

   ImageMorph
   LabelMorph
   LayoutMorph
   TextParagraphMorph
   InnerTextMorph

Then incrementally migrate:

   HoverHelpMorph
   MenuMorph
   PluggableMorph
   TextParagraphMorph
   TranscriptMorph

This also brings up usage and testing of PlacedMorphs [Non-BoxedMorphs] 
in Layouts.

We have not used this much and the behavior of vector-graphic Morphs 
rescaled by Layouts seem kind of random/uneven.

Perhaps chat on Wednesday?

Would it be OK if I make up a changeSet refactoring the above with 
PaddedMorph ?

Cheers,
-KenD
PS: Thanks for off-by-one display fix for ImageMorph>>drawOn: !


More information about the Cuis-dev mailing list