Skip to content

Commit

Permalink
fix #863 test.aria.popups.Popup failure in IE7
Browse files Browse the repository at this point in the history
The bean for popup configuration has also been updated.
  • Loading branch information
flongo authored and divdavem committed Nov 28, 2013
1 parent a2a3cc9 commit 69ba4ee
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/aria/popups/Beans.js
Expand Up @@ -54,7 +54,7 @@ Aria.beanDefinitions({
},
"absolutePosition" : {
$type : "AbsolutePositionConfig",
$description : "Takes priority over domReference if defined. The exact coordinates where the popup should be displayed. Anchors and offsets will still be applied",
$description : "The exact coordinates where the popup should be displayed. The position of the popup is computed firstly with respect to the domReference, if it is provided in the configuration. The computed positions are then overridden by the properties specified inside absolutePosition.",
$default : null
},
"center" : {
Expand Down
3 changes: 2 additions & 1 deletion test/aria/popups/Popup.js
Expand Up @@ -178,7 +178,7 @@ Aria.classDefinition({

var myDiv = document.createElement("div");
myDiv.id = "myDiv";
myDiv.style.cssText = "position:absolute;left:50px;top:50px;";
myDiv.style.cssText = "position:absolute;left:50px;top:50px;height:0px;";
document.body.appendChild(myDiv);

var conf = {
Expand Down Expand Up @@ -306,6 +306,7 @@ Aria.classDefinition({
this.assertTrue(evtOpenRaised, "Event popupOpen not raised");
this.assertTrue(evtCloseRaised, "Event popupClose not raised");

aria.popups.PopupManager.$unregisterListeners(this);
popup.$dispose();
}

Expand Down

0 comments on commit 69ba4ee

Please sign in to comment.