Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Usability issue when copying fields #8245

Open
Robert941 opened this issue Jul 6, 2022 · 5 comments
Open

Usability issue when copying fields #8245

Robert941 opened this issue Jul 6, 2022 · 5 comments
Assignees

Comments

@Robert941
Copy link

Robert941 commented Jul 6, 2022

Steps to reproduce issue
Open entry editor
Modify password field
Click "Copy Password" button on upper toolbar
Old password is copied to clipboard instead of new

Current solution
Click "Apply" before clicking "Copy Password"

Why this is an issue
This does not match the behavior of the "Copy Password" button in the password generator.
User could copy unintended data to clipboard (failure could go unnoticed due to password masking with asterisks).

Notes
This applies to the username, password, and URL fields.

Considerations
Preferable to maintain ability to click "Cancel" and discard any edits (implementing an "Auto Apply" feature would not be ideal).

@droidmonkey
Copy link
Member

This was brought up before and we decided not to change the behavior as it is incredibly hard to do so. You are editing an entry, so any changes you make are not baked into the database representation in memory until you hit apply or ok. In order to fix this we'd have to make hundreds of code changes. We just aren't going to do that, sorry.

@droidmonkey droidmonkey closed this as not planned Won't fix, can't repro, duplicate, stale Jul 6, 2022
@Robert941
Copy link
Author

Robert941 commented Jul 6, 2022

@droidmonkey What about adding a "Copy" button next to the field similar to the password generator's "copy" button?

@droidmonkey
Copy link
Member

You can select the text and right click copy or press Ctrl+C once selected. I actually thought about this more and we can pipe copy requests to the edit view if it is visible. I think this is possible and desirable.

@droidmonkey droidmonkey reopened this Jul 6, 2022
@droidmonkey droidmonkey added the ux label Jul 6, 2022
@droidmonkey droidmonkey self-assigned this Jul 6, 2022
@Robert941
Copy link
Author

That would be great since highlighting and copying introduces two risks.

  1. Accidentally modifying the text in the field
  2. Accidentally not highlighting the entire field (only copying part of the field)

@sjvudp
Copy link

sjvudp commented Feb 15, 2023

droidmonkey commented on Jul 6, 2022

(...) it is incredibly hard to do so (...)

Taking a value from RAM is "incredibly hard"??? I mean if you can display it, why can't you copy it?

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

No branches or pull requests

3 participants