ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Perform ol.Map#setTarget string-to-Element conversion in event handler

Browse Source 
With this change, ol.Map#setTarget no longer changes the type of the
value passed to it.
This commit is contained in:
Tom Payne
2013-09-03 15:18:09 +02:00
parent 85509e1a40
commit 53223be376
1 changed files with 24 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
src/ol/map.js
View File

@@ -220,6 +220,12 @@ ol.Map = function(options) {
this.viewport_.className = 'ol-touch';
}
/**
* @private
* @type {Element}
*/
this.target_ = null;
/**
* @private
* @type {Element}
@@ -451,10 +457,11 @@ ol.Map.prototype.getRenderer = function() {
/**
* Get the element in which this map is rendered.
* @return {Element|undefined} Target.
* @return {Element|string|undefined} Target.
*/
ol.Map.prototype.getTarget = function() {
return /** @type {Element|undefined} */ (this.get(ol.MapProperty.TARGET));
return /** @type {Element|string|undefined} */ (
this.get(ol.MapProperty.TARGET));
};
goog.exportProperty(
ol.Map.prototype,
@@ -743,15 +750,20 @@ ol.Map.prototype.handleSizeChanged_ = function() {
* @private
*/
ol.Map.prototype.handleTargetChanged_ = function() {
// target may be undefined, null or an Element. If it's not
// an Element we remove the viewport from the DOM. If it's
// an Element we append the viewport element to it.
// target may be undefined, null, a string or an Element.
// If it's a string we convert it to an Element before proceeding.
// If it's not now an Element we remove the viewport from the DOM.
// If it's an Element we append the viewport element to it.
var target = this.getTarget();
if (!goog.dom.isElement(target)) {
if (goog.isDef(target)) {
this.target_ = goog.dom.getElement(target);
} else {
this.target_ = null;
}
if (goog.isNull(this.target_)) {
goog.dom.removeNode(this.viewport_);
} else {
goog.asserts.assert(goog.isDefAndNotNull(target));
goog.dom.appendChild(target, this.viewport_);
goog.dom.appendChild(this.target_, this.viewport_);
}
this.updateSize();
// updateSize calls setSize, so no need to call this.render
@@ -1025,9 +1037,6 @@ goog.exportProperty(
* @param {Element|string|undefined} target Target.
*/
ol.Map.prototype.setTarget = function(target) {
if (goog.isDef(target)) {
target = goog.dom.getElement(target);
}
this.set(ol.MapProperty.TARGET, target);
};
goog.exportProperty(
@@ -1065,12 +1074,11 @@ ol.Map.prototype.unfreezeRendering = function() {
* third-party code changes the size of the map viewport.
*/
ol.Map.prototype.updateSize = function() {
var target = this.getTarget();
if (goog.isDef(target)) {
var size = goog.style.getSize(target);
this.setSize([size.width, size.height]);
} else {
if (goog.isNull(this.target_)) {
this.setSize(undefined);
} else {
var size = goog.style.getSize(this.target_);
this.setSize([size.width, size.height]);
}
};