Closed Bug 601903 Opened 14 years ago Closed 14 years ago

Update static-files/md/dev-guide/programs.md for e10s context-menu changes

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)

static-files/md/dev-guide/programs.md has a context-menu example that needs to be updated after bug 578849 lands.
Attached patch patchSplinter Review
In addition to e10s changes, this also changes the example to use your more interesting translation add-on.
Assignee: nobody → adw
Status: NEW → ASSIGNED
Attachment #484195 - Flags: review?(myk)
Comment on attachment 484195 [details] [diff] [review]
patch

>+        var req = request.Request({
            ...
>+        });
>+        req.get();

This could be simplified to:

        request.Request({
          ...
        }).get();

But perhaps it's clearer the way it is.

In any case, looks good, r=myk.
Attachment #484195 - Flags: review?(myk) → review+
http://hg.mozilla.org/labs/jetpack-sdk/rev/895e79514ce6
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: -- → 0.9
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: