{"action":"created","issue":{"url":"https://api.github.com/repos/Leaflet/Leaflet.markercluster/issues/603","labels_url":"https://api.github.com/repos/Leaflet/Leaflet.markercluster/issues/603/labels{/name}","comments_url":"https://api.github.com/repos/Leaflet/Leaflet.markercluster/issues/603/comments","events_url":"https://api.github.com/repos/Leaflet/Leaflet.markercluster/issues/603/events","html_url":"https://github.com/Leaflet/Leaflet.markercluster/pull/603","id":117747330,"number":603,"title":"Leaflet master merge corrected test suites","user":{"login":"ghybs","id":12232803,"avatar_url":"https://avatars.githubusercontent.com/u/12232803?v=3","gravatar_id":"","url":"https://api.github.com/users/ghybs","html_url":"https://github.com/ghybs","followers_url":"https://api.github.com/users/ghybs/followers","following_url":"https://api.github.com/users/ghybs/following{/other_user}","gists_url":"https://api.github.com/users/ghybs/gists{/gist_id}","starred_url":"https://api.github.com/users/ghybs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ghybs/subscriptions","organizations_url":"https://api.github.com/users/ghybs/orgs","repos_url":"https://api.github.com/users/ghybs/repos","events_url":"https://api.github.com/users/ghybs/events{/privacy}","received_events_url":"https://api.github.com/users/ghybs/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-11-19T05:59:38Z","updated_at":"2015-12-21T21:46:14Z","closed_at":"2015-12-21T21:46:02Z","pull_request":{"url":"https://api.github.com/repos/Leaflet/Leaflet.markercluster/pulls/603","html_url":"https://github.com/Leaflet/Leaflet.markercluster/pull/603","diff_url":"https://github.com/Leaflet/Leaflet.markercluster/pull/603.diff","patch_url":"https://github.com/Leaflet/Leaflet.markercluster/pull/603.patch"},"body":"Hi,\r\n\r\nI updated all test suites so that they fit now Leaflet 1.0.0-beta.2.\r\nThe main difference is in vector layers (circles, circleMarkers) HTML Elements not having the same inserting path in DOM.\r\nI also took the opportunity to change the test suites to avoid creating maps between each test, which takes up PhantomJS memory and can make it hang in the long run.\r\n\r\nI also found an issue in Spiderfier when unspiderfying `L.Circle`'s: since they have a `_latLng`, they can be clustered and spiderfied, but unlike `L.Marker`'s, they do not have the `setOpacity` nor `setZindexOffset` methods. Simply added a check for instance type.\r\n\r\nI also added the latest commit.\r\n\r\nFinally, I updated all example pages to use Leaflet 1.0.0-beta.2 assets (CSS, JS and -src.js).\r\n\r\nNow all tests should pass, and Leaflet-master-merge could be merged into Leaflet-master."},"comment":{"url":"https://api.github.com/repos/Leaflet/Leaflet.markercluster/issues/comments/166433005","html_url":"https://github.com/Leaflet/Leaflet.markercluster/pull/603#issuecomment-166433005","issue_url":"https://api.github.com/repos/Leaflet/Leaflet.markercluster/issues/603","id":166433005,"user":{"login":"danzel","id":393086,"avatar_url":"https://avatars.githubusercontent.com/u/393086?v=3","gravatar_id":"","url":"https://api.github.com/users/danzel","html_url":"https://github.com/danzel","followers_url":"https://api.github.com/users/danzel/followers","following_url":"https://api.github.com/users/danzel/following{/other_user}","gists_url":"https://api.github.com/users/danzel/gists{/gist_id}","starred_url":"https://api.github.com/users/danzel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danzel/subscriptions","organizations_url":"https://api.github.com/users/danzel/orgs","repos_url":"https://api.github.com/users/danzel/repos","events_url":"https://api.github.com/users/danzel/events{/privacy}","received_events_url":"https://api.github.com/users/danzel/received_events","type":"User","site_admin":false},"created_at":"2015-12-21T21:46:14Z","updated_at":"2015-12-21T21:46:14Z","body":"Thanks, merged! Now to catch up all the new commits :)"}}