Autofocus to password input in webui...

[dasnyderx@…]

Currently, whenever the user goes initially goes to the login page on the web UI, they have to manually change the focus to the password input box (by clicking on the input box or tabbing through the links). Attached, you will find a diff (against the -current version of deluge) to change the focus to the password input box on window load. You will notice from the diff that I have coded the event within a script element on the login page. I looked through the mootools stuff and site, but it appears that the 'addEvents' method is only included with the mootools event package. Since, I didn't see that this mootools package was included with deluge, I therefore hand-coded the focus for the password input box on window load. If you have a better suggestion on how to implement this, I would be glad to hear and implement it.

BTW, you will note that the diff also includes changes to the login failure error message. This is due to the fact that there is a space missing between the comma and the 'try' word. I would also recommend that the error CSS class declaration (in template_style.css) be expanded beyond 200 pixels. The error phrase, "Password is invalid, try again.", is longer than the error class' tiny 200 pixel length.

[dasnyderx@…]

Diff to the login.html page for password auto-focus

[anonymous]

Thanks, I like a report with a patch. fixed in r4340

[mvoncken]

r4341 , error class set to 400px.

We're in transition from plain js to mootools. It worked great for webui-ajax, webui-html will use more of mootools & mooui in the future.

[mvoncken]

Note : fixed in 1.1, not in 1.0.

[anonymous]

Milestone 1.1.0 deleted