{"action":"created","issue":{"url":"https://api.github.com/repos/Baystation12/Baystation12/issues/7986","labels_url":"https://api.github.com/repos/Baystation12/Baystation12/issues/7986/labels{/name}","comments_url":"https://api.github.com/repos/Baystation12/Baystation12/issues/7986/comments","events_url":"https://api.github.com/repos/Baystation12/Baystation12/issues/7986/events","html_url":"https://github.com/Baystation12/Baystation12/pull/7986","id":55928619,"number":7986,"title":"Adds on-sprite belt icons","user":{"login":"Hubblenaut","id":6383576,"avatar_url":"https://avatars.githubusercontent.com/u/6383576?v=3","gravatar_id":"","url":"https://api.github.com/users/Hubblenaut","html_url":"https://github.com/Hubblenaut","followers_url":"https://api.github.com/users/Hubblenaut/followers","following_url":"https://api.github.com/users/Hubblenaut/following{/other_user}","gists_url":"https://api.github.com/users/Hubblenaut/gists{/gist_id}","starred_url":"https://api.github.com/users/Hubblenaut/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Hubblenaut/subscriptions","organizations_url":"https://api.github.com/users/Hubblenaut/orgs","repos_url":"https://api.github.com/users/Hubblenaut/repos","events_url":"https://api.github.com/users/Hubblenaut/events{/privacy}","received_events_url":"https://api.github.com/users/Hubblenaut/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Baystation12/Baystation12/labels/Cannot%20Merge","name":"Cannot Merge","color":"e11d21"},{"url":"https://api.github.com/repos/Baystation12/Baystation12/labels/feature","name":"feature","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-01-29T17:17:50Z","updated_at":"2015-02-16T15:28:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Baystation12/Baystation12/pulls/7986","html_url":"https://github.com/Baystation12/Baystation12/pull/7986","diff_url":"https://github.com/Baystation12/Baystation12/pull/7986.diff","patch_url":"https://github.com/Baystation12/Baystation12/pull/7986.patch"},"body":"According to [this forum page](http://baystation12.net/forums/viewtopic.php?f=5&t=11992).\r\n\r\nContains changes for engineering- and security belts. Also adds a handheld sprite for grenades.\r\n\r\nOn a different, but similar subject:\r\nAny objections or reasons against having the belt slot occupying a layer between uniform and (exo)suit? Instead of having belts constantly appear on top of jackets, labcoats, ect."},"comment":{"url":"https://api.github.com/repos/Baystation12/Baystation12/issues/comments/74524753","html_url":"https://github.com/Baystation12/Baystation12/pull/7986#issuecomment-74524753","issue_url":"https://api.github.com/repos/Baystation12/Baystation12/issues/7986","id":74524753,"user":{"login":"Hubblenaut","id":6383576,"avatar_url":"https://avatars.githubusercontent.com/u/6383576?v=3","gravatar_id":"","url":"https://api.github.com/users/Hubblenaut","html_url":"https://github.com/Hubblenaut","followers_url":"https://api.github.com/users/Hubblenaut/followers","following_url":"https://api.github.com/users/Hubblenaut/following{/other_user}","gists_url":"https://api.github.com/users/Hubblenaut/gists{/gist_id}","starred_url":"https://api.github.com/users/Hubblenaut/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Hubblenaut/subscriptions","organizations_url":"https://api.github.com/users/Hubblenaut/orgs","repos_url":"https://api.github.com/users/Hubblenaut/repos","events_url":"https://api.github.com/users/Hubblenaut/events{/privacy}","received_events_url":"https://api.github.com/users/Hubblenaut/received_events","type":"User","site_admin":false},"created_at":"2015-02-16T15:28:17Z","updated_at":"2015-02-16T15:28:17Z","body":"Can be merged again.\r\nRegarding the images not being cached: Having single items updating the belt's on-mob overlay would require storage code to be partly rewritten to allow checks for that, something I quite honestly do not want to mess with.\r\nSince we're generating just a small number of images here (7 max.) and not messing around with actual icons, the processing is client-side and not even noticeable since the operations take a few milliseconds at most.\r\n\r\nMoving the icon update code to belt.dm will also not remove the bit in update_icons.dm where it checks for a belt (and would then retrieve a cached image from a variable in the belt as a snowflakey check). Not really something I'd like, since it's just unnecessary moving around of variables.\r\n\r\nThis PR should be ready to pull as it is."}}