removed contain and replaced it with maxWidth / maxHeight

[jfboeve]

• Removed contain
• Introduced maxWidth / maxHeight
• also now properly loading canvas fonts
• changed update order
• overwriting some CoreNode functions to make them do less work than needed.

[wouterlucas]

Whats the driver / rationale to remove contain? does maxWidth / maxHeight provide more control?
Other then its easier to understand.

Does this align with the Blits text syntax / implementation @michielvandergeest ?

[jfboeve]

Whats the driver / rationale to remove contain? does maxWidth / maxHeight provide more control? Other then its easier to understand.

Does this align with the Blits text syntax / implementation @michielvandergeest ?

Here's the Blits docs for text. maxWidth and maxHeight are already implemented, but underwater its normalized to contain and width and height and stuff.

Removing contain in favor of maxWidth/maxHeight makes removes the checks for maxheight and width checks in the generating phase of the Text. This also simplifies adjusting the textnode width / height according what it's rendering position will be, this also means we can more easily dynamically change the maxwidth / maxheight, which would not be possible with the current width/height/contain setup.

[wouterlucas]

nice ok - thanks for elaborating!