Review of "Quit Button and IM Menu" version 5

Details Page Preview

Your Social state is not the Computer state, is it? :·) Bugs: If you find an empty (or quite empty) Computer menu, it’s probably a bug in another extension. Please send me the list of yours (activated or not), I’ll find it and contact his developper.

Extension Homepage
https://github.com/Obsidien/QuitButton

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

All Versions

Version Status
10 Active
9 Inactive
8 Inactive
7 Inactive
6 Rejected
5 Rejected
4 Active
3 Rejected
2 Active
1 Inactive

Previous Reviews on this Version

Jasper St. Pierre posted a review
Uh, why did you remove all semicolons?
Obsidien posted a review
I’m not a usual javascript coder, and I’m trying to make code readable. Is that a bad solution?
Obsidien posted a review
Jasper St. Pierre posted a review
You're trying to make code easier for me to review. Removing all semicolons makes a bunch of unnecessary diffs. It's not recommended in JS, and it looks suspicious.
Obsidien posted a review
Sorry for the additional work, I’ll think of that next time.