Skip to content

Improve change password dialogue #3565

Closed

Description

@te-online

Recently, a user came to me saying that he couldn't change his password.
Only later I found out that he pressed the “change password” button before even typing, because to him the inputs looked disabled.

The biggest issue is probably that we don't have labels here. I stumbled upon this myself, entering two times the new password, not seeing that the first field requires the current password.

Suggestions for optimization:

  • Add labels to the input fields.
  • Create two input fields for the new password, to make sure you typed it correctly.
  • Ask for the current password with a modal dialogue like in the App Store when installing an app.

bildschirmfoto 2017-02-21 um 16 27 51
current change password form on the »personal« page

I know it would be really great to have a pull-request from me, but I can't do it at the moment – maybe someone else feels that this is an issue, too, and has the resources to quickly fix it. 😊 😳

@eppfel @jancborchardt ?

Activity

  1. nickvergessen commented on Feb 22, 2017

    @nickvergessen
    Member

    Sounds good to me

  2. mallory commented on Mar 21, 2017

    @mallory

    I just want to bump @floeschie's issue on the usability flaw in hiding the password reset link until after a failed attempt. The login page must have the password reset link visible at all times. This is expected behaviour. Without it, administrators like me are spending too much time helping our users gain access. Not to mention that it subtly reinforces to new users (which is most users) that this new platform is too hard for them. Thank you!

  3. added
    1. to developAccepted and waiting to be taken care of
    good first issueSmall tasks with clear documentation about how and in which place you need to fix things in.
    and removed on Apr 7, 2017
  4. simison commented on Jun 10, 2017

    @simison

    Heya @te-online and others! 👋

    I feel like I could grab this as my first Nextcloud issue as I like to work both with code and design.

    Looking at the code it looks like it would need a good overhaul, but I don't want to get too deep into it. I'm sure there are other more important issues to work on!

    I'll do:

    • Move labels from inputs to over the field and leave inputs empty
    • add visibility toggle to old password field as well
    • disable input button until both passwords are filled

    Probably (not):

    • add 3rd input field to confirm new password

    Question:

    • Does this need to support IE9 anymore? Edit: nope, IE11+
  5. jancborchardt commented on Jun 10, 2017

    @jancborchardt
    Member

    @simison great! :) Agree with your suggestion for solution, and also with your doubt about adding a 3rd field (since we have the option to make the password visible).

  6. simison commented on Jun 10, 2017

    @simison

    since we have the option to make the password visible).

    and there's always the password recovery (and I second @chaoticfree, it should be visible at login page without needing to type in wrong pass). I didn't find an issue for that tho.

  7. nickvergessen commented on Jun 12, 2017

    @nickvergessen
    Member

    and there's always the password recovery

    Sadly not, 1. depends on the user backend, 2. with encryption enabled and without master key you just loose everything...

    We could also just have new password and confirm and then use the modal which you have to enter when you are logged in for too long already.

  8. simison commented on Jun 12, 2017

    @simison

    We could also just have new password and confirm and then use the modal which you have to enter when you are logged in for too long already.

    True! On the other hand majority of apps/sites asks for the old password first and then new one. Mixing this other way around might be confusing as users expect to see an input for old password in the form.

    WDYT? I'm not strongly invested in this so either way is fine! Just wanted to point this out.

    Do you know where I could look for the code how to implement such modal?

  9. nickvergessen commented on Jun 12, 2017

    @nickvergessen
    Member

    OC.Dialogs is the magic word in JS source code:

    OC.PasswordConfirmation = {

    var OCdialogs = {

  10. 7 remaining items

  11. self-assigned this
    on Feb 25, 2018
  12. MorrisJobke commented on Jan 7, 2019

    @MorrisJobke
    Member

    See also #5181 where it is highlighted, that the password input would be useful for accounts with encryption enabled.

  13. skjnldsv commented on Jan 8, 2019

    @skjnldsv
    Member

    @skjnldsv skjnldsv self-assigned this on 25 Feb 2018

    I need to stop doing that

  14. removed their assignment
    on Jan 8, 2019
  15. Victorybrown commented on Oct 18, 2020

    @Victorybrown
  16. CarlSchwan commented on Sep 1, 2022

    @CarlSchwan
    Member

    Done with #33429

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    1. to developAccepted and waiting to be taken care ofdesignDesign, UI, UX, etc.enhancementfeature: settingsgood first issueSmall tasks with clear documentation about how and in which place you need to fix things in.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions