{"action":"created","issue":{"url":"https://api.github.com/repos/empirical-org/Quill-Grammar/issues/30","labels_url":"https://api.github.com/repos/empirical-org/Quill-Grammar/issues/30/labels{/name}","comments_url":"https://api.github.com/repos/empirical-org/Quill-Grammar/issues/30/comments","events_url":"https://api.github.com/repos/empirical-org/Quill-Grammar/issues/30/events","html_url":"https://github.com/empirical-org/Quill-Grammar/issues/30","id":63360797,"number":30,"title":"brainpop comments","user":{"login":"ccowdery","id":6025451,"avatar_url":"https://avatars.githubusercontent.com/u/6025451?v=3","gravatar_id":"","url":"https://api.github.com/users/ccowdery","html_url":"https://github.com/ccowdery","followers_url":"https://api.github.com/users/ccowdery/followers","following_url":"https://api.github.com/users/ccowdery/following{/other_user}","gists_url":"https://api.github.com/users/ccowdery/gists{/gist_id}","starred_url":"https://api.github.com/users/ccowdery/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ccowdery/subscriptions","organizations_url":"https://api.github.com/users/ccowdery/orgs","repos_url":"https://api.github.com/users/ccowdery/repos","events_url":"https://api.github.com/users/ccowdery/events{/privacy}","received_events_url":"https://api.github.com/users/ccowdery/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-03-21T04:36:57Z","updated_at":"2015-03-28T01:50:13Z","closed_at":null,"body":"Bug # 1\r\nIn the princess and the turtle story, there are no parentheses in the text encapsulating the errors, and yet the directions still state that the students should remember to delete them before pressing submit.\r\n\r\nNote: Since we are electing to pivot towards the type 1 word model, rather than the whole sentence model, having an auto select feature that would select the whole word when its clicked on, including the parentheses errors, so that its 1 click to select the error, then type the correct word, then click to blank space to finish the change.\r\n\r\nThere is also a possibility that we would want to put in a \"revert to native text\" function on the stories, so that in case students are working on a passage and they get over their heads, they can press the revert button to set the whole passage back to its starting text without any of their additions.\r\n\r\nBUG # 2\r\nWhen I finished the story, it gave an incorrect text saying I found 5 of 1 errors, it always incorrectly lists amount of errors found.\r\n\r\n![bug 1](https://cloud.githubusercontent.com/assets/6025451/6763704/8f876d5c-cf5f-11e4-9fb9-2632e670d177.png)\r\n\r\nNOTE:  I find pages that the dialog box moves around on to be somewhat disctracting, so an idea would be to make the error box on the app stay stationary, while the page moves around it so the student doesn't have to move the mouse to click the next button again, not an urgent change.\r\n\r\nBUG # 3\r\nThe grammatical concepts listed on the errors on the errors found all say Grammar Concept TODO\r\n\r\n![image](https://cloud.githubusercontent.com/assets/6025451/6763721/08b565a8-cf60-11e4-9c7f-490b4b899b14.png)\r\n\r\nNOTE: I think that the \"check work\" button should disappear after you click it once and it takes you through the errors. Having it there at the bottom, once the student is reviewing their work on the story, can only confuse the student. Once the review phase of the story has started, the button need to disappear or  be renamed.\r\n\r\nBUG # 4\r\nCustom results page does not load at all, just gives the text \"Custom Results Page\"\r\n\r\n![image](https://cloud.githubusercontent.com/assets/6025451/6763737/7b54851c-cf60-11e4-86f6-7b51f3e92a43.png)\r\n\r\nNOTE: there might be a button that takes you back to back to the home page at BrainPop showing the Quill landing page, but if the results page does not load, then there is no way to get back. \r\n\r\nNOTE # 2: Quill does not show up if the word is searched on the brainpop search bar right above our page. We need to get that fixed asap, really urgent.\r\n\r\nBUG # 5\r\nThe correct answer to sentence questions after the paragrpaph has been completed do not appear.\r\nIt reads instead, \"Incorrect Answer: correct answer here\"\r\n\r\n![image](https://cloud.githubusercontent.com/assets/6025451/6763748/11904278-cf61-11e4-9574-3ab8933c53c5.png)\r\n\r\nBUG # 6\r\nSome of the spacing of errors is incorrect, where there are too many spaces before the slash (choice 1 / choice 2) is correct, but we have an extra space before the (near / on) error, as well as some other errors, in the Apollo 8 story.\r\n\r\n![image](https://cloud.githubusercontent.com/assets/6025451/6763769/48631cc0-cf62-11e4-83d0-8db0d10b236d.png)\r\n\r\n\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/empirical-org/Quill-Grammar/issues/comments/87141104","html_url":"https://github.com/empirical-org/Quill-Grammar/issues/30#issuecomment-87141104","issue_url":"https://api.github.com/repos/empirical-org/Quill-Grammar/issues/30","id":87141104,"user":{"login":"wlaurance","id":1059214,"avatar_url":"https://avatars.githubusercontent.com/u/1059214?v=3","gravatar_id":"","url":"https://api.github.com/users/wlaurance","html_url":"https://github.com/wlaurance","followers_url":"https://api.github.com/users/wlaurance/followers","following_url":"https://api.github.com/users/wlaurance/following{/other_user}","gists_url":"https://api.github.com/users/wlaurance/gists{/gist_id}","starred_url":"https://api.github.com/users/wlaurance/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wlaurance/subscriptions","organizations_url":"https://api.github.com/users/wlaurance/orgs","repos_url":"https://api.github.com/users/wlaurance/repos","events_url":"https://api.github.com/users/wlaurance/events{/privacy}","received_events_url":"https://api.github.com/users/wlaurance/received_events","type":"User","site_admin":false},"created_at":"2015-03-28T01:50:13Z","updated_at":"2015-03-28T01:50:13Z","body":"Went with, 'You must make at least X edits', where x = Math.floor((number of errors) / 2)"}}