Review of "WorkspaceBar" version 2

Details Page Preview

A toolbar for your panel that allows you to switch workspaces. Latest update includes a panel position option of left, center, and right. Also there is an option to display the Overview on mouse entry. Right-click extension to access preferences. Development builds at: https://github.com/mbokil/workspacebar

Extension Homepage
http://markbokil.com/downloads/extensions/

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
7 Active
6 Active
5 Rejected
4 Rejected
3 Rejected
2 Rejected
1 Active

Previous Reviews on this Version

Jasper St. Pierre rejected
Do not use eval. Use bracket notation if you want to get/set a property by variable name.
mbokil posted a review
I have tried many different combinations of bracket notation but I have been unable to get it to work right. The only thing that worked for me so far was eval. Everything else fails. Any suggestions? this.boxPosition = "_leftBox"; Main.panel.[this.boxPosition].insert_child_at_index(this.buttonBox,0);