{"action":"created","issue":{"url":"https://api.github.com/repos/tomchadwin/qgis2web/issues/501","repository_url":"https://api.github.com/repos/tomchadwin/qgis2web","labels_url":"https://api.github.com/repos/tomchadwin/qgis2web/issues/501/labels{/name}","comments_url":"https://api.github.com/repos/tomchadwin/qgis2web/issues/501/comments","events_url":"https://api.github.com/repos/tomchadwin/qgis2web/issues/501/events","html_url":"https://github.com/tomchadwin/qgis2web/pull/501","id":233324800,"number":501,"title":"Oversized clusters","user":{"login":"OlaKov","id":28122345,"avatar_url":"https://avatars3.githubusercontent.com/u/28122345?v=3","gravatar_id":"","url":"https://api.github.com/users/OlaKov","html_url":"https://github.com/OlaKov","followers_url":"https://api.github.com/users/OlaKov/followers","following_url":"https://api.github.com/users/OlaKov/following{/other_user}","gists_url":"https://api.github.com/users/OlaKov/gists{/gist_id}","starred_url":"https://api.github.com/users/OlaKov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OlaKov/subscriptions","organizations_url":"https://api.github.com/users/OlaKov/orgs","repos_url":"https://api.github.com/users/OlaKov/repos","events_url":"https://api.github.com/users/OlaKov/events{/privacy}","received_events_url":"https://api.github.com/users/OlaKov/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":31,"created_at":"2017-06-02T23:23:54Z","updated_at":"2017-06-10T17:36:05Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tomchadwin/qgis2web/pulls/501","html_url":"https://github.com/tomchadwin/qgis2web/pull/501","diff_url":"https://github.com/tomchadwin/qgis2web/pull/501.diff","patch_url":"https://github.com/tomchadwin/qgis2web/pull/501.patch"},"body":"Hi Tom,\r\n\r\nI made few changes to the `olStyleScripts.py` to fix the following:\r\n\r\n- Oversize cluster issue – clustering is now restricted to `30px`.\r\n- Labels did not show up for a single data point within a clustered layer – this is now fixed.\r\n\r\nThe `olStyleScripts.py` now generates different `_style.js` files, based on if the layer is clustered or not. It seemed to make more sense to check if the layer is clustered on the backend. \r\n\r\nPlease note that categorized, graduated or rule based layers are currently not set to cluster. For example, in the `olLayerScripts.py` file, only single symbol layers are included:\r\n```\r\nif (cluster and isinstance(renderer, QgsSingleSymbolRendererV2)):\r\n    cluster = True\r\n```\r\nI noticed an error in on `Line 458` where `if props[\"style\"] == \"no\":`  was showing me  `KeyError: 'style'`, I changed this to `props[\"color\"] == \"no\":`\r\n\r\nPlease check it over and let me know if anything needs to be changed!\r\n\r\n![clusters](https://cloud.githubusercontent.com/assets/28122345/26748059/b4fe3b9c-47b7-11e7-84ce-19fb92de8ceb.png)\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/tomchadwin/qgis2web/issues/comments/307579525","html_url":"https://github.com/tomchadwin/qgis2web/pull/501#issuecomment-307579525","issue_url":"https://api.github.com/repos/tomchadwin/qgis2web/issues/501","id":307579525,"user":{"login":"OlaKov","id":28122345,"avatar_url":"https://avatars3.githubusercontent.com/u/28122345?v=3","gravatar_id":"","url":"https://api.github.com/users/OlaKov","html_url":"https://github.com/OlaKov","followers_url":"https://api.github.com/users/OlaKov/followers","following_url":"https://api.github.com/users/OlaKov/following{/other_user}","gists_url":"https://api.github.com/users/OlaKov/gists{/gist_id}","starred_url":"https://api.github.com/users/OlaKov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OlaKov/subscriptions","organizations_url":"https://api.github.com/users/OlaKov/orgs","repos_url":"https://api.github.com/users/OlaKov/repos","events_url":"https://api.github.com/users/OlaKov/events{/privacy}","received_events_url":"https://api.github.com/users/OlaKov/received_events","type":"User","site_admin":false},"created_at":"2017-06-10T17:36:05Z","updated_at":"2017-06-10T17:36:05Z","body":"Yep, looks good! I think  2 of the travis tests passed, but the last one got interrupted by the coveralls or something.."}}
{"actor":{"display_login":"OlaKov"}}