diff --git a/NOTES.md b/NOTES.md index bd65ad544..33fa38abe 100644 --- a/NOTES.md +++ b/NOTES.md @@ -211,3 +211,9 @@ in Firefox but is in webkit so we need to use spaces. And trailing spaces are not included in getComputedTextLength: * https://bugzilla.mozilla.org/show_bug.cgi?id=346694 + +webkit doesn't let querySelectorAll select camelcase elements: + +* https://bugs.webkit.org/show_bug.cgi?id=46800 +* https://bugs.webkit.org/show_bug.cgi?id=83438 +* https://github.com/mbostock/d3/issues/925 diff --git a/js/id/format/xml.js b/js/id/format/xml.js index cf8b4111e..920ea3933 100644 --- a/js/id/format/xml.js +++ b/js/id/format/xml.js @@ -69,7 +69,6 @@ iD.format.XML = { }).map(iD.format.XML.rep) } }; - console.log(rep); return (new XMLSerializer()).serializeToString(JXON.unbuild(rep)); }, reps: { diff --git a/js/id/renderer/style.js b/js/id/renderer/style.js index 8749cc1e3..c9d2bd521 100644 --- a/js/id/renderer/style.js +++ b/js/id/renderer/style.js @@ -48,16 +48,10 @@ iD.Style.markerimage = function(d) { return 'icons/unknown.png'; }; -iD.Style.TAG_CLASSES = { - 'highway': true, - 'railway': true, - 'motorway': true, - 'amenity': true, - 'landuse': true, - 'building': true, - 'oneway': true, - 'bridge': true -}; +iD.Style.TAG_CLASSES = iD.Util.trueObj([ + 'highway', 'railway', 'motorway', 'amenity', + 'landuse', 'building', 'oneway', 'bridge' +]); iD.Style.styleClasses = function(pre) { return function(d) { diff --git a/js/id/renderer/tiles.js b/js/id/renderer/tiles.js index 680df78b8..527e9092d 100644 --- a/js/id/renderer/tiles.js +++ b/js/id/renderer/tiles.js @@ -1,4 +1,3 @@ -// a minimal map tile client, to be turned on and off etc. iD.Tiles = function(selection, projection) { var t = {}, tile = d3.geo.tile(); @@ -18,7 +17,6 @@ iD.Tiles = function(selection, projection) { return 'http://ecn.t' + t + '.tiles.virtualearth.net/tiles/a' + u + '.jpeg?g=587&mkt=en-gb&n=z'; } - // derive the tiles onscreen, remove those offscreen and position tiles // correctly for the currentstate of `projection` function redraw() {