Khan / khan-windows

Khan Academy for Windows 8
31 stars 15 forks source link

Various fixes and features #100

Closed bbondy closed 10 years ago

bbondy commented 10 years ago

Sorry for some of the reformatting mixed in, it'll make it more tedious to review. I went through each of the modified CSS and did a Edit -> Advanced -> Reformat with the hierarchical option on to avoid any future inconsistencies and hard reviews.

bbondy commented 10 years ago

By the way I think this is the last of it for the store release.

As part of the release after this one... I also have work locally that supports showing watched videos like it does online. I haven't done the reporting of watched videos yet. Both of those features should go together for the release after this one.

bbondy commented 10 years ago

Another note, feel free to also pull in the videos watched. But we'll probably release without that changeset. We can just tag the previous commit for release and if we need any additional fixes we can just git rebase -i

anprak commented 10 years ago

This is good for now but once we start reporting progress, it'll need some refactoring to use just one list to track progress instead of two (one for what we have on server and one local).

Thanks

bbondy commented 10 years ago

Thanks Anand, I'm planning to do exactly that refactoring in issue #5. The progress indicator stuff will be done independently. We can do a store release after all of that though at the same time. I have progress reporting working locally as well by the way (but no visual indicator yet).