search

mywot / firefox-xul

WOT's current XUL-based Firefox extension
http://www.mywot.com/wiki/Firefox
GNU General Public License v3.0
2 stars 8 forks source link

Wrap global variables #80

Closed samitolvanen closed 10 years ago

samitolvanen commented 10 years ago

From the previous review:

I have the following issues which should be addressed before your next update:

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

Problem globals include:

   wdump