{"action":"created","comment":{"url":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls/comments/56825258","id":56825258,"diff_hunk":"@@ -15,6 +15,7 @@ MousePosControl.prototype.create = function(wrapper) {\n \t\t\tvar xzy = ui.latLngToMC(event.latlng, 64);\n \t\t\tdocument.getElementById(\"mouse-move-div\").innerHTML = '<div class=\"btn-group\" role=\"group\">'\n \t\t\t+ '<button type=\"button\" class=\"btn btn-default\">' + \"X: \" + Math.round(xzy[0]) + '</button>'\n+\t\t\t+ '<button type=\"button\" class=\"btn btn-default\">' + \"Y: \" + Math.round(xzy[2]) + '</button>'","path":"src/data/template/static/js/control/mousepos.js","position":null,"original_position":4,"commit_id":"83e1c74575a54f5056b3829a7e5fa7f39316f7ee","original_commit_id":"ccbee5f0aebd1e99b88bfa65068b08e05a6fd072","user":{"login":"m0r13","id":1788152,"avatar_url":"https://avatars.githubusercontent.com/u/1788152?v=3","gravatar_id":"","url":"https://api.github.com/users/m0r13","html_url":"https://github.com/m0r13","followers_url":"https://api.github.com/users/m0r13/followers","following_url":"https://api.github.com/users/m0r13/following{/other_user}","gists_url":"https://api.github.com/users/m0r13/gists{/gist_id}","starred_url":"https://api.github.com/users/m0r13/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/m0r13/subscriptions","organizations_url":"https://api.github.com/users/m0r13/orgs","repos_url":"https://api.github.com/users/m0r13/repos","events_url":"https://api.github.com/users/m0r13/events{/privacy}","received_events_url":"https://api.github.com/users/m0r13/received_events","type":"User","site_admin":false},"body":"Alright then, let's not confuse the users much more.","created_at":"2016-03-21T13:58:18Z","updated_at":"2016-03-21T13:58:18Z","html_url":"https://github.com/mapcrafter/mapcrafter/pull/205#discussion_r56825258","pull_request_url":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls/205","_links":{"self":{"href":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls/comments/56825258"},"html":{"href":"https://github.com/mapcrafter/mapcrafter/pull/205#discussion_r56825258"},"pull_request":{"href":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls/205"}}},"pull_request":{"url":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls/205","id":62653635,"html_url":"https://github.com/mapcrafter/mapcrafter/pull/205","diff_url":"https://github.com/mapcrafter/mapcrafter/pull/205.diff","patch_url":"https://github.com/mapcrafter/mapcrafter/pull/205.patch","issue_url":"https://api.github.com/repos/mapcrafter/mapcrafter/issues/205","number":205,"state":"open","locked":false,"title":"Interface redesign","user":{"login":"jamestaylr","id":1166237,"avatar_url":"https://avatars.githubusercontent.com/u/1166237?v=3","gravatar_id":"","url":"https://api.github.com/users/jamestaylr","html_url":"https://github.com/jamestaylr","followers_url":"https://api.github.com/users/jamestaylr/followers","following_url":"https://api.github.com/users/jamestaylr/following{/other_user}","gists_url":"https://api.github.com/users/jamestaylr/gists{/gist_id}","starred_url":"https://api.github.com/users/jamestaylr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jamestaylr/subscriptions","organizations_url":"https://api.github.com/users/jamestaylr/orgs","repos_url":"https://api.github.com/users/jamestaylr/repos","events_url":"https://api.github.com/users/jamestaylr/events{/privacy}","received_events_url":"https://api.github.com/users/jamestaylr/received_events","type":"User","site_admin":false},"body":"This pull request redesigns the Mapcrafter output interface and address bugs.\r\n\r\n  - [x] Adds new styles for visual appeal\r\n  - [x] Removes unused Y coordinate\r\n  - [x] Fixes bug pertaining to `id` (`control-wrapper-x`) assignment by introducing `name()` method to subclasses\r\n  - [x] Remove unused CSS classes\r\n\r\nI'm hoping this PR can prompt a discussion. I think Mapcrafter is one of the best (if not the best) Minecraft mappers, but it could use a visual redesign. Components like the compass direction (which this PR preserves), I think, could also be removed.\r\n\r\nOne downside of the PR is that it introduces an external dependency. These styles, however, could be manually pulled out of the larger library to ensure things properly run offline, while keeping the size of the dependencies down.\r\n\r\nThanks so much for considering the PR!\r\n\r\n![Image of new interface](http://i.imgur.com/kAJA6Zn.jpg)\r\n","created_at":"2016-03-12T21:57:33Z","updated_at":"2016-03-21T13:58:18Z","closed_at":null,"merged_at":null,"merge_commit_sha":null,"assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls/205/commits","review_comments_url":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls/205/comments","review_comment_url":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls/comments{/number}","comments_url":"https://api.github.com/repos/mapcrafter/mapcrafter/issues/205/comments","statuses_url":"https://api.github.com/repos/mapcrafter/mapcrafter/statuses/83e1c74575a54f5056b3829a7e5fa7f39316f7ee","head":{"label":"jamestaylr:interface-redesign","ref":"interface-redesign","sha":"83e1c74575a54f5056b3829a7e5fa7f39316f7ee","user":{"login":"jamestaylr","id":1166237,"avatar_url":"https://avatars.githubusercontent.com/u/1166237?v=3","gravatar_id":"","url":"https://api.github.com/users/jamestaylr","html_url":"https://github.com/jamestaylr","followers_url":"https://api.github.com/users/jamestaylr/followers","following_url":"https://api.github.com/users/jamestaylr/following{/other_user}","gists_url":"https://api.github.com/users/jamestaylr/gists{/gist_id}","starred_url":"https://api.github.com/users/jamestaylr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jamestaylr/subscriptions","organizations_url":"https://api.github.com/users/jamestaylr/orgs","repos_url":"https://api.github.com/users/jamestaylr/repos","events_url":"https://api.github.com/users/jamestaylr/events{/privacy}","received_events_url":"https://api.github.com/users/jamestaylr/received_events","type":"User","site_admin":false},"repo":{"id":53720402,"name":"mapcrafter","full_name":"jamestaylr/mapcrafter","owner":{"login":"jamestaylr","id":1166237,"avatar_url":"https://avatars.githubusercontent.com/u/1166237?v=3","gravatar_id":"","url":"https://api.github.com/users/jamestaylr","html_url":"https://github.com/jamestaylr","followers_url":"https://api.github.com/users/jamestaylr/followers","following_url":"https://api.github.com/users/jamestaylr/following{/other_user}","gists_url":"https://api.github.com/users/jamestaylr/gists{/gist_id}","starred_url":"https://api.github.com/users/jamestaylr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jamestaylr/subscriptions","organizations_url":"https://api.github.com/users/jamestaylr/orgs","repos_url":"https://api.github.com/users/jamestaylr/repos","events_url":"https://api.github.com/users/jamestaylr/events{/privacy}","received_events_url":"https://api.github.com/users/jamestaylr/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/jamestaylr/mapcrafter","description":"High Performance Minecraft Map Renderer.","fork":true,"url":"https://api.github.com/repos/jamestaylr/mapcrafter","forks_url":"https://api.github.com/repos/jamestaylr/mapcrafter/forks","keys_url":"https://api.github.com/repos/jamestaylr/mapcrafter/keys{/key_id}","collaborators_url":"https://api.github.com/repos/jamestaylr/mapcrafter/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/jamestaylr/mapcrafter/teams","hooks_url":"https://api.github.com/repos/jamestaylr/mapcrafter/hooks","issue_events_url":"https://api.github.com/repos/jamestaylr/mapcrafter/issues/events{/number}","events_url":"https://api.github.com/repos/jamestaylr/mapcrafter/events","assignees_url":"https://api.github.com/repos/jamestaylr/mapcrafter/assignees{/user}","branches_url":"https://api.github.com/repos/jamestaylr/mapcrafter/branches{/branch}","tags_url":"https://api.github.com/repos/jamestaylr/mapcrafter/tags","blobs_url":"https://api.github.com/repos/jamestaylr/mapcrafter/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/jamestaylr/mapcrafter/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/jamestaylr/mapcrafter/git/refs{/sha}","trees_url":"https://api.github.com/repos/jamestaylr/mapcrafter/git/trees{/sha}","statuses_url":"https://api.github.com/repos/jamestaylr/mapcrafter/statuses/{sha}","languages_url":"https://api.github.com/repos/jamestaylr/mapcrafter/languages","stargazers_url":"https://api.github.com/repos/jamestaylr/mapcrafter/stargazers","contributors_url":"https://api.github.com/repos/jamestaylr/mapcrafter/contributors","subscribers_url":"https://api.github.com/repos/jamestaylr/mapcrafter/subscribers","subscription_url":"https://api.github.com/repos/jamestaylr/mapcrafter/subscription","commits_url":"https://api.github.com/repos/jamestaylr/mapcrafter/commits{/sha}","git_commits_url":"https://api.github.com/repos/jamestaylr/mapcrafter/git/commits{/sha}","comments_url":"https://api.github.com/repos/jamestaylr/mapcrafter/comments{/number}","issue_comment_url":"https://api.github.com/repos/jamestaylr/mapcrafter/issues/comments{/number}","contents_url":"https://api.github.com/repos/jamestaylr/mapcrafter/contents/{+path}","compare_url":"https://api.github.com/repos/jamestaylr/mapcrafter/compare/{base}...{head}","merges_url":"https://api.github.com/repos/jamestaylr/mapcrafter/merges","archive_url":"https://api.github.com/repos/jamestaylr/mapcrafter/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/jamestaylr/mapcrafter/downloads","issues_url":"https://api.github.com/repos/jamestaylr/mapcrafter/issues{/number}","pulls_url":"https://api.github.com/repos/jamestaylr/mapcrafter/pulls{/number}","milestones_url":"https://api.github.com/repos/jamestaylr/mapcrafter/milestones{/number}","notifications_url":"https://api.github.com/repos/jamestaylr/mapcrafter/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/jamestaylr/mapcrafter/labels{/name}","releases_url":"https://api.github.com/repos/jamestaylr/mapcrafter/releases{/id}","deployments_url":"https://api.github.com/repos/jamestaylr/mapcrafter/deployments","created_at":"2016-03-12T08:02:53Z","updated_at":"2016-03-12T08:02:54Z","pushed_at":"2016-03-21T00:36:13Z","git_url":"git://github.com/jamestaylr/mapcrafter.git","ssh_url":"git@github.com:jamestaylr/mapcrafter.git","clone_url":"https://github.com/jamestaylr/mapcrafter.git","svn_url":"https://github.com/jamestaylr/mapcrafter","homepage":"http://mapcrafter.org","size":6497,"stargazers_count":0,"watchers_count":0,"language":"C++","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"mapcrafter:master","ref":"master","sha":"93186799c757b9daf4a3f7ad8fdb983bdf539660","user":{"login":"mapcrafter","id":8353290,"avatar_url":"https://avatars.githubusercontent.com/u/8353290?v=3","gravatar_id":"","url":"https://api.github.com/users/mapcrafter","html_url":"https://github.com/mapcrafter","followers_url":"https://api.github.com/users/mapcrafter/followers","following_url":"https://api.github.com/users/mapcrafter/following{/other_user}","gists_url":"https://api.github.com/users/mapcrafter/gists{/gist_id}","starred_url":"https://api.github.com/users/mapcrafter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mapcrafter/subscriptions","organizations_url":"https://api.github.com/users/mapcrafter/orgs","repos_url":"https://api.github.com/users/mapcrafter/repos","events_url":"https://api.github.com/users/mapcrafter/events{/privacy}","received_events_url":"https://api.github.com/users/mapcrafter/received_events","type":"Organization","site_admin":false},"repo":{"id":7298064,"name":"mapcrafter","full_name":"mapcrafter/mapcrafter","owner":{"login":"mapcrafter","id":8353290,"avatar_url":"https://avatars.githubusercontent.com/u/8353290?v=3","gravatar_id":"","url":"https://api.github.com/users/mapcrafter","html_url":"https://github.com/mapcrafter","followers_url":"https://api.github.com/users/mapcrafter/followers","following_url":"https://api.github.com/users/mapcrafter/following{/other_user}","gists_url":"https://api.github.com/users/mapcrafter/gists{/gist_id}","starred_url":"https://api.github.com/users/mapcrafter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mapcrafter/subscriptions","organizations_url":"https://api.github.com/users/mapcrafter/orgs","repos_url":"https://api.github.com/users/mapcrafter/repos","events_url":"https://api.github.com/users/mapcrafter/events{/privacy}","received_events_url":"https://api.github.com/users/mapcrafter/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/mapcrafter/mapcrafter","description":"High Performance Minecraft Map Renderer.","fork":false,"url":"https://api.github.com/repos/mapcrafter/mapcrafter","forks_url":"https://api.github.com/repos/mapcrafter/mapcrafter/forks","keys_url":"https://api.github.com/repos/mapcrafter/mapcrafter/keys{/key_id}","collaborators_url":"https://api.github.com/repos/mapcrafter/mapcrafter/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/mapcrafter/mapcrafter/teams","hooks_url":"https://api.github.com/repos/mapcrafter/mapcrafter/hooks","issue_events_url":"https://api.github.com/repos/mapcrafter/mapcrafter/issues/events{/number}","events_url":"https://api.github.com/repos/mapcrafter/mapcrafter/events","assignees_url":"https://api.github.com/repos/mapcrafter/mapcrafter/assignees{/user}","branches_url":"https://api.github.com/repos/mapcrafter/mapcrafter/branches{/branch}","tags_url":"https://api.github.com/repos/mapcrafter/mapcrafter/tags","blobs_url":"https://api.github.com/repos/mapcrafter/mapcrafter/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/mapcrafter/mapcrafter/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/mapcrafter/mapcrafter/git/refs{/sha}","trees_url":"https://api.github.com/repos/mapcrafter/mapcrafter/git/trees{/sha}","statuses_url":"https://api.github.com/repos/mapcrafter/mapcrafter/statuses/{sha}","languages_url":"https://api.github.com/repos/mapcrafter/mapcrafter/languages","stargazers_url":"https://api.github.com/repos/mapcrafter/mapcrafter/stargazers","contributors_url":"https://api.github.com/repos/mapcrafter/mapcrafter/contributors","subscribers_url":"https://api.github.com/repos/mapcrafter/mapcrafter/subscribers","subscription_url":"https://api.github.com/repos/mapcrafter/mapcrafter/subscription","commits_url":"https://api.github.com/repos/mapcrafter/mapcrafter/commits{/sha}","git_commits_url":"https://api.github.com/repos/mapcrafter/mapcrafter/git/commits{/sha}","comments_url":"https://api.github.com/repos/mapcrafter/mapcrafter/comments{/number}","issue_comment_url":"https://api.github.com/repos/mapcrafter/mapcrafter/issues/comments{/number}","contents_url":"https://api.github.com/repos/mapcrafter/mapcrafter/contents/{+path}","compare_url":"https://api.github.com/repos/mapcrafter/mapcrafter/compare/{base}...{head}","merges_url":"https://api.github.com/repos/mapcrafter/mapcrafter/merges","archive_url":"https://api.github.com/repos/mapcrafter/mapcrafter/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/mapcrafter/mapcrafter/downloads","issues_url":"https://api.github.com/repos/mapcrafter/mapcrafter/issues{/number}","pulls_url":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls{/number}","milestones_url":"https://api.github.com/repos/mapcrafter/mapcrafter/milestones{/number}","notifications_url":"https://api.github.com/repos/mapcrafter/mapcrafter/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/mapcrafter/mapcrafter/labels{/name}","releases_url":"https://api.github.com/repos/mapcrafter/mapcrafter/releases{/id}","deployments_url":"https://api.github.com/repos/mapcrafter/mapcrafter/deployments","created_at":"2012-12-23T19:19:31Z","updated_at":"2016-03-17T10:23:51Z","pushed_at":"2016-03-21T00:36:14Z","git_url":"git://github.com/mapcrafter/mapcrafter.git","ssh_url":"git@github.com:mapcrafter/mapcrafter.git","clone_url":"https://github.com/mapcrafter/mapcrafter.git","svn_url":"https://github.com/mapcrafter/mapcrafter","homepage":"http://mapcrafter.org","size":8082,"stargazers_count":165,"watchers_count":165,"language":"C++","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":45,"mirror_url":null,"open_issues_count":33,"forks":45,"open_issues":33,"watchers":165,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls/205"},"html":{"href":"https://github.com/mapcrafter/mapcrafter/pull/205"},"issue":{"href":"https://api.github.com/repos/mapcrafter/mapcrafter/issues/205"},"comments":{"href":"https://api.github.com/repos/mapcrafter/mapcrafter/issues/205/comments"},"review_comments":{"href":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls/205/comments"},"review_comment":{"href":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/mapcrafter/mapcrafter/pulls/205/commits"},"statuses":{"href":"https://api.github.com/repos/mapcrafter/mapcrafter/statuses/83e1c74575a54f5056b3829a7e5fa7f39316f7ee"}}}}