Typo commented on a Page, Control  -  Jul 19, 2008

This script was not ready for release.

First off, you really need to close your brackets. It\'s one of the most basic parts of scripting and there really is no reason to be lazy about it. Just remember. EVERY single last { gets a } , no exceptions!

You should update a list when an item is added or removed. The user shouldn\'t have to close and open the dialog to see the list updated.

You have included a disconnect and connect to server option but the disconnect one doesnt specify which conection it closes if you have multiple networks opened and the connect one doesnt ask for a server or even open a new server window, it just disconects you by issuing /server with nothing else or connects you to whatever server u were on before if you arent already conected.

I dont understand why you used a dialog to enter the userhost for the userhost lookup when you made it look exactly like a standard $input box. If you hit the cancel button in the userhost dialog it still pops the userhost window up without any userhost in it.

Also, you didnt include a way to just open the broser in the homepage and if you hit the cancel button in the website entry box your confronted with \"* /run: insufficient parameters (line 316, script2.mrc)\" in the status window.

You dont check to see if the dialog is already open before trying to open it from the menu and why did you use a submenu in the rightclick menu instead of making the title open the control panel?

I\'m not even looking any further into this script because Im tired of typing about it so there may be plenty more wrong.

You need to take more time with your code before you release it.

Good luck.

 Respond  
Are you sure you want to unfollow this person?
Are you sure you want to delete this?
Click "Unsubscribe" to stop receiving notices pertaining to this post.
Click "Subscribe" to resume notices pertaining to this post.