Page MenuHomePhabricator

Change credential improvements / regressions with AuthManager
Closed, ResolvedPublic

Description

The change password form should look like this:

Screen Shot 2016-05-19 at 2.45.20 PM.png (771×368 px, 111 KB)

However with AuthManager it looks like this:

Screen Shot 2016-05-19 at 2.44.05 PM.png (770×378 px, 128 KB)

things to fix:

  • No logo should show up on the change password screen

[] No old password field to confirm you are still the same user

  • Label of "Change" password button should be "Change credentials"

Note there is an issue with the current form and your form that would be great to fix in process if you can:

  • Should be a margin between change password button and retype new password

Event Timeline

No old password field to confirm you are still the same user

AuthManager does it differently, because it's not that simple anymore. See the explanation at https://www.mediawiki.org/wiki/Manual:SessionManager_and_AuthManager/Updating_tips#User_class_password-checking_methods.

Label of change password button but should be "Change password"

Not likely, considering the page may be used for changing things other than passwords. That's why it's named "Change credentials" instead of "Change password".

Change credentials instead of Change password would be fine. The label change however is a little ambiguous. You may want to design Design input on that.

Thanks for elaborating on the change password field. I'll remove that.

Jdlrobson updated the task description. (Show Details)
Jdlrobson updated the task description. (Show Details)
Jdlrobson added a project: Design.

The lack of the old password field is intentional, AuthManager uses a different security model (see T136101).

(Oops. Shouldn't comment on a task that I haven't refreshed in a week.)

Jdlrobson triaged this task as High priority.
Jdlrobson added a project: Web-Team-Backlog.

Change 290782 had a related patch set uploaded (by Gergő Tisza):
Fix various AuthManager style regressions

https://gerrit.wikimedia.org/r/290782

Change 290782 merged by jenkins-bot:
Fix various AuthManager style regressions

https://gerrit.wikimedia.org/r/290782

Change 290937 had a related patch set uploaded (by Gergő Tisza):
Change button text for *Credentials special pages

https://gerrit.wikimedia.org/r/290937

Change 290941 had a related patch set uploaded (by Gergő Tisza):
Use HTMLForm style sheets on mobile

https://gerrit.wikimedia.org/r/290941

Change 290937 merged by jenkins-bot:
Change button text for *Credentials special pages

https://gerrit.wikimedia.org/r/290937

Change 290941 merged by jenkins-bot:
Use HTMLForm style sheets on mobile

https://gerrit.wikimedia.org/r/290941