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.

Sign up for GitHub

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

Jump to bottom

Copy TOTP value in preview pane on double click #9545

Closed
ytqxyd opened this issue Jun 10, 2023 · 3 comments · Fixed by #9647
Closed

Copy TOTP value in preview pane on double click #9545

ytqxyd opened this issue Jun 10, 2023 · 3 comments · Fixed by #9647
Assignees
@droidmonkey
Labels
feature: TOTP ux
Milestone
v2.7.6

Comments

@ytqxyd
Copy link

ytqxyd commented Jun 10, 2023

Sum

keepassxc

Add a copy button that allow quickly copy TOTP from this menu

Examples

image

Add a button here

Context

It's more clear where to get and copy TOTP code
@droidmonkey
Copy link
Member

Not going to add another button, just implement double click to copy

@droidmonkey droidmonkey changed the title Add copy TOTP button next to display current TOTP value Copy TOTP value in preview pane on double click Jun 10, 2023
@foss-
Copy link

foss- commented Jun 26, 2023

+1 for this feature request (double click is a good solution). When adding value to clipboard, please remove the space in the middle of the 6 digits.

Currently when copying the TOTP to clipboard the result is 123 456. However when pasting this may then result in 123 45 since the field the value is pasted to only allows 6 digits.

@droidmonkey droidmonkey added this to the v2.7.6 milestone Jun 26, 2023
@droidmonkey droidmonkey self-assigned this Jun 26, 2023
droidmonkey added a commit that referenced this issue Jul 18, 2023
@droidmonkey
Copy TOTP on preview panel on double click
654c76c 
* Closes #9545
@droidmonkey droidmonkey mentioned this issue Jul 18, 2023
Merged
droidmonkey added a commit that referenced this issue Jul 20, 2023
@droidmonkey
Copy TOTP on preview panel on double click
375ca63 
* Closes #9545
droidmonkey added a commit that referenced this issue Aug 6, 2023
@droidmonkey
Copy TOTP on preview panel on double click
ea77ee6 
* Closes #9545
@foss-
Copy link

foss- commented Aug 7, 2023

Verified in KeePassXC - Version 2.8.0-snapshot
Build Type: Snapshot
Revision: 5fb26d6

Thanks for implementing.

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

@droidmonkey droidmonkey

Labels
feature: TOTP ux
Projects
None yet
Milestone
v2.7.6
Development

Successfully merging a pull request may close this issue.

Several UI/UX Fixes keepassxreboot/keepassxc
3 participants
@droidmonkey @ytqxyd @foss-