Bugreport: ContentDisplay [SOLVED]

[Ahrengot]

Hey Jack!

 

I am working on another video player that implements your awesome VideoLoader, and i noticed some strange behavior.

 

Since you added the fitWidth and fitHeight setters, whenever you call them and thereby call the _update() method in ContentDisplay a new background is drawn in the ContentDisplay on top of the old one instead of replacing it. This means Content Display will report a wrong width/height, dispatch mouse events it shouldn't etc. as the background expands beyond the limits provided by the new width/height.

 

ContentDisplay.as line 110

if (_fitWidth > 0 && _fitHeight > 0) {
graphics.beginFill(_bgColor, _bgAlpha);
graphics.drawRect(left, top, _fitWidth, _fitHeight);
graphics.endFill();
}

 

This fixed it for me:

if (_fitWidth > 0 && _fitHeight > 0) {
graphics.clear();
graphics.beginFill(_bgColor, _bgAlpha);
graphics.drawRect(left, top, _fitWidth, _fitHeight);
graphics.endFill();
}

[GreenSock]

Ha ha - silly me. Forgot the graphics.clear(). Thanks for noticing that and letting me know. It's fixed in the latest version. By the way, it shouldn't be tucked inside that if() statement in case fitWidth/fitHeight are set to NaN to remove the background.

 

Snag the latest version at https://www.greensock.com/account/)

[Ahrengot]

It amazes me how fast you are at fixing and updating things like this. Many thanks