[CLOSED] Initial implementation of modules

[core-ai-bot]

Issue by gruehle
Thursday Jan 05, 2012 at 23:18 GMT
Originally opened as adobe/brackets#45

---

This is a first pass at modularizing Brackets source code. When viewing the diffs, you probably want to add "?w=1" to the end of the URL to ignore white space.

Here is a link to the diff without whitespace: https://github.com/adobe/brackets/pull/45/files?w=1

---

gruehle included the following code: https://github.com/adobe/brackets/pull/45/commits

[core-ai-bot]

Comment by jasonsanjose
Friday Jan 06, 2012 at 00:26 GMT

---

Should we consider following this http://requirejs.org/docs/jquery.html for using jQuery with modules?

[core-ai-bot]

Comment by gruehle
Friday Jan 06, 2012 at 01:08 GMT

---

(In reply to Jason's comment above):

The require-jquery file is a combination of require.js and jquery.js. They say that they regularly integrate jQuery, but we would be at the mercy of require.js to make sure our jQuery is up to date. I figured it was safest just to keep them separate.

[core-ai-bot]

Comment by peterflynn
Friday Jan 06, 2012 at 19:19 GMT

---

Looks good to me -- merging now.