iplanwebsites/newtab-bookmarks — pull request #3 by SBoudrias (2012-10-23 02:44:34+00:00)
Title: Add AMD modules and reorganise code in separate module

Hiya,

There's a first good step toward an easier to manage project. I separated code between different files and concern and added/tested AMD support by chrome app (works without any trouble).

This is a first step, but there's big trouble around architecture in the project.

View / models need to be clearly separated. Right now, models trigger directly function on their view, and views tend to know and call function on their container. This also bring some trouble with recursive dependencies, I've been able to hack around most of them, but this is really something to be address in near future. Else, risk is the project became un-maintainable.

Let me know if you see anything to change before merging.