cartodb/Leaflet
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Make Icon/DivIcon reuse not try reuse the wrong dom element type.

Browse Source
This commit is contained in:
danzel 2013-06-12 16:05:42 +12:00
parent 7f2eff363b
commit fa45913505
4 changed files with 16 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
spec/suites/layer/marker/MarkerSpec.js
View File

@ -47,7 +47,7 @@ describe("Marker", function () {
marker.setIcon(new L.Icon({iconUrl: 'icon1.png' }));
expect(marker._icon.afterIcon.src).to.contain('icon2.png');
expect(marker._icon.src).to.contain('icon1.png');
});
it("changes an image to a DivIcon", function () {
@ -56,7 +56,7 @@ describe("Marker", function () {
marker.setIcon(new L.DivIcon({html: 'Inner1Text' }));
expect(marker._icon.afterIcon.innerHTML).to.contain('Inner1Text');
expect(marker._icon.innerHTML).to.contain('Inner1Text');
});
});

2
src/layer/marker/DivIcon.js
View File

@ -17,7 +17,7 @@ L.DivIcon = L.Icon.extend({
},
createIcon: function (oldIcon) {
var div = oldIcon || document.createElement('div'),
var div = (oldIcon && oldIcon.tagName == 'DIV') ? oldIcon : document.createElement('div'),
options = this.options;
if (options.html !== false) {

2
src/layer/marker/Icon.js
View File

@ -41,7 +41,7 @@ L.Icon = L.Class.extend({
}
var img;
if (!oldIcon) {
if (!oldIcon || oldIcon.tagName != 'IMG') {
img = this._createImg(src);
} else {
img = this._createImg(src, oldIcon);

12
src/layer/marker/Marker.js
View File

@ -109,6 +109,18 @@ L.Marker = L.Class.extend({
this._icon = options.icon.createIcon();
} else {
this._icon = this.options.icon.createIcon(this._icon);
//If the icon isn't being reused
if (reuseIcon !== this._icon) {
//Temporarily reset this._icon to be the one we want to remove so _removeIcon will remove it
var newIcon = this._icon;
this._icon = reuseIcon;
this._removeIcon();
reuseIcon = false;
this._icon = newIcon;
}
}
if (options.title) {