{"action":"created","issue":{"url":"https://api.github.com/repos/Khan/live-editor/issues/262","labels_url":"https://api.github.com/repos/Khan/live-editor/issues/262/labels{/name}","comments_url":"https://api.github.com/repos/Khan/live-editor/issues/262/comments","events_url":"https://api.github.com/repos/Khan/live-editor/issues/262/events","html_url":"https://github.com/Khan/live-editor/issues/262","id":58702268,"number":262,"title":"Autocomplete of color() inside array produces unnecessary semi-colon","user":{"login":"pamelafox","id":297042,"avatar_url":"https://avatars.githubusercontent.com/u/297042?v=3","gravatar_id":"","url":"https://api.github.com/users/pamelafox","html_url":"https://github.com/pamelafox","followers_url":"https://api.github.com/users/pamelafox/followers","following_url":"https://api.github.com/users/pamelafox/following{/other_user}","gists_url":"https://api.github.com/users/pamelafox/gists{/gist_id}","starred_url":"https://api.github.com/users/pamelafox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pamelafox/subscriptions","organizations_url":"https://api.github.com/users/pamelafox/orgs","repos_url":"https://api.github.com/users/pamelafox/repos","events_url":"https://api.github.com/users/pamelafox/events{/privacy}","received_events_url":"https://api.github.com/users/pamelafox/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Khan/live-editor/labels/bug","name":"bug","color":"e11d21"},{"url":"https://api.github.com/repos/Khan/live-editor/labels/editor","name":"editor","color":"02e10c"},{"url":"https://api.github.com/repos/Khan/live-editor/labels/pjs","name":"pjs","color":"f7c6c7"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-02-24T07:17:47Z","updated_at":"2016-01-08T23:06:54Z","closed_at":null,"body":"For example:\r\n```\r\nvar bla = [\r\n  color: color(255, 0, 0);\r\n];\r\n```\r\nWe could decide to never put the semi-colon for that autocomplete. Or be smarter via regex/ast. \r\nOr decide that students can figure it out, which I'm sure they do. :-)\r\n"},"comment":{"url":"https://api.github.com/repos/Khan/live-editor/issues/comments/170152575","html_url":"https://github.com/Khan/live-editor/issues/262#issuecomment-170152575","issue_url":"https://api.github.com/repos/Khan/live-editor/issues/262","id":170152575,"user":{"login":"pamelafox","id":297042,"avatar_url":"https://avatars.githubusercontent.com/u/297042?v=3","gravatar_id":"","url":"https://api.github.com/users/pamelafox","html_url":"https://github.com/pamelafox","followers_url":"https://api.github.com/users/pamelafox/followers","following_url":"https://api.github.com/users/pamelafox/following{/other_user}","gists_url":"https://api.github.com/users/pamelafox/gists{/gist_id}","starred_url":"https://api.github.com/users/pamelafox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pamelafox/subscriptions","organizations_url":"https://api.github.com/users/pamelafox/orgs","repos_url":"https://api.github.com/users/pamelafox/repos","events_url":"https://api.github.com/users/pamelafox/events{/privacy}","received_events_url":"https://api.github.com/users/pamelafox/received_events","type":"User","site_admin":false},"created_at":"2016-01-08T23:06:54Z","updated_at":"2016-01-08T23:06:54Z","body":"Just for color(), do you mean? The semi-colon is handy for fill/stroke, because you can immediately start picking your color and see the effect, and you don't have to fix a syntax error first.\r\n\r\nLeaving it out for color() seems reasonable to me. Hopefully I haven't used it in any talk-throughs, as it could potentially affect them if I did. (Though we technically have versioning which helps)."}}