{"action":"created","issue":{"url":"https://api.github.com/repos/endless-sky/endless-sky/issues/345","labels_url":"https://api.github.com/repos/endless-sky/endless-sky/issues/345/labels{/name}","comments_url":"https://api.github.com/repos/endless-sky/endless-sky/issues/345/comments","events_url":"https://api.github.com/repos/endless-sky/endless-sky/issues/345/events","html_url":"https://github.com/endless-sky/endless-sky/pull/345","id":119867443,"number":345,"title":"Visual indication of whether all planets in a system have been visited","user":{"login":"guillochon","id":4998058,"avatar_url":"https://avatars.githubusercontent.com/u/4998058?v=3","gravatar_id":"","url":"https://api.github.com/users/guillochon","html_url":"https://github.com/guillochon","followers_url":"https://api.github.com/users/guillochon/followers","following_url":"https://api.github.com/users/guillochon/following{/other_user}","gists_url":"https://api.github.com/users/guillochon/gists{/gist_id}","starred_url":"https://api.github.com/users/guillochon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/guillochon/subscriptions","organizations_url":"https://api.github.com/users/guillochon/orgs","repos_url":"https://api.github.com/users/guillochon/repos","events_url":"https://api.github.com/users/guillochon/events{/privacy}","received_events_url":"https://api.github.com/users/guillochon/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2015-12-02T04:52:50Z","updated_at":"2015-12-10T18:14:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/endless-sky/endless-sky/pulls/345","html_url":"https://github.com/endless-sky/endless-sky/pull/345","diff_url":"https://github.com/endless-sky/endless-sky/pull/345.diff","patch_url":"https://github.com/endless-sky/endless-sky/pull/345.patch"},"body":"**Update**: Now that the visitedPlanets variable has been added to the master branch, this PR simply adds the visual indication that all planets in a system have been visited.\r\n\r\nI've added a \"visitedPlanets\" variable to the PlayerInfo structure that keeps track of all the planets the player has visited. This data is also saved and loaded to the pilot file.\r\n\r\nSystems for which all planets have been visited will appear as filled circles on the map, whereas those in which at least one planet has not been visited will be continue to be shown as open circles. Uninhabited systems with no landable objects will appear filled once they are visited.\r\n\r\nI think this is very useful information to show to the player, especially if one is interested in exploring all corners of the galaxy (and remembering where one has actually seen all there is to see!). Here is what it looks like:\r\n\r\n![image](https://cloud.githubusercontent.com/assets/4998058/11537838/0062ff08-98ed-11e5-8107-cca40e5bc972.png)\r\n\r\nI've created a poll to vote on whether this PR should be merged, needs some more work, or should not be merged: http://poll.lab.io/ic7wCN0vdhmIwa-eyV6fbA\r\n"},"comment":{"url":"https://api.github.com/repos/endless-sky/endless-sky/issues/comments/163706795","html_url":"https://github.com/endless-sky/endless-sky/pull/345#issuecomment-163706795","issue_url":"https://api.github.com/repos/endless-sky/endless-sky/issues/345","id":163706795,"user":{"login":"frameworld","id":875554,"avatar_url":"https://avatars.githubusercontent.com/u/875554?v=3","gravatar_id":"","url":"https://api.github.com/users/frameworld","html_url":"https://github.com/frameworld","followers_url":"https://api.github.com/users/frameworld/followers","following_url":"https://api.github.com/users/frameworld/following{/other_user}","gists_url":"https://api.github.com/users/frameworld/gists{/gist_id}","starred_url":"https://api.github.com/users/frameworld/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/frameworld/subscriptions","organizations_url":"https://api.github.com/users/frameworld/orgs","repos_url":"https://api.github.com/users/frameworld/repos","events_url":"https://api.github.com/users/frameworld/events{/privacy}","received_events_url":"https://api.github.com/users/frameworld/received_events","type":"User","site_admin":false},"created_at":"2015-12-10T18:14:38Z","updated_at":"2015-12-10T18:14:38Z","body":"* Needs more work.\r\n\r\n1 players should get a clue that this feature exists and how they can extract information from it.\r\n\r\n2 Some missions only appear with a combat rating of 8000+ do I really have explored Tarazed to it's fullest extend when visiting it with a combat rating of just 7000?\r\n\r\n3 The circle may have multiple dots - one per planet.\r\n\r\n4 Colour-coding has issues.\r\n\r\nPlanet and system colours should be separate. Somehow. They don't mean the same."}}