[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