Thanks for working in this! =)

  Cheers,

    Eduardo

On Sat, 11 Mar 2023 at 11:48, karl ramberg <karlramberg@gmail.com> wrote:

Attached is a change set that fixes turning text wrapFlag both on and off in TextMorphForEditView.

I'm not familiar with the inner workings of these methods to say they are the correct fixes.

The end result looks right but implementation is maybe a little clumsy.

Best,

Karl

On Sat, Mar 11, 2023 at 10:44 AM karl ramberg <karlramberg@gmail.com> wrote:

When a you turn preference wrapFlag off in TextMorphForEditViewyou get

Assertion failure in Morph>>#doLayoutIn:

...

"2.2) Do one additional run on the layout if requested in #layoutInBounds:."
(self hasProperty: #doLayoutAgain) ifTrue: [
self removeProperty: #doLayoutAgain.
layout flushLayoutCache.
layout layout: self in: layoutBounds].
self assert: (self hasProperty: #doLayoutAgain) not].

...

But redoing the layout can add the property #doLayoutAgain.

I think this has to be changed to :

...

"2.2) Do one additional run on the layout if requested in #layoutInBounds:."
(self hasProperty: #doLayoutAgain) ifTrue: [
layout flushLayoutCache.
layout layout: self in: layoutBounds.
self removeProperty: #doLayoutAgain.].
self assert: [(self hasProperty: #doLayoutAgain) not]].

...

Best,

Karl