Set focus when admin Users page loads

[Ryan J Ollos]

Set the focus to #username when the /admin/accounts/users page loads, and set the focus to #password when the /admin/accounts/users/<username> page loads.

[Ryan J Ollos]

I'd also suggest a refactoring, such as (untested):

<label py:choose="">Password:<br />
  <input type="password" name="password" class="textwidget"
         py:when="password_change_enabled" />
  <input type="password" name="password" class="textwidget"
         py:otherwise="" disabled="disabled" />
</label>

->

<label>Password:<br />
  <input type="password" name="password" class="textwidget"
         py:attrs="{'disabled': not password_change_enabled and
                                'disabled' or None}" />
</label>

[Ryan J Ollos]

Patch against r13290 of the trunk.

[Jun Omae]

Replying to rjollos:

I'd also suggest a refactoring, such as (untested):

I would do like this.

<label for="account-editor-password">Password:<br />
  <input id="account-editor-password"
         type="password" name="password" class="textwidget"
         disabled="${not password_change_enabled and 'disabled' or None}" />
</label>

Or

<label for="account-editor-password">Password:<br />
  <input id="account-editor-password"
         type="password" name="password" class="textwidget"
         disabled="${('disabled', None)[bool(password_change_enabled)]}" />
</label>

[Ryan J Ollos]

Replying to jun66j5:

Replying to rjollos:

I'd also suggest a refactoring, such as (untested):

I would do like this

The first of your two suggestions looks best to me.

[Steffen Hoffmann]

In 13323:

AccountManagerPlugin: Set sensible focus to an input field on page load, refs #11213 and #11215.

Template improvements are done too, mainly following valuable suggestions by
Jun and Ryan - thanks to both of you.

[Ryan J Ollos]

Here is another possibility using ​presentation.classes (untested):

disabled="${classes(disabled=not password_change_enabled)}"