Merge pull request #432 from ahocevar/iframe-opacity

Long URLs messing up Opacity. r=@elemoine
This commit is contained in:
ahocevar
2012-04-23 14:14:08 -07:00
3 changed files with 8 additions and 3 deletions

View File

@@ -1286,7 +1286,12 @@ OpenLayers.Layer = OpenLayers.Class({
this.opacity = opacity;
var childNodes = this.div.childNodes;
for(var i = 0, len = childNodes.length; i < len; ++i) {
var element = childNodes[i].firstChild || childNodes[i];
var element = childNodes[i].lastChild || childNodes[i];
if (element.nodeName.toLowerCase() === "iframe") {
// changing opacity on IFrame elements does not work, so
// we set it on the parent
element = element.parentNode;
}
OpenLayers.Util.modifyDOMElement(element, null, null, null,
null, null, null, opacity);
}

View File

@@ -470,7 +470,7 @@ OpenLayers.Renderer.SVG = OpenLayers.Class(OpenLayers.Renderer.Elements, {
*/
createDefs: function() {
var defs = this.nodeFactory(this.container.id + "_defs", "defs");
this.rendererRoot.appendChild(defs);
this.rendererRoot.insertBefore(defs, this.root);
return defs;
},

View File

@@ -5256,7 +5256,7 @@ OpenLayers.Renderer.SVG2 = OpenLayers.Class(OpenLayers.Renderer.NG, {
*/
createDefs: function() {
var defs = this.nodeFactory(this.container.id + "_defs", "defs");
this.rendererRoot.appendChild(defs);
this.rendererRoot.insertBefore(defs, this.root);
return defs;
},