[Cuis-dev] minimumExtent

[Juan Vuletich]

Hi Ken,

Testing with your packages, and following your video (btw, both video 
and audio are now excellent!), I fond yet a couple of issues, hence this 
addendum. I believe these are good now.

Thanks,

On 3/8/2020 3:55 PM, Juan Vuletich via Cuis-dev wrote:
> Hi Ken,
>
> I took a look at your first version. When looking at the existing code 
> in the base image, I found some details I didn't quite like, so I 
> couldn't help fixing them, and came up with a slightly different version.
>
> I didn't yet have time to check Morphic-Misc1, but I see today you 
> sent another version. So, it makes sense to give you my take on this, 
> so you can take a look. I'm not really sure if any changes are needed 
> to Morphic-Misc1, but please give this a try.
>
> Thanks!
>
> On 3/7/2020 8:41 PM, ken.dickey--- via Cuis-dev wrote:
>> Hi Juan,
>>
>> I have refined a few things and (using #fileIn rather than #install) 
>> created a single, unified changeSet.
>>
>> One issue I have is that of SystemWindow.
>>
>> I added a #SystemWindow>>minimumExtent which composes extents of its 
>> label and layout, but I think it would be simpler to just answer a 
>> reasonable extent and let subclasses override if they want something 
>> smaller.
>>
>> If you take a look at the IEDictWindow Class in 'IA-EN-Dictionary', I 
>> just do the simple thing at the top/window level rather than using 
>> LayoutSpec's and this seems simpler to me.  (pull the latest)
>>
>> I do like protecting Browser windows from resizing too small as this 
>> currently looks wacky to me.
>>
>> I think setting a reasonable #SystemWindow>>minimumExtent would do 
>> the job simply and with less computation.
>>
>> Anyway, when you get a chance to play with this, I'd appreciate your 
>> feedback.
>>
>> ..And thanks once again for Cuis!!!
>> -KenD
>
>


-- 
Juan Vuletich
www.cuis-smalltalk.org
https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev
https://github.com/jvuletich
https://www.linkedin.com/in/juan-vuletich-75611b3
@JuanVuletich