codeforamerica / bizfriendly-web

An experiment in digital literacy lessons. The experiment is now over.
http://bizfriend.ly
22 stars 67 forks source link

Redo lesson population on Learn page #21

Closed ondrae closed 11 years ago

ondrae commented 11 years ago

Description

I'm using jQuery to populate a table. A different jQuery line for each field and row. There should be a better way to do it. Maybe load all the possible lessons, then only let certain ones be visible?

Skills Needed

HTML, jQuery, Bootstrap

Tips

ondrae commented 11 years ago

Thanks @mholubowski. It looks and functions great. Next time submit a pull request so that we can all see the changes, review, and then approve. For instance, the lesson column name changed from lesson.url to lesson.third_party_service. With a pull request, we'd catch that before the code made it to the master branch. Thanks for the refactor though, it'll be much easier to style now that the table is built right.