[Pharo-project] A lil simplification of MorphTreeNodeMorph
alain.plantec at yahoo.com
Wed Apr 6 10:01:20 CEST 2011
Le 06/04/2011 09:30, Tudor Girba a écrit :
> 60% sounds excellent!
> If you are looking into MorphTreeMorph, would it be possible to look into the problem of making the submorphs fill the entire horizontal space?
arrrh! yes sorry,
it is on my todo list and the intergration of the cleaner Gary's
but no time for this now :(
> Let me give you an example:
> - download the Glamorous Toolkit image:
> - run GTCoder open
> - select a package, select a class, expand a method and you will see that the text submorph does not scale either horizontally, or vertically
> I also attached a screenshot.
> Would it be possible to fix that, too? :)
> On 6 Apr 2011, at 08:57, Igor Stasenko wrote:
>> On 5 April 2011 21:39, Alain Plantec<alain.plantec at yahoo.com> wrote:
>>> Le 05/04/2011 20:08, Igor Stasenko a écrit :
>>>> Cut out some code and extra morphs.
>>>> It speeds up rendering to 60% of original.
>>> thanks for reviewing this code!
>>>> Please verify if it positioning stuff correctly. Because in my image
>>>> Settings are now OK.
>>>> I need your feedback to know if it works correctly in other places,
>>>> where tree morph used.
>>> you need to remove all references to spacers inst var since it seems to be
>>> not used anymore now.
>>> another point, try all examples in Morphic-MorphTreeWidget-Examples.
>>> now, the height of each row is constant.
>> Yes, so what it should be instead?
>>> It is because of the RowLayout that is used for MorphTreeNodeMorph instead
>>> of a ProportionalLayout.
>>> see ClassTreeExample before and after your changes.
>>> ClassTreeExample new openOn: Object
>> i will
>> Best regards,
>> Igor Stasenko AKA sig.
> "Reasonable is what we are accustomed with."
More information about the Pharo-project