bruizmartin / pons.context.search

A Firefox extension for enabling pons.de context search
https://addons.mozilla.org/en-US/firefox/addon/pons-context-search/
1 stars 1 forks source link

Wrap JS code into a JS object to avoid conflicts with other installed addons #1

Closed bruizmartin closed 10 years ago

bruizmartin commented 10 years ago

Reviewer: Baris Derin

Comments: This version didn't pass full review because of the following issues:

1) In order to prevent conflicts with other add-ons that may be installed by users, you need to wrap your "loose" variables and functions within a JavaScript object. You can see examples on how to do this at

https://developer.mozilla.org/en/XUL_School/JavaScript_Object_Management.

You need to correct them to get full approval. Thanks.