Closed Bug 571114 Opened 14 years ago Closed 14 years ago

Programs (main.js) need more documentation

Categories

(Add-on SDK Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: adw, Assigned: adw)

References

Details

Attachments

(1 file)

Programs (or maybe just main.js) need to be documented more thoroughly, especially since bug 549324 added support for exporting an onUnload method, which is intended for use by high-level developers.  We'll need to be able to point those developers to docs when we release 0.5.  Right now there's only an example in the tutorial section.
Noelle, any thoughts on how this might fit into the current doc structure?
*nod* I think this should fit with the restructure (https://bugzilla.mozilla.org/show_bug.cgi?id=565204) but if it somehow slips; I can add a blurb re: onUnload to the current "Programs" page.
Attached patch patch for 0.5Splinter Review
Instead of making a new Programs section in the appendix like I had planned, it seems sufficient for 0.5 to add a subsection to the Programs section in the tutorial.  The subsection is small enough that I think it doesn't get in the tutorial's way.  When the doc restructurings of bug 565204 and bug 571285 land, this subsection can find a better home.
Assignee: nobody → adw
Status: NEW → ASSIGNED
Attachment #451786 - Flags: review?(myk)
Comment on attachment 451786 [details] [diff] [review]
patch for 0.5

Atul, it'd be great if you could do a quick review too.  In addition to adding a new subsection to Programs about main() and onUnload(), I updated the glossary entry for Jetpack Program, and changed "activate" terminology to "load" and "unload".
Attachment #451786 - Flags: review?(avarma)
Comment on attachment 451786 [details] [diff] [review]
patch for 0.5

Yeah! I like it, great explanations.
Attachment #451786 - Flags: review?(avarma) → review+
Comment on attachment 451786 [details] [diff] [review]
patch for 0.5

This all looks great, r=myk + a=myk for landing during the freeze.
Attachment #451786 - Flags: review?(myk) → review+
http://hg.mozilla.org/labs/jetpack-sdk/rev/03a61f944c4d
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: -- → 0.5
The Add-on SDK is no longer a Mozilla Labs experiment and has become a big enough project to warrant its own Bugzilla product, so the "Add-on SDK" product has been created for it, and I am moving its bugs to that product.

To filter bugmail related to this change, filter on the word "looptid".
Component: Jetpack SDK → General
Product: Mozilla Labs → Add-on SDK
QA Contact: jetpack-sdk → general
Version: Trunk → unspecified
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: