Closed
Bug 571114
Opened 14 years ago
Closed 14 years ago
Programs (main.js) need more documentation
Categories
(Add-on SDK Graveyard :: General, defect)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
0.5
People
(Reporter: adw, Assigned: adw)
References
Details
Attachments
(1 file)
6.13 KB,
patch
|
myk
:
review+
avarma
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•14 years ago
|
||
Noelle, any thoughts on how this might fit into the current doc structure?
Comment 2•14 years ago
|
||
*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.
Assignee | ||
Comment 4•14 years ago
|
||
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 | ||
Comment 5•14 years ago
|
||
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 6•14 years ago
|
||
Comment on attachment 451786 [details] [diff] [review] patch for 0.5 Yeah! I like it, great explanations.
Attachment #451786 -
Flags: review?(avarma) → review+
Comment 7•14 years ago
|
||
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+
Assignee | ||
Comment 8•14 years ago
|
||
http://hg.mozilla.org/labs/jetpack-sdk/rev/03a61f944c4d
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: -- → 0.5
Comment 9•14 years ago
|
||
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.
Description
•