pullup r10278 to the 2.9 branch (closes #2414)
git-svn-id: http://svn.openlayers.org/branches/openlayers/2.9@10279 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf
This commit is contained in:
@@ -93,6 +93,7 @@ OpenLayers.Element = {
|
||||
|
||||
/**
|
||||
* APIFunction: getDimensions
|
||||
* *Deprecated*. Returns dimensions of the element passed in.
|
||||
*
|
||||
* Parameters:
|
||||
* element - {DOMElement}
|
||||
|
||||
@@ -1402,11 +1402,9 @@ OpenLayers.Map = OpenLayers.Class({
|
||||
var size = new OpenLayers.Size(this.div.clientWidth,
|
||||
this.div.clientHeight);
|
||||
|
||||
// Workaround for the fact that hidden elements return 0 for size.
|
||||
if (size.w == 0 && size.h == 0 || isNaN(size.w) && isNaN(size.h)) {
|
||||
var dim = OpenLayers.Element.getDimensions(this.div);
|
||||
size.w = dim.width;
|
||||
size.h = dim.height;
|
||||
size.w = this.div.offsetWidth;
|
||||
size.h = this.div.offsetHeight;
|
||||
}
|
||||
if (size.w == 0 && size.h == 0 || isNaN(size.w) && isNaN(size.h)) {
|
||||
size.w = parseInt(this.div.style.width);
|
||||
|
||||
@@ -110,37 +110,6 @@
|
||||
t.ok(OpenLayers.Element.getHeight(elem) == elem.offsetHeight, "offsetHeight returned");
|
||||
}
|
||||
|
||||
function test_Element_getDimensions(t) {
|
||||
if (!t.open_window) {
|
||||
t.plan(0);
|
||||
return;
|
||||
} else {
|
||||
t.plan(4);
|
||||
}
|
||||
|
||||
//shown
|
||||
t.open_window( "BaseTypes/Element.html", function( wnd ) {
|
||||
t.delay_call(3, function() {
|
||||
var elem = wnd.document.getElementById("elemID");
|
||||
|
||||
var dims = OpenLayers.Element.getDimensions(elem);
|
||||
t.eq(dims.width, 50, "width correct when displayed");
|
||||
t.eq(dims.height, 100, "height correct when displayed")
|
||||
|
||||
elem.style.display = "none";
|
||||
dims = OpenLayers.Element.getDimensions(elem);
|
||||
t.eq(dims.width, 50, "width correct when hidden");
|
||||
t.eq(dims.height, 100, "height correct when hidden")
|
||||
|
||||
|
||||
});
|
||||
});
|
||||
|
||||
//hidden
|
||||
|
||||
|
||||
}
|
||||
|
||||
function test_hasClass(t) {
|
||||
t.plan(14);
|
||||
var has = OpenLayers.Element.hasClass;
|
||||
|
||||
Reference in New Issue
Block a user