{"action":"created","comment":{"url":"https://api.github.com/repos/google/tie/pulls/comments/129398114","pull_request_review_id":52167082,"id":129398114,"diff_hunk":"@@ -733,6 +739,27 @@ tie.directive('learnerView', [function() {\n         };\n \n         /**\n+          * Highlights syntax errors in the coding UI\n+          *\n+          * @param {number} lineNumber\n+          */\n+        var highlightLine = function(lineNumber) {\n+          var actualLineNumber = lineNumber - 1;\n+          var codeLines = document.querySelectorAll('.CodeMirror-line');\n+          codeLines[actualLineNumber].className += ' tie-syntax-error-line';","path":"client/components/LearnerViewDirective.js","position":49,"original_position":49,"commit_id":"154540e98a8af9724d90db8746a9218544e6d6ff","original_commit_id":"154540e98a8af9724d90db8746a9218544e6d6ff","user":{"login":"seanlip","id":10575562,"avatar_url":"https://avatars0.githubusercontent.com/u/10575562?v=4","gravatar_id":"","url":"https://api.github.com/users/seanlip","html_url":"https://github.com/seanlip","followers_url":"https://api.github.com/users/seanlip/followers","following_url":"https://api.github.com/users/seanlip/following{/other_user}","gists_url":"https://api.github.com/users/seanlip/gists{/gist_id}","starred_url":"https://api.github.com/users/seanlip/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seanlip/subscriptions","organizations_url":"https://api.github.com/users/seanlip/orgs","repos_url":"https://api.github.com/users/seanlip/repos","events_url":"https://api.github.com/users/seanlip/events{/privacy}","received_events_url":"https://api.github.com/users/seanlip/received_events","type":"User","site_admin":false},"body":"Pull class name out into a named variable if you're going to use it in multiple places (see comment in clearHighlight() below).","created_at":"2017-07-25T19:10:34Z","updated_at":"2017-07-25T19:10:34Z","html_url":"https://github.com/google/tie/pull/380#discussion_r129398114","pull_request_url":"https://api.github.com/repos/google/tie/pulls/380","_links":{"self":{"href":"https://api.github.com/repos/google/tie/pulls/comments/129398114"},"html":{"href":"https://github.com/google/tie/pull/380#discussion_r129398114"},"pull_request":{"href":"https://api.github.com/repos/google/tie/pulls/380"}}},"pull_request":{"url":"https://api.github.com/repos/google/tie/pulls/380","id":132142203,"html_url":"https://github.com/google/tie/pull/380","diff_url":"https://github.com/google/tie/pull/380.diff","patch_url":"https://github.com/google/tie/pull/380.patch","issue_url":"https://api.github.com/repos/google/tie/issues/380","number":380,"state":"open","locked":false,"title":"Syntax error underlining","user":{"login":"eledavi","id":29108819,"avatar_url":"https://avatars2.githubusercontent.com/u/29108819?v=4","gravatar_id":"","url":"https://api.github.com/users/eledavi","html_url":"https://github.com/eledavi","followers_url":"https://api.github.com/users/eledavi/followers","following_url":"https://api.github.com/users/eledavi/following{/other_user}","gists_url":"https://api.github.com/users/eledavi/gists{/gist_id}","starred_url":"https://api.github.com/users/eledavi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eledavi/subscriptions","organizations_url":"https://api.github.com/users/eledavi/orgs","repos_url":"https://api.github.com/users/eledavi/repos","events_url":"https://api.github.com/users/eledavi/events{/privacy}","received_events_url":"https://api.github.com/users/eledavi/received_events","type":"User","site_admin":false},"body":"* Added function to highlight/underline syntax errors\r\n* Updated syntax error highlighting feature to highlight/un-highlight syntax errors as the user's code changes\r\n* Updated line number that is highlighted\r\n\r\n\r\n","created_at":"2017-07-24T20:51:15Z","updated_at":"2017-07-25T19:10:34Z","closed_at":null,"merged_at":null,"merge_commit_sha":"fb2716f284b67871b3ecdbb544ea8e2842f70c1a","assignee":null,"assignees":[],"requested_reviewers":[{"login":"eyurko","id":3400725,"avatar_url":"https://avatars1.githubusercontent.com/u/3400725?v=4","gravatar_id":"","url":"https://api.github.com/users/eyurko","html_url":"https://github.com/eyurko","followers_url":"https://api.github.com/users/eyurko/followers","following_url":"https://api.github.com/users/eyurko/following{/other_user}","gists_url":"https://api.github.com/users/eyurko/gists{/gist_id}","starred_url":"https://api.github.com/users/eyurko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eyurko/subscriptions","organizations_url":"https://api.github.com/users/eyurko/orgs","repos_url":"https://api.github.com/users/eyurko/repos","events_url":"https://api.github.com/users/eyurko/events{/privacy}","received_events_url":"https://api.github.com/users/eyurko/received_events","type":"User","site_admin":false},{"login":"theresa-lee","id":4167437,"avatar_url":"https://avatars1.githubusercontent.com/u/4167437?v=4","gravatar_id":"","url":"https://api.github.com/users/theresa-lee","html_url":"https://github.com/theresa-lee","followers_url":"https://api.github.com/users/theresa-lee/followers","following_url":"https://api.github.com/users/theresa-lee/following{/other_user}","gists_url":"https://api.github.com/users/theresa-lee/gists{/gist_id}","starred_url":"https://api.github.com/users/theresa-lee/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/theresa-lee/subscriptions","organizations_url":"https://api.github.com/users/theresa-lee/orgs","repos_url":"https://api.github.com/users/theresa-lee/repos","events_url":"https://api.github.com/users/theresa-lee/events{/privacy}","received_events_url":"https://api.github.com/users/theresa-lee/received_events","type":"User","site_admin":false}],"milestone":null,"commits_url":"https://api.github.com/repos/google/tie/pulls/380/commits","review_comments_url":"https://api.github.com/repos/google/tie/pulls/380/comments","review_comment_url":"https://api.github.com/repos/google/tie/pulls/comments{/number}","comments_url":"https://api.github.com/repos/google/tie/issues/380/comments","statuses_url":"https://api.github.com/repos/google/tie/statuses/154540e98a8af9724d90db8746a9218544e6d6ff","head":{"label":"google:syntax-error-underlining","ref":"syntax-error-underlining","sha":"154540e98a8af9724d90db8746a9218544e6d6ff","user":{"login":"google","id":1342004,"avatar_url":"https://avatars1.githubusercontent.com/u/1342004?v=4","gravatar_id":"","url":"https://api.github.com/users/google","html_url":"https://github.com/google","followers_url":"https://api.github.com/users/google/followers","following_url":"https://api.github.com/users/google/following{/other_user}","gists_url":"https://api.github.com/users/google/gists{/gist_id}","starred_url":"https://api.github.com/users/google/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/google/subscriptions","organizations_url":"https://api.github.com/users/google/orgs","repos_url":"https://api.github.com/users/google/repos","events_url":"https://api.github.com/users/google/events{/privacy}","received_events_url":"https://api.github.com/users/google/received_events","type":"Organization","site_admin":false},"repo":{"id":81500046,"name":"tie","full_name":"google/tie","owner":{"login":"google","id":1342004,"avatar_url":"https://avatars1.githubusercontent.com/u/1342004?v=4","gravatar_id":"","url":"https://api.github.com/users/google","html_url":"https://github.com/google","followers_url":"https://api.github.com/users/google/followers","following_url":"https://api.github.com/users/google/following{/other_user}","gists_url":"https://api.github.com/users/google/gists{/gist_id}","starred_url":"https://api.github.com/users/google/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/google/subscriptions","organizations_url":"https://api.github.com/users/google/orgs","repos_url":"https://api.github.com/users/google/repos","events_url":"https://api.github.com/users/google/events{/privacy}","received_events_url":"https://api.github.com/users/google/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/google/tie","description":null,"fork":false,"url":"https://api.github.com/repos/google/tie","forks_url":"https://api.github.com/repos/google/tie/forks","keys_url":"https://api.github.com/repos/google/tie/keys{/key_id}","collaborators_url":"https://api.github.com/repos/google/tie/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/google/tie/teams","hooks_url":"https://api.github.com/repos/google/tie/hooks","issue_events_url":"https://api.github.com/repos/google/tie/issues/events{/number}","events_url":"https://api.github.com/repos/google/tie/events","assignees_url":"https://api.github.com/repos/google/tie/assignees{/user}","branches_url":"https://api.github.com/repos/google/tie/branches{/branch}","tags_url":"https://api.github.com/repos/google/tie/tags","blobs_url":"https://api.github.com/repos/google/tie/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/google/tie/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/google/tie/git/refs{/sha}","trees_url":"https://api.github.com/repos/google/tie/git/trees{/sha}","statuses_url":"https://api.github.com/repos/google/tie/statuses/{sha}","languages_url":"https://api.github.com/repos/google/tie/languages","stargazers_url":"https://api.github.com/repos/google/tie/stargazers","contributors_url":"https://api.github.com/repos/google/tie/contributors","subscribers_url":"https://api.github.com/repos/google/tie/subscribers","subscription_url":"https://api.github.com/repos/google/tie/subscription","commits_url":"https://api.github.com/repos/google/tie/commits{/sha}","git_commits_url":"https://api.github.com/repos/google/tie/git/commits{/sha}","comments_url":"https://api.github.com/repos/google/tie/comments{/number}","issue_comment_url":"https://api.github.com/repos/google/tie/issues/comments{/number}","contents_url":"https://api.github.com/repos/google/tie/contents/{+path}","compare_url":"https://api.github.com/repos/google/tie/compare/{base}...{head}","merges_url":"https://api.github.com/repos/google/tie/merges","archive_url":"https://api.github.com/repos/google/tie/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/google/tie/downloads","issues_url":"https://api.github.com/repos/google/tie/issues{/number}","pulls_url":"https://api.github.com/repos/google/tie/pulls{/number}","milestones_url":"https://api.github.com/repos/google/tie/milestones{/number}","notifications_url":"https://api.github.com/repos/google/tie/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/google/tie/labels{/name}","releases_url":"https://api.github.com/repos/google/tie/releases{/id}","deployments_url":"https://api.github.com/repos/google/tie/deployments","created_at":"2017-02-09T22:11:24Z","updated_at":"2017-07-25T07:13:07Z","pushed_at":"2017-07-25T18:11:21Z","git_url":"git://github.com/google/tie.git","ssh_url":"git@github.com:google/tie.git","clone_url":"https://github.com/google/tie.git","svn_url":"https://github.com/google/tie","homepage":null,"size":1573,"stargazers_count":135,"watchers_count":135,"language":"JavaScript","has_issues":true,"has_projects":true,"has_downloads":false,"has_wiki":true,"has_pages":false,"forks_count":27,"mirror_url":null,"open_issues_count":39,"forks":27,"open_issues":39,"watchers":135,"default_branch":"master"}},"base":{"label":"google:master","ref":"master","sha":"4158a566af87e3c48869d0517949a218760f8ad2","user":{"login":"google","id":1342004,"avatar_url":"https://avatars1.githubusercontent.com/u/1342004?v=4","gravatar_id":"","url":"https://api.github.com/users/google","html_url":"https://github.com/google","followers_url":"https://api.github.com/users/google/followers","following_url":"https://api.github.com/users/google/following{/other_user}","gists_url":"https://api.github.com/users/google/gists{/gist_id}","starred_url":"https://api.github.com/users/google/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/google/subscriptions","organizations_url":"https://api.github.com/users/google/orgs","repos_url":"https://api.github.com/users/google/repos","events_url":"https://api.github.com/users/google/events{/privacy}","received_events_url":"https://api.github.com/users/google/received_events","type":"Organization","site_admin":false},"repo":{"id":81500046,"name":"tie","full_name":"google/tie","owner":{"login":"google","id":1342004,"avatar_url":"https://avatars1.githubusercontent.com/u/1342004?v=4","gravatar_id":"","url":"https://api.github.com/users/google","html_url":"https://github.com/google","followers_url":"https://api.github.com/users/google/followers","following_url":"https://api.github.com/users/google/following{/other_user}","gists_url":"https://api.github.com/users/google/gists{/gist_id}","starred_url":"https://api.github.com/users/google/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/google/subscriptions","organizations_url":"https://api.github.com/users/google/orgs","repos_url":"https://api.github.com/users/google/repos","events_url":"https://api.github.com/users/google/events{/privacy}","received_events_url":"https://api.github.com/users/google/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/google/tie","description":null,"fork":false,"url":"https://api.github.com/repos/google/tie","forks_url":"https://api.github.com/repos/google/tie/forks","keys_url":"https://api.github.com/repos/google/tie/keys{/key_id}","collaborators_url":"https://api.github.com/repos/google/tie/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/google/tie/teams","hooks_url":"https://api.github.com/repos/google/tie/hooks","issue_events_url":"https://api.github.com/repos/google/tie/issues/events{/number}","events_url":"https://api.github.com/repos/google/tie/events","assignees_url":"https://api.github.com/repos/google/tie/assignees{/user}","branches_url":"https://api.github.com/repos/google/tie/branches{/branch}","tags_url":"https://api.github.com/repos/google/tie/tags","blobs_url":"https://api.github.com/repos/google/tie/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/google/tie/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/google/tie/git/refs{/sha}","trees_url":"https://api.github.com/repos/google/tie/git/trees{/sha}","statuses_url":"https://api.github.com/repos/google/tie/statuses/{sha}","languages_url":"https://api.github.com/repos/google/tie/languages","stargazers_url":"https://api.github.com/repos/google/tie/stargazers","contributors_url":"https://api.github.com/repos/google/tie/contributors","subscribers_url":"https://api.github.com/repos/google/tie/subscribers","subscription_url":"https://api.github.com/repos/google/tie/subscription","commits_url":"https://api.github.com/repos/google/tie/commits{/sha}","git_commits_url":"https://api.github.com/repos/google/tie/git/commits{/sha}","comments_url":"https://api.github.com/repos/google/tie/comments{/number}","issue_comment_url":"https://api.github.com/repos/google/tie/issues/comments{/number}","contents_url":"https://api.github.com/repos/google/tie/contents/{+path}","compare_url":"https://api.github.com/repos/google/tie/compare/{base}...{head}","merges_url":"https://api.github.com/repos/google/tie/merges","archive_url":"https://api.github.com/repos/google/tie/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/google/tie/downloads","issues_url":"https://api.github.com/repos/google/tie/issues{/number}","pulls_url":"https://api.github.com/repos/google/tie/pulls{/number}","milestones_url":"https://api.github.com/repos/google/tie/milestones{/number}","notifications_url":"https://api.github.com/repos/google/tie/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/google/tie/labels{/name}","releases_url":"https://api.github.com/repos/google/tie/releases{/id}","deployments_url":"https://api.github.com/repos/google/tie/deployments","created_at":"2017-02-09T22:11:24Z","updated_at":"2017-07-25T07:13:07Z","pushed_at":"2017-07-25T18:11:21Z","git_url":"git://github.com/google/tie.git","ssh_url":"git@github.com:google/tie.git","clone_url":"https://github.com/google/tie.git","svn_url":"https://github.com/google/tie","homepage":null,"size":1573,"stargazers_count":135,"watchers_count":135,"language":"JavaScript","has_issues":true,"has_projects":true,"has_downloads":false,"has_wiki":true,"has_pages":false,"forks_count":27,"mirror_url":null,"open_issues_count":39,"forks":27,"open_issues":39,"watchers":135,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/google/tie/pulls/380"},"html":{"href":"https://github.com/google/tie/pull/380"},"issue":{"href":"https://api.github.com/repos/google/tie/issues/380"},"comments":{"href":"https://api.github.com/repos/google/tie/issues/380/comments"},"review_comments":{"href":"https://api.github.com/repos/google/tie/pulls/380/comments"},"review_comment":{"href":"https://api.github.com/repos/google/tie/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/google/tie/pulls/380/commits"},"statuses":{"href":"https://api.github.com/repos/google/tie/statuses/154540e98a8af9724d90db8746a9218544e6d6ff"}}}}
{"actor":{"display_login":"seanlip"}}