{"issue":{"milestone":null,"user":{"id":854976,"login":"stchangg","url":"https://api.github.com/users/stchangg","avatar_url":"https://secure.gravatar.com/avatar/102c913ae6b638d3b0c7539afe329535?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"102c913ae6b638d3b0c7539afe329535"},"title":"New exercise: Interactive telling time","created_at":"2011-12-18T10:47:23Z","updated_at":"2011-12-18T20:33:27Z","labels":[],"state":"open","id":2592892,"pull_request":{"patch_url":"https://github.com/Khan/khan-exercises/pull/8913.patch","html_url":"https://github.com/Khan/khan-exercises/pull/8913","diff_url":"https://github.com/Khan/khan-exercises/pull/8913.diff"},"html_url":"https://github.com/Khan/khan-exercises/issues/8913","closed_at":null,"body":"Trello bug: \"Telling time, interactive\" in \"Queued up\"\r\nFeedback much appreciated!\r\n\r\n## CHANGED\r\n\r\n### telling_time_interactive.html\r\n* \"Set the clock to 3:45\". User moves clock hands accordingly.\r\n* Hints might need some editing from a more experienced hint-writer.\r\n\r\n### time.js\r\n* Copied and extended `analogClock()` from `graphie-helpers.js` to a more customizable `addAnalogClock()` function (modeled after `addMovablePoint()`) inside `time.js`\r\n* NOTE: the functions `coordToDegrees()` and `degreesToCoord()` may belong in other packages. Any ideas where?\r\n* NOTE: `roundToNearest()` belongs in `math.js`. See below.\r\n\r\n### interactive.js\r\n* Added a `snapPoints` option to `movablePoint.constraints.fixedDistance` to allow snapping around a circle (`fixedDistance` restricts the point's motion to a circle). `snapPoints` is the number of points that can be snapped to around the circle.\r\n* NOTE: to implement this functionality, there is a bit of redundancy with functions in `angles.js` and `time.js`, but this might be unavoidable since `interactive.js` may be used without the others.\r\n\r\n### angle.js\r\n* Added `toDegrees()` function\r\n\r\n## NO CHANGE CURRENTLY, BUT SHOULD CHANGE\r\n\r\n### telling_time.html & telling_time_0.5.html\r\n* NOTE: should be updated using new `time.js` package (if accepted)\r\n\r\n### math.js\r\n* NOTE: I tried to add `roundToNearest()` (currently in `time.js`) to `math.js` but for some reason I couldn't get it to work. Something about how `math.js` is compiled? Is it a special file compared to the others?\r\n\r\n### graphie-helpers.js\r\n* NOTE: if `time.js` is accepted and `telling_time.html` & `telling_time_0.5.html` are updated, we should remove the `analogClock()` function from here.","number":8913,"url":"https://api.github.com/repos/Khan/khan-exercises/issues/8913","comments":2,"assignee":null},"comment":{"user":{"id":854976,"login":"stchangg","url":"https://api.github.com/users/stchangg","avatar_url":"https://secure.gravatar.com/avatar/102c913ae6b638d3b0c7539afe329535?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"102c913ae6b638d3b0c7539afe329535"},"created_at":"2011-12-18T20:33:27Z","updated_at":"2011-12-18T20:33:27Z","id":3196735,"body":"Whoops, I didn't mean to commit the changes to `unit_circle.html` or to `graphie-helpers.js`...","url":"https://api.github.com/repos/Khan/khan-exercises/issues/comments/3196735"},"action":"created"}