{"action":"created","issue":{"url":"https://api.github.com/repos/jlongster/debugger.html/issues/436","repository_url":"https://api.github.com/repos/jlongster/debugger.html","labels_url":"https://api.github.com/repos/jlongster/debugger.html/issues/436/labels{/name}","comments_url":"https://api.github.com/repos/jlongster/debugger.html/issues/436/comments","events_url":"https://api.github.com/repos/jlongster/debugger.html/issues/436/events","html_url":"https://github.com/jlongster/debugger.html/pull/436","id":168208990,"number":436,"title":"(discussion) Add translate function","user":{"login":"jasonLaster","id":254562,"avatar_url":"https://avatars.githubusercontent.com/u/254562?v=3","gravatar_id":"","url":"https://api.github.com/users/jasonLaster","html_url":"https://github.com/jasonLaster","followers_url":"https://api.github.com/users/jasonLaster/followers","following_url":"https://api.github.com/users/jasonLaster/following{/other_user}","gists_url":"https://api.github.com/users/jasonLaster/gists{/gist_id}","starred_url":"https://api.github.com/users/jasonLaster/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jasonLaster/subscriptions","organizations_url":"https://api.github.com/users/jasonLaster/orgs","repos_url":"https://api.github.com/users/jasonLaster/repos","events_url":"https://api.github.com/users/jasonLaster/events{/privacy}","received_events_url":"https://api.github.com/users/jasonLaster/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-07-28T22:11:42Z","updated_at":"2016-07-29T23:14:09Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jlongster/debugger.html/pulls/436","html_url":"https://github.com/jlongster/debugger.html/pull/436","diff_url":"https://github.com/jlongster/debugger.html/pull/436.diff","patch_url":"https://github.com/jlongster/debugger.html/pull/436.patch"},"body":"One of the big blockers for landing the new debugger is translating our strings with l10n. \r\n\r\nThe approach we're planning on taking advantage of is a function that is a no-op in local development and l10n in the panel. \r\n\r\nI started working on the no-op function which will let us make sure we have string coverage before we land in the panel.\r\n\r\nNotice the styles that highlight the matched text in an obnoxious red. The goal here is that when we want to check it should be easy to see the strings.\r\n\r\n![screen shot 2016-07-28 at 6 09 51 pm](https://cloud.githubusercontent.com/assets/254562/17231192/6c011d9a-54ee-11e6-8b93-24faa4694850.png)\r\n"},"comment":{"url":"https://api.github.com/repos/jlongster/debugger.html/issues/comments/236316342","html_url":"https://github.com/jlongster/debugger.html/pull/436#issuecomment-236316342","issue_url":"https://api.github.com/repos/jlongster/debugger.html/issues/436","id":236316342,"user":{"login":"jlongster","id":17031,"avatar_url":"https://avatars.githubusercontent.com/u/17031?v=3","gravatar_id":"","url":"https://api.github.com/users/jlongster","html_url":"https://github.com/jlongster","followers_url":"https://api.github.com/users/jlongster/followers","following_url":"https://api.github.com/users/jlongster/following{/other_user}","gists_url":"https://api.github.com/users/jlongster/gists{/gist_id}","starred_url":"https://api.github.com/users/jlongster/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jlongster/subscriptions","organizations_url":"https://api.github.com/users/jlongster/orgs","repos_url":"https://api.github.com/users/jlongster/repos","events_url":"https://api.github.com/users/jlongster/events{/privacy}","received_events_url":"https://api.github.com/users/jlongster/received_events","type":"User","site_admin":false},"created_at":"2016-07-29T23:14:09Z","updated_at":"2016-07-29T23:14:09Z","body":"We should probably call it `formatStr` or whatever the current L10N API uses for consistency, and because we will need this function to take parameters (for localizing strings like \"has % items\"). Also, the current l10n system does not work like this. Each string is given a name and the API is `formatStr(\"sourceLabel\")`. The strings are listed in a properties file with these keys.\r\n\r\nWe should probably hold off on this until we really know more about our strategy. Are we wanting to leverage the current system? It seems like we should if we want to land for real soon, in which this PR doesn't really work.\r\n\r\nI also don't think we need to localize the toolbox. Lots of strings there and it's only for local development, and we don't have a way to localize strings for things outside of mozilla-central."}}
{"actor":{"display_login":"jlongster"}}