{"comment":{"url":"https://api.github.com/repos/mupen64plus-ae/mupen64plus-ae/comments/10092997","html_url":"https://github.com/mupen64plus-ae/mupen64plus-ae/commit/b8e62d5c1bda4e0ae6998eb968b00b42bcb9bb61#commitcomment-10092997","id":10092997,"user":{"login":"littleguy77","id":2266445,"avatar_url":"https://avatars.githubusercontent.com/u/2266445?v=3","gravatar_id":"","url":"https://api.github.com/users/littleguy77","html_url":"https://github.com/littleguy77","followers_url":"https://api.github.com/users/littleguy77/followers","following_url":"https://api.github.com/users/littleguy77/following{/other_user}","gists_url":"https://api.github.com/users/littleguy77/gists{/gist_id}","starred_url":"https://api.github.com/users/littleguy77/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/littleguy77/subscriptions","organizations_url":"https://api.github.com/users/littleguy77/orgs","repos_url":"https://api.github.com/users/littleguy77/repos","events_url":"https://api.github.com/users/littleguy77/events{/privacy}","received_events_url":"https://api.github.com/users/littleguy77/received_events","type":"User","site_admin":false},"position":33,"line":157,"path":"src/paulscode/android/mupen64plusae/GalleryActivity.java","commit_id":"b8e62d5c1bda4e0ae6998eb968b00b42bcb9bb61","created_at":"2015-03-09T05:33:42Z","updated_at":"2015-03-09T05:33:42Z","body":"I noticed that the use of `metrics.density` was removed in this commit.  Was that change intentional?  It seems we are mixing pixels and dip in some of the calculations above.  For example, width was previously in dip.  Now it is calculated using a mix of pixels (`metrics.widthPixels`) and dip (`galleryHalfSpacing`)."}}