Minor "bug" in the user password form

This forum is for programmers who have questions about the source code.
Post Reply
richardwaite
Posts: 57
Joined: Thu Nov 15, 2007 6:51 am

Minor "bug" in the user password form

Post by richardwaite » Thu Jul 08, 2010 6:44 am

In FormUserPassword the tab indexes don't make much sense to me. "New Password" is 0, "OK" is 1, "Cancel" is 2, and "New Password Again" is 3. I would expect it to be: "New Password" 0, "New Password Again" 1, "OK" 2, "Cancel" 3. Also, pressing the Enter key does not seem to trigger "OK". I'm not sure how to change that, but the tab indexes are pretty easy to change. Not really a big deal, just some quirks I've noticed.

User avatar
jordansparks
Site Admin
Posts: 5770
Joined: Sun Jun 17, 2007 3:59 pm
Location: Salem, Oregon
Contact:

Re: Minor "bug" in the user password form

Post by jordansparks » Thu Jul 08, 2010 9:01 am

That is common in many forms. There are so many forms where textboxes accept an Enter, that we really have gotten away from an Enter closing the form. Yes, this is one case where we could have Enter close the form, but it would be more of an exception to the norm. As for tab order, we try to maintain it. But if it gets out of order, we classify it as an annoyance rather than a bug.
Jordan Sparks, DMD
http://www.opendental.com

richardwaite
Posts: 57
Joined: Thu Nov 15, 2007 6:51 am

Re: Minor "bug" in the user password form

Post by richardwaite » Fri Jul 09, 2010 12:57 pm

What would be the best way to fix these annoyances, then? I would be billing to email or post diffs if that would be acceptable. I'm speaking about only minor things here, like tab order or other UI quirks.

User avatar
jordansparks
Site Admin
Posts: 5770
Joined: Sun Jun 17, 2007 3:59 pm
Location: Salem, Oregon
Contact:

Re: Minor "bug" in the user password form

Post by jordansparks » Tue Jul 13, 2010 2:28 pm

Your post looks malformed. I don't understand "billing to email". Our only allowed way for outside developers to make alterations is with a plugin. Tab order could be easily changed with a plugin. The plugin would also prevent later alteration. One disadvantage is that if we ever change a control name, you wouldn't know it and your plugin would crash. This could be avoided by a quick testing script or by writing a program that validates your control names.
Jordan Sparks, DMD
http://www.opendental.com

richardwaite
Posts: 57
Joined: Thu Nov 15, 2007 6:51 am

Re: Minor "bug" in the user password form

Post by richardwaite » Wed Jul 14, 2010 12:22 pm

Sorry, "billing" should be "willing." So you do not accept patches from outside developers at all? A plugin for this kind of change just strikes me as overkill. It would have to be installed on all machines and maintained - does tab order really change that often? The only reason I brought this specific issue up is every new user I've seen setup a user and try to change the password hits tab and is confused because it does not do what they expect. It would be nice to have this and similar issues go away, and I'm willing to devote time to it - but I don't think plugins are the answer.

User avatar
jordansparks
Site Admin
Posts: 5770
Joined: Sun Jun 17, 2007 3:59 pm
Location: Salem, Oregon
Contact:

Re: Minor "bug" in the user password form

Post by jordansparks » Thu Jul 15, 2010 8:54 pm

Changed it in that form.
Jordan Sparks, DMD
http://www.opendental.com

richardwaite
Posts: 57
Joined: Thu Nov 15, 2007 6:51 am

Re: Minor "bug" in the user password form

Post by richardwaite » Fri Jul 16, 2010 10:55 am

Thanks.

Post Reply