{"action":"created","issue":{"url":"https://api.github.com/repos/jmoenig/Snap--Build-Your-Own-Blocks/issues/1149","repository_url":"https://api.github.com/repos/jmoenig/Snap--Build-Your-Own-Blocks","labels_url":"https://api.github.com/repos/jmoenig/Snap--Build-Your-Own-Blocks/issues/1149/labels{/name}","comments_url":"https://api.github.com/repos/jmoenig/Snap--Build-Your-Own-Blocks/issues/1149/comments","events_url":"https://api.github.com/repos/jmoenig/Snap--Build-Your-Own-Blocks/issues/1149/events","html_url":"https://github.com/jmoenig/Snap--Build-Your-Own-Blocks/issues/1149","id":136333371,"number":1149,"title":"Tables Need Testing In Older Browsers","user":{"login":"cycomachead","id":1505907,"avatar_url":"https://avatars.githubusercontent.com/u/1505907?v=3","gravatar_id":"","url":"https://api.github.com/users/cycomachead","html_url":"https://github.com/cycomachead","followers_url":"https://api.github.com/users/cycomachead/followers","following_url":"https://api.github.com/users/cycomachead/following{/other_user}","gists_url":"https://api.github.com/users/cycomachead/gists{/gist_id}","starred_url":"https://api.github.com/users/cycomachead/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cycomachead/subscriptions","organizations_url":"https://api.github.com/users/cycomachead/orgs","repos_url":"https://api.github.com/users/cycomachead/repos","events_url":"https://api.github.com/users/cycomachead/events{/privacy}","received_events_url":"https://api.github.com/users/cycomachead/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/jmoenig/Snap--Build-Your-Own-Blocks/labels/browser-specific","name":"browser-specific","color":"eb6420"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-02-25T09:42:53Z","updated_at":"2016-02-25T09:48:25Z","closed_at":null,"body":"As far as I understand, `String::repeat` is only available in the most recent browsers...\r\n\r\nhttps://github.com/jmoenig/Snap--Build-Your-Own-Blocks/blob/master/tables.js#L150\r\n\r\n\r\nSince these are off by default, I don't expect we will see much of a problem for now, but we're limiting ourselves to Chrome < 10 months, Safari < 5 months, and potentially no IE. "},"comment":{"url":"https://api.github.com/repos/jmoenig/Snap--Build-Your-Own-Blocks/issues/comments/188695879","html_url":"https://github.com/jmoenig/Snap--Build-Your-Own-Blocks/issues/1149#issuecomment-188695879","issue_url":"https://api.github.com/repos/jmoenig/Snap--Build-Your-Own-Blocks/issues/1149","id":188695879,"user":{"login":"jmoenig","id":2596285,"avatar_url":"https://avatars.githubusercontent.com/u/2596285?v=3","gravatar_id":"","url":"https://api.github.com/users/jmoenig","html_url":"https://github.com/jmoenig","followers_url":"https://api.github.com/users/jmoenig/followers","following_url":"https://api.github.com/users/jmoenig/following{/other_user}","gists_url":"https://api.github.com/users/jmoenig/gists{/gist_id}","starred_url":"https://api.github.com/users/jmoenig/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmoenig/subscriptions","organizations_url":"https://api.github.com/users/jmoenig/orgs","repos_url":"https://api.github.com/users/jmoenig/repos","events_url":"https://api.github.com/users/jmoenig/events{/privacy}","received_events_url":"https://api.github.com/users/jmoenig/received_events","type":"User","site_admin":false},"created_at":"2016-02-25T09:48:25Z","updated_at":"2016-02-25T09:48:25Z","body":"It's always good to test things on older browsers, but let's not forget that String::repeat is officially standard now, and Snap relies on other ES6 features such as Map anyway (hence the minimum requirements we specified elsewhere). But you're right, maybe we should replace it by a more defensive code. I'm open to a PR :-)"}}