GitHub PushEvent at 2014-09-01 17:13:26
Repository: perseus
Actor: AkA84
Payload: {"shas":[["fe93719f35dbf720580a99132e2502b4ee71fe86","a5181038bb66f42ede329124992d4e7c7761d537@gmail.com","Added a coefficient for input numbers.","Luke Magill",true],["35d57bd03d9516ca057da70de0e403d8370c51f0","a5181038bb66f42ede329124992d4e7c7761d537@gmail.com","Merge branch 'master' of https://github.com/Khan/perseus","Luke Magill",true],["f8930c4011a38a189e886ebd5b06b69ddf68be35","8ccb5427407acc438a6e02b9c659ae5155d12eae@Alessandros-MacBook-Pro.local","Add remove widget button\n\nSummary: Added a 'remove' button on the header of the widget editor to quickly remove it from the question\n\nMostly an ice breaker for me as this is my first PR, so I used this little change (hopefully somewhat useful) as a way to get familiar with React and Perseus.\n\nWhat has been done:\n* Wrote a widgetPlaceholder string as to avoid repetition in the code\n* Wrote a widgetRegExp string for the same reason (although it could be improved)\n* Wrote a _handleWidgetEditorRemove function that removes the selected widget from the textarea content and calls re-rendering\n* Placed a remove button in the header of the WidgetEditor","Alessandro",false],["88d14d2d072893a7e81c4a324a54aa5adc336a2d","596727c8a0ea4db3ba2ceceedccbacd3d7b371b8@khanacademy.org","simple-markdown: add <autolinks> and <auto@mailto>\n\nSummary: automatic <links> and <mailto@addresses>\n\nTest Plan: `make test`\n\nReviewers: emily\n\nReviewed By: emily\n\nSubscribers: alpert\n\nDifferential Revision: http://phabricator.khanacademy.org/D12565","Jack Toole",true],["4ad7e8cf35cff61ba0fc6445fc67ee0a1bd98d18","596727c8a0ea4db3ba2ceceedccbacd3d7b371b8@khanacademy.org","simple-markdown: parse freeform urls\n\nTest Plan: `make test`\n\nReviewers: emily\n\nReviewed By: emily\n\nDifferential Revision: http://phabricator.khanacademy.org/D12574","Jack Toole",true],["f4ca0de1ac046658c4e7e228f27995ee871782ac","596727c8a0ea4db3ba2ceceedccbacd3d7b371b8@khanacademy.org","simple-markdown: Add reflink support\n\nSummary:\n    [Reflinks][1] are fun--they reference a \"def\" defined elsewhere in\n    the markdown. To handle these, we keep extra state about what defs\n    and what reflinks have been seen so far on the `state` object\n    during the parse. Then when we reach a ref or a def, we fill in\n    the newly acquired information into the reflink AST node.\n\n    Yeah, mutation :(. But this works really surprisingly cleanly.\n    The goal was to keep all the parsing inside the standard parse\n    loop, so that anyone extending the parser still has all basic\n    functionality intact, and I think this succeeds at that goal.\n\n    [1] http://example.com/def\n\nTest Plan:\n`make test`\nExperimented with it in the simple-markdown-tester widget\n\nReviewers: emily\n\nReviewed By: emily\n\nDifferential Revision: http://phabricator.khanacademy.org/D12614","Jack Toole",true],["a27116c2e8f1d63f1ec4885f824fb16352cb7e50","cf61774138ee77c0bd8a130240f9ab38c5800dba@tin.it","Move RegExp instantiation outside of the render function","Alessandro Verdura",false],["f728391e6ed4bca440b248e5f45743636f9eb591","596727c8a0ea4db3ba2ceceedccbacd3d7b371b8@khanacademy.org","renderer: better handling of invalid widget types\n\nSummary:\nItem x66d49f1bab5f8fa4 was crashing on the site because\nof references to an \"image-widget\". This change makes\nthe renderer code-path resilient to nonexistant widgets\n(in addition to the editor, which already worked).\n\nTest plan:\nOpen localhost:9000/testrender.html, which has an\ninvalid \"image-widget\" widget (the type is just\n\"image\" these days).\n\n(the changes in testrender.html are just for testing)\n\nAuditors: michelle","Jack Toole",true],["6b6900a88e8cf02cf2febf84b74cb7a6d258a96a","596727c8a0ea4db3ba2ceceedccbacd3d7b371b8@khanacademy.org","categorizer: fix labels for mobile\n\nSummary:\n<label> elements don't work correctly on mobile safari, so here's a\npseudo-label implementation that should work on both web and mobile.\n\nTest Plan:\ntested on [localhost](http://localhost:9000/test.html#content=%7B%22question%22%3A%7B%22content%22%3A%22%5B%5B%E2%98%83%20categorizer%202%5D%5D%22%2C%22images%22%3A%7B%22https%3A%2F%2Fka-perseus-graphie.s3.amazonaws.com%2Fda8df81c78b22f5c69d477d8eabfb583968eaf84.png%22%3A%7B%22width%22%3A400%2C%22height%22%3A70%7D%2C%22https%3A%2F%2Fka-perseus-graphie.s3.amazonaws.com%2Fb59fc02ca1aae800977b8793ed22f647a1aa75ee.png%22%3A%7B%22width%22%3A425%2C%22height%22%3A150%7D%7D%2C%22widgets%22%3A%7B%22categorizer%202%22%3A%7B%22type%22%3A%22categorizer%22%2C%22graded%22%3Atrue%2C%22options%22%3A%7B%22items%22%3A%5B%221%22%2C%222%22%2C%223%22%5D%2C%22categories%22%3A%5B%22A%22%2C%22B%22%2C%22C%22%5D%2C%22values%22%3A%5B%5D%2C%22randomizeItems%22%3Afalse%7D%2C%22version%22%3A%7B%22major%22%3A0%2C%22minor%22%3A0%7D%7D%7D%7D%2C%22answerArea%22%3A%7B%22type%22%3A%22multiple%22%2C%22options%22%3A%7B%22content%22%3A%22%22%2C%22images%22%3A%7B%7D%2C%22widgets%22%3A%7B%7D%7D%2C%22calculator%22%3Afalse%7D%2C%22itemDataVersion%22%3A%7B%22major%22%3A0%2C%22minor%22%3A1%7D%2C%22hints%22%3A%5B%5D%7D)\n\nand on mobile with the exercise \"rules for multiplying and dividing\nnegative numbers\"\n\nReviewers: michelle, eater\n\nReviewed By: eater\n\nDifferential Revision: http://phabricator.khanacademy.org/D12700","Jack Toole",true],["cfd5b6d5dca9e33724b0159686982f2a4129d73f","a5181038bb66f42ede329124992d4e7c7761d537@gmail.com","Switching coefficient checkbox to main answer.","Luke Magill",true],["b56dc774c78869dc446290a75f0a5f77ba2a3270","a5181038bb66f42ede329124992d4e7c7761d537@gmail.com","Merge branch 'master' of https://github.com/Khan/perseus","Luke Magill",true],["7dbeca446b48c38f66f0d3ae414d77c3d9d1a3b1","7212a9e01329ea93a57f574bd9bf77695d5fdca4@khanacademy.org","Add `make server-offline ...` to makefile so you can run perseus without an internet connection\n\nTest Plan:\nDisconnect from the internet\nRun `make server-offline PORT=9000`\nVisit http://localhost:9000 and be amazed\n\nReviewers: aria\n\nReviewed By: aria\n\nDifferential Revision: http://phabricator.khanacademy.org/D12737","Michelle Todd",true],["067a0f3d8a368c2beb857016b4e79bea78ef6b01","a5181038bb66f42ede329124992d4e7c7761d537@gmail.com","Merge branch 'master' of https://github.com/Khan/perseus\n\nConflicts:\n\tMakefile","Luke Magill",true],["5759090c0e9d54526132c54ac0bc141d5cb9f917","cf9ecc127c93992ba1214128e86fc9f20ec04686@gmail.com","Merge pull request #9 from AkA84/add-remove-widget-button\n\nAdd remove widget button","Aria Buckles",true],["25fc85e64abee2d2367c6d69fdbcd82e5e0e03df","596727c8a0ea4db3ba2ceceedccbacd3d7b371b8@khanacademy.org","simple-markdown: parse tables with and without pipes\n\nSummary:\ngithub flavored markdown supports two types of tables:\n\n    | leading | pipes |\n\nand\n\n    no-pipe | tables\n\nthis supports those as `table` and `nptable`. The logic and\nregexes are adapted from marked.js\n\nTest Plan: `make test`\n\nReviewers: emily\n\nReviewed By: emily\n\nDifferential Revision: http://phabricator.khanacademy.org/D12698","Jack Toole",true],["6df9b3e214ade8f25134f805ed18c0a4a5ce607a","596727c8a0ea4db3ba2ceceedccbacd3d7b371b8@khanacademy.org","simple-markdown: add image+refimage support\n\nSummary:\n    This adds support for inline ![](images) and\n    [ref][images]\n\n    [images]: <url>\n\nThis change also renamed LINK_HREF to LINK_HREF_AND_TITLE\nto be more accurate about what it is capturing\n\nThis change also fixes titles on regular inline links, and\nadds test cases verifying that\n\nTest Plan: Make both types of images in the simple-markdown-tester\n\nReviewers: emily\n\nReviewed By: emily\n\nDifferential Revision: http://phabricator.khanacademy.org/D12709","Jack Toole",true],["5a1976818f8dff49da12a522f6849f69b2b8f256","596727c8a0ea4db3ba2ceceedccbacd3d7b371b8@khanacademy.org","simple-markdown: handle escaping better\n\nSummary:\nPreviously, we had a list of characters that we would break\ntext on to check for possible other syntax, and a list of\ncharacters that were valid to escape. With this change,\nwe break text on any symbol, and can escape any symbol.\n\nTest Plan: `make test`\n\nReviewers: emily\n\nReviewed By: emily\n\nDifferential Revision: http://phabricator.khanacademy.org/D12714","Jack Toole",true],["8e458fd92454fdebd15898b0a768f54ab0b8cae0","596727c8a0ea4db3ba2ceceedccbacd3d7b371b8@khanacademy.org","simple-markdown: Add <br> elements\n\nSummary:\nline breaks in markdown are sort of weird:\ntwo spaces at the end of a line inside a paragraph.\n\nTest Plan:\n`make test`\nAlso experimented a bit in the simple-markdown-tester\n\nReviewers: emily\n\nReviewed By: emily\n\nDifferential Revision: http://phabricator.khanacademy.org/D12772","Jack Toole",true],["fa9acff4534ae9d6cd734bbf980a1916af5bde4a","596727c8a0ea4db3ba2ceceedccbacd3d7b371b8@khanacademy.org","passage: add footnote parsing to passage-markdown\n\nSummary:\n```\nAdds basic footnote syntax, which takes any `^` characters\nin a passage and places a superscript denoting a footnote\nthere, which content creators can put in the \"footnotes\"\nbox of the widget.\n\nSome other possibilities we could implement are having\ninline footnotes  like^(this), or reference footnotes\nthat reference a footnote declared later^[1]\n\n[1]: such as down here\n```\n\nTest Plan: `make test`\n\nReviewers: michelle\n\nReviewed By: michelle\n\nSubscribers: emily\n\nDifferential Revision: http://phabricator.khanacademy.org/D12715","Jack Toole",true],["9630ae383522bf63aa63a0e9c106d11f465e724c","596727c8a0ea4db3ba2ceceedccbacd3d7b371b8@khanacademy.org","Passage: rewrite passage refs to use passage-markdown\n\nSummary:\nChanges how the passage widget supports markdown and references\nto use simple-markdown/passage-markdown and hopefully be simpler\nand less hacky\n\nTest Plan:\ncheck out [this item](http://localhost:9000/test.html#content=%7B%22question%22%3A%7B%22content%22%3A%22%5B%5B%E2%98%83%20passage%202%5D%5D%5Cn%5Cn%5B%5B%E2%98%83%20passage-ref%201%5D%5D%5Cn%5Cn%5B%5B%E2%98%83%20passage-ref%202%5D%5D%5Cn%5Cn%5B%5B%E2%98%83%20passage-ref%203%5D%5D%22%2C%22images%22%3A%7B%22https%3A%2F%2Fka-perseus-graphie.s3.amazonaws.com%2Fda8df81c78b22f5c69d477d8eabfb583968eaf84.png%22%3A%7B%22width%22%3A400%2C%22height%22%3A70%7D%2C%22https%3A%2F%2Fka-perseus-graphie.s3.amazonaws.com%2Fb59fc02ca1aae800977b8793ed22f647a1aa75ee.png%22%3A%7B%22width%22%3A425%2C%22height%22%3A150%7D%7D%2C%22widgets%22%3A%7B%22passage%202%22%3A%7B%22type%22%3A%22passage%22%2C%22graded%22%3Atrue%2C%22options%22%3A%7B%22passageText%22%3A%22this%20is%20a%20aklsdj%20fkldjafs%20lkdfja%20kldfjflda%20l%3Bdjsfkldfsajdsfkl%3Bdsf%20jalkdfsj%20dsflkfjdsa%20l%3Bkdfajdfl%20%3Bdfsajdlfks%3Bj%20dflkdsfj%20dslfkj%20dsflkdsfj%20%7B%7B%2A%2Aklsdfjsdflk%20%7B%7B__fdsjlksdf__%7D%7D%20jfsdlklj%20fdslkfdsj%20lkfsdjsdflkjdsfkl%20dsfj%20lkdfsjdsfl%2A%2A%7D%7D%5Cn%5Cndfskjladfafjklfdjfdsklsd%20fjl%20d%20sfkj%20ds%20f%20lkdfsj%20l%20k%20fd%20s%20jdfskl%20sfdjkl%20dsfjf%20d%20lk%20d%20fs%20j%20dl%20k%20s%20j%20s%20dfkldsfjdsklf%5Cn%5Cndkl%20jkfdsah%20fjkfjd%20a%20djjfdaldfajs%20kdfsajal%3Bkdfasjkdfsakj%20dfskfd%20djfkdf%20dkajdk%20jadfkjadf%20__%7B%7Bdafksjkdfkjfdk%7D%7D__%20oiwjhqwe%20oew%20ewj%20weroijkojdsx%2Cmnacx%20lkhdskldsfz%20dskl%20k%22%2C%22showLineNumbers%22%3Atrue%7D%2C%22version%22%3A%7B%22major%22%3A0%2C%22minor%22%3A0%7D%7D%2C%22passage-ref%201%22%3A%7B%22type%22%3A%22passage-ref%22%2C%22graded%22%3Atrue%2C%22options%22%3A%7B%22passageNumber%22%3A2%2C%22referenceNumber%22%3A1%7D%2C%22version%22%3A%7B%22major%22%3A0%2C%22minor%22%3A0%7D%7D%2C%22passage-ref%202%22%3A%7B%22type%22%3A%22passage-ref%22%2C%22graded%22%3Atrue%2C%22options%22%3A%7B%22passageNumber%22%3A2%2C%22referenceNumber%22%3A2%7D%2C%22version%22%3A%7B%22major%22%3A0%2C%22minor%22%3A0%7D%7D%2C%22passage-ref%203%22%3A%7B%22type%22%3A%22passage-ref%22%2C%22graded%22%3Atrue%2C%22options%22%3A%7B%22passageNumber%22%3A2%2C%22referenceNumber%22%3A3%7D%2C%22version%22%3A%7B%22major%22%3A0%2C%22minor%22%3A0%7D%7D%7D%7D%2C%22answerArea%22%3A%7B%22type%22%3A%22multiple%22%2C%22options%22%3A%7B%22content%22%3A%22%22%2C%22images%22%3A%7B%7D%2C%22widgets%22%3A%7B%7D%7D%2C%22calculator%22%3Afalse%7D%2C%22itemDataVersion%22%3A%7B%22major%22%3A0%2C%22minor%22%3A1%7D%2C%22hints%22%3A%5B%5D%7D)\nNote that the second reference is inside the first reference.\nNote the references are correct.\n\nReviewers: michelle\n\nReviewed By: michelle\n\nSubscribers: alpert, alex, joel\n\nDifferential Revision: http://phabricator.khanacademy.org/D12258","Jack Toole",true]],"size":24,"ref":"refs/heads/master","head":"480695641a2ee38396b5fc4b5567952f1421c12c"}