{"action":"created","issue":{"url":"https://api.github.com/repos/terrestris/BasiGX/issues/36","labels_url":"https://api.github.com/repos/terrestris/BasiGX/issues/36/labels{/name}","comments_url":"https://api.github.com/repos/terrestris/BasiGX/issues/36/comments","events_url":"https://api.github.com/repos/terrestris/BasiGX/issues/36/events","html_url":"https://github.com/terrestris/BasiGX/pull/36","id":122055033,"number":36,"title":"Read layer properties for hovering from fields","user":{"login":"marcjansen","id":227934,"avatar_url":"https://avatars.githubusercontent.com/u/227934?v=3","gravatar_id":"","url":"https://api.github.com/users/marcjansen","html_url":"https://github.com/marcjansen","followers_url":"https://api.github.com/users/marcjansen/followers","following_url":"https://api.github.com/users/marcjansen/following{/other_user}","gists_url":"https://api.github.com/users/marcjansen/gists{/gist_id}","starred_url":"https://api.github.com/users/marcjansen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marcjansen/subscriptions","organizations_url":"https://api.github.com/users/marcjansen/orgs","repos_url":"https://api.github.com/users/marcjansen/repos","events_url":"https://api.github.com/users/marcjansen/events{/privacy}","received_events_url":"https://api.github.com/users/marcjansen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-12-14T14:38:29Z","updated_at":"2015-12-14T15:01:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/terrestris/BasiGX/pulls/36","html_url":"https://github.com/terrestris/BasiGX/pull/36","diff_url":"https://github.com/terrestris/BasiGX/pull/36.diff","patch_url":"https://github.com/terrestris/BasiGX/pull/36.patch"},"body":"This makes it easier for applications to determine the fields\r\nwhich are used to controll hovering. If, for example you want\r\nto disable hovering of a layer temporarily, you can now just\r\ndetermine the correct property, and then set the value for it\r\nto `false`.\r\n\r\nThis commit also adds basic tests that document the behaviour.\r\n\r\nOnce this is merged, we cannot 100% guarantee, that `hoverfield`\r\nand `hoverable` properties exist on our layer instances.\r\nApplications should now use the static fields from the\r\nhoverplugin.\r\n\r\nIf an application has the need for different fields for hovering\r\nthe plugin class can be extended and the statics can be reassigned\r\nto match the new environment. See the tests for an example of\r\nthis."},"comment":{"url":"https://api.github.com/repos/terrestris/BasiGX/issues/comments/164459202","html_url":"https://github.com/terrestris/BasiGX/pull/36#issuecomment-164459202","issue_url":"https://api.github.com/repos/terrestris/BasiGX/issues/36","id":164459202,"user":{"login":"weskamm","id":1381363,"avatar_url":"https://avatars.githubusercontent.com/u/1381363?v=3","gravatar_id":"","url":"https://api.github.com/users/weskamm","html_url":"https://github.com/weskamm","followers_url":"https://api.github.com/users/weskamm/followers","following_url":"https://api.github.com/users/weskamm/following{/other_user}","gists_url":"https://api.github.com/users/weskamm/gists{/gist_id}","starred_url":"https://api.github.com/users/weskamm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/weskamm/subscriptions","organizations_url":"https://api.github.com/users/weskamm/orgs","repos_url":"https://api.github.com/users/weskamm/repos","events_url":"https://api.github.com/users/weskamm/events{/privacy}","received_events_url":"https://api.github.com/users/weskamm/received_events","type":"User","site_admin":false},"created_at":"2015-12-14T15:01:39Z","updated_at":"2015-12-14T15:01:39Z","body":"lgtm, please merge!"}}