{"action":"created","issue":{"url":"https://api.github.com/repos/npct/pct-shiny/issues/137","repository_url":"https://api.github.com/repos/npct/pct-shiny","labels_url":"https://api.github.com/repos/npct/pct-shiny/issues/137/labels{/name}","comments_url":"https://api.github.com/repos/npct/pct-shiny/issues/137/comments","events_url":"https://api.github.com/repos/npct/pct-shiny/issues/137/events","html_url":"https://github.com/npct/pct-shiny/pull/137","id":134547698,"number":137,"title":"Labels consistency and info shown per scenario","user":{"login":"audev","id":7117429,"avatar_url":"https://avatars.githubusercontent.com/u/7117429?v=3","gravatar_id":"","url":"https://api.github.com/users/audev","html_url":"https://github.com/audev","followers_url":"https://api.github.com/users/audev/followers","following_url":"https://api.github.com/users/audev/following{/other_user}","gists_url":"https://api.github.com/users/audev/gists{/gist_id}","starred_url":"https://api.github.com/users/audev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/audev/subscriptions","organizations_url":"https://api.github.com/users/audev/orgs","repos_url":"https://api.github.com/users/audev/repos","events_url":"https://api.github.com/users/audev/events{/privacy}","received_events_url":"https://api.github.com/users/audev/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-02-18T11:04:24Z","updated_at":"2016-02-21T17:18:26Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/npct/pct-shiny/pulls/137","html_url":"https://github.com/npct/pct-shiny/pull/137","diff_url":"https://github.com/npct/pct-shiny/pull/137.diff","patch_url":"https://github.com/npct/pct-shiny/pull/137.patch"},"body":"@Robinlovelace   Changes following James preferences on wording and info shown. Tested yesterday with latest Bedforshire data, no issues seen.\r\n\r\nStill pretty sure still a bit more can be added re e.g. route types, so I won't close the issue yet.\r\n"},"comment":{"url":"https://api.github.com/repos/npct/pct-shiny/issues/comments/186862289","html_url":"https://github.com/npct/pct-shiny/pull/137#issuecomment-186862289","issue_url":"https://api.github.com/repos/npct/pct-shiny/issues/137","id":186862289,"user":{"login":"nikolai-b","id":6289830,"avatar_url":"https://avatars.githubusercontent.com/u/6289830?v=3","gravatar_id":"","url":"https://api.github.com/users/nikolai-b","html_url":"https://github.com/nikolai-b","followers_url":"https://api.github.com/users/nikolai-b/followers","following_url":"https://api.github.com/users/nikolai-b/following{/other_user}","gists_url":"https://api.github.com/users/nikolai-b/gists{/gist_id}","starred_url":"https://api.github.com/users/nikolai-b/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nikolai-b/subscriptions","organizations_url":"https://api.github.com/users/nikolai-b/orgs","repos_url":"https://api.github.com/users/nikolai-b/repos","events_url":"https://api.github.com/users/nikolai-b/events{/privacy}","received_events_url":"https://api.github.com/users/nikolai-b/received_events","type":"User","site_admin":false},"created_at":"2016-02-21T17:18:26Z","updated_at":"2016-02-21T17:18:26Z","body":"I was thinking it'd might be easier to keep track if we passed in named vectors into the table generating functions, e.g.\r\n```R\r\ntableCommon <- function(dataLables){\r\n  paste0('\r\n     <tr>\r\n       <td> Total commutes </td>\r\n       <td> ', dataLabels['totalCommutes'], '</td>\r\n     </tr>\r\n     <tr>\r\n       <td> Cycle commutes </td>\r\n       <td> ', dataLabels['cycleCommutesAbs'], '(', datadataLables['cycleCommutesPerc'], '&#37;) </td>\r\n     </tr>\r\n     <tr>\r\n       <td> Cycle commutes (scenario) </td>\r\n       <td> ', dataLabels['cycleCommutesAbs'], ' </td>\r\n     </tr>\r\n     <tr>\r\n       <td> Increase in commutes &nbsp; </td>\r\n       <td> ', dataLabels['cycleCommutesInc'], ' </td>\r\n     </tr>\r\n     ')\r\n}\r\n```\r\n\r\nWhich could be called with `tableCommon('totalCommutes' = 422, 'cycleCommutesAbs' = 21, ...)`  What do you think?  I like code reuse here as it means the tables always appear similar to users."}}