I won’t have time to investigate it today. Rolling back to 
d3e27d65344ad27912000e1e3205b96243d8828e
fixes it.

From: Yishay Weiss<mailto:yishayj...@hotmail.com>
Sent: Monday, March 16, 2020 10:09 AM
To: dev@royale.apache.org<mailto:dev@royale.apache.org>
Subject: RE: Layouts: Changing widthChanged/heightChanged/sizeChanged event 
rules

MX|Panel

________________________________
From: Yishay Weiss <yishayj...@hotmail.com>
Sent: Monday, March 16, 2020 10:08:49 AM
To: dev@royale.apache.org <dev@royale.apache.org>
Subject: RE: Layouts: Changing widthChanged/heightChanged/sizeChanged event 
rules


I’ve just updated Royale and am seeing weird behavior in Panel layout. The 
content is taking up all the space and obliterating the title and the rest of 
the chrome elements. Not sure if it’s related to these changes. Will try to 
isolate it.



From: Alex Harui<mailto:aha...@adobe.com.INVALID>
Sent: Saturday, March 14, 2020 7:12 AM
To: dev@royale.apache.org<mailto:dev@royale.apache.org>
Subject: Layouts: Changing widthChanged/heightChanged/sizeChanged event rules



Hi,

I am wanting to change the way UIBase setWidthAndHeight dispatches its events 
to cut down on the number of layout passes.

Right now, if you change both the width and height of a component, you get 3 
events (widthChanged, heightChanged, sizeChanged).  I want to change that so 
that you only get sizeChanged.  This change would cause the "meaning" of 
widthChanged to mean that only the width changed during a resize so you can 
optimize some of the layout code and similar for heightChanged.   And thus, you 
will only ever get one event out of setWidthAndHeight, it will either be:

- "widthChanged" meaning that only the width has changed
- "heightChanged" meaning that only the height has changed
- "sizeChanged" meaning that both width and height have changed

Does anyone know of any code that would break if we made this change?  
Everywhere I look, code listens to all 3 and does not react differently so 
sometimes lays out 3 times, often once before the height has changed.

Thoughts?
-Alex


Reply via email to