...
- There are passing tests for new functionality
- The code is internationalized - at a minimum all human readable text are in strings in the options block
- The public API is commented
- Core logic is available through the public API (hint: your event handlers often contain core logic)
- ARIA roles are used where appropriate
- All functionality is accessible can be navigated to and controlled via the keyboard
- Code is broken up into small, well named things
- All javascript JavaScript files pass jsLint
- All html HTML files pass the w3c W3C validator
- All json JSON files pass json lint