{"action":"created","issue":{"url":"https://api.github.com/repos/w3c/manifest/issues/450","repository_url":"https://api.github.com/repos/w3c/manifest","labels_url":"https://api.github.com/repos/w3c/manifest/issues/450/labels{/name}","comments_url":"https://api.github.com/repos/w3c/manifest/issues/450/comments","events_url":"https://api.github.com/repos/w3c/manifest/issues/450/events","html_url":"https://github.com/w3c/manifest/issues/450","id":147329315,"number":450,"title":"Drop density descriptor from spec","user":{"login":"marcoscaceres","id":870154,"avatar_url":"https://avatars.githubusercontent.com/u/870154?v=3","gravatar_id":"","url":"https://api.github.com/users/marcoscaceres","html_url":"https://github.com/marcoscaceres","followers_url":"https://api.github.com/users/marcoscaceres/followers","following_url":"https://api.github.com/users/marcoscaceres/following{/other_user}","gists_url":"https://api.github.com/users/marcoscaceres/gists{/gist_id}","starred_url":"https://api.github.com/users/marcoscaceres/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marcoscaceres/subscriptions","organizations_url":"https://api.github.com/users/marcoscaceres/orgs","repos_url":"https://api.github.com/users/marcoscaceres/repos","events_url":"https://api.github.com/users/marcoscaceres/events{/privacy}","received_events_url":"https://api.github.com/users/marcoscaceres/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/w3c/manifest/labels/atrisk","name":"atrisk","color":"b60205"},{"url":"https://api.github.com/repos/w3c/manifest/labels/P1","name":"P1","color":"e11d21"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2016-04-11T06:38:22Z","updated_at":"2016-04-13T08:28:50Z","closed_at":null,"body":"There seems to be a lot of confusion about how to properly use the `density` member. It was designed to support discriminating against the user's screen: that is to say, if the end-user screen greater than or equal to X, then use this icon. \r\n\r\nWhen we started this effort a few years ago, this use case was driven by iOS developers that were having to create specific icons for 1x and 2x - not only for performance, but because 2x icons were getting scaled incorrectly in 1x displays. However, this is a bit of an edge case nowadays because most phones have +2x displays. \r\n\r\nBecause of misuse and confusion around this feature, we are seeing a lot that is using density as a descriptor of the icon instead. E.g., a 36x36 pixel icon is \"0.75\"... which doesn't make any sense. Anyway, this gets worst for larger icons, where some developers have specified things like \"density\": \"4\" - where those icons would be fine to use on a 2x density display. \r\n\r\nAs such, I propose we drop density from the spec.  \r\n\r\n@mounirlamouri, @kenchris, @RobDolinMS, @boyofgreen, wdyt? "},"comment":{"url":"https://api.github.com/repos/w3c/manifest/issues/comments/209301273","html_url":"https://github.com/w3c/manifest/issues/450#issuecomment-209301273","issue_url":"https://api.github.com/repos/w3c/manifest/issues/450","id":209301273,"user":{"login":"mounirlamouri","id":573590,"avatar_url":"https://avatars.githubusercontent.com/u/573590?v=3","gravatar_id":"","url":"https://api.github.com/users/mounirlamouri","html_url":"https://github.com/mounirlamouri","followers_url":"https://api.github.com/users/mounirlamouri/followers","following_url":"https://api.github.com/users/mounirlamouri/following{/other_user}","gists_url":"https://api.github.com/users/mounirlamouri/gists{/gist_id}","starred_url":"https://api.github.com/users/mounirlamouri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mounirlamouri/subscriptions","organizations_url":"https://api.github.com/users/mounirlamouri/orgs","repos_url":"https://api.github.com/users/mounirlamouri/repos","events_url":"https://api.github.com/users/mounirlamouri/events{/privacy}","received_events_url":"https://api.github.com/users/mounirlamouri/received_events","type":"User","site_admin":false},"created_at":"2016-04-13T08:28:50Z","updated_at":"2016-04-13T08:28:50Z","body":"Chrome actually uses `density` but we got a lot of issues with websites over using it. Unfortunately, it seems to all boil down to some early examples using `density` for all images in the Manifest. I have to say that I've never met anyone who actually understood `density` without me explaining :)"}}