Date: 2009-11-17 10:24:33 -0800 (Tue, 17 Nov 2009)
Cleaning up some FIXMEs assigned to me in the code:
In UIMC, we now call invalidateParentSizeAndDL() on setting explicitMaxWidth. We already do this for explicitMaxHeight. I also changed a FIXME around invalidateLayering() in to a TODO.
In GroupBase, I removed a FIXME around the focusPane as we are using the overlay API now.
In DataGroup, I changed a FIXME to a TODO with some clarifying comments.
In SkinnableContainer, I removed a FIXME and just commented on why it was needed. I also removed another FIXME and some code now that it seems like we don?\226?\128?\153t need to do the array conversion with IDefferedInstance since the compiler handles it for us now.
In SkinnableDataContainer, I changed a FIXME around event delegation to a TODO.
In DropDownController, when you drag around the volume bar, the popup stays open even if the mouse isn?\226?\128?\153t over the open button or the dropdown. However, when the mouse is released, we should immediately close the drop down, rather than wait for another mouseMove to occur. This involved some refactoring of some code in to a private helper method. The "@private" protected methods in DropDownController should probably be made mx_internal. I will do that in a subsequent checkin.
QE notes: -
Doc notes: -
Reviewer: Jason for the DropDownController change, Glenn for the rest
Tests run: checkintests, mustella FCK, Panel, SkinnableContainer, DropDownList
Is noteworthy for integration: No