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

Introduce security option to enable copy on doubleclick #6433

Merged
merged 6 commits into from
Apr 24, 2021

Conversation

xvallspl
Copy link
Contributor

@xvallspl xvallspl commented Apr 21, 2021

Introduces a checkbox in the security section of the preferences to enable the
current feature (now disabled by default) of copying DatabaseView entries
columns, such as username and password, on double click. Addresses issue #1575.

  • Implement new checkbox
  • Document the new functionality in the User Guide
  • Think of a better descriptive text for the checkbox
  • Figure out how to test it

Screenshots

Screenshot 2021-04-21 at 06 45 18

Testing strategy

Figuring this out

Type of change

  • ✅ New feature (change that adds functionality)
  • ✅ Breaking change (causes existing functionality to change)
  • ✅ Documentation (non-code change)

Introduces a checkbox in the security section of the preferences to enable the
current feature (now disabled by default) of copying DatabaseView entries
columns, such as username and password, on double click. Addresses issue #1575.
src/gui/DatabaseWidget.cpp Outdated Show resolved Hide resolved
src/gui/DatabaseWidget.cpp Outdated Show resolved Hide resolved
src/gui/DatabaseWidget.cpp Outdated Show resolved Hide resolved
@xvallspl
Copy link
Contributor Author

Any ideas on how to test this?

@droidmonkey
Copy link
Member

Just test manually, this is low impact and automated tests involve timing and other crap that regularly breaks testing in CI

@xvallspl xvallspl marked this pull request as ready for review April 21, 2021 17:30
@xvallspl
Copy link
Contributor Author

xvallspl commented Apr 21, 2021

Then this one should be ready to go if it passes the CI

src/gui/DatabaseWidget.cpp Outdated Show resolved Hide resolved
Copy link
Member

@droidmonkey droidmonkey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Elegant!

@droidmonkey
Copy link
Member

droidmonkey commented Apr 23, 2021

Need to run make format

@xvallspl
Copy link
Contributor Author

Sorry, I guess my clangd plugin is misconfigured

@phoerious
Copy link
Member

Could also just be a different version. Make sure you have the latest.

@droidmonkey droidmonkey changed the title introduce a security option to enable copy on doubleclick (Issue #1575) Introduce security option to enable copy on doubleclick Apr 24, 2021
@droidmonkey droidmonkey merged commit 7b7f52c into develop Apr 24, 2021
@droidmonkey droidmonkey deleted the feature/checkbox-enable-doubleclick-copy branch April 24, 2021 15:35
@droidmonkey droidmonkey added this to the v2.7.0 milestone Apr 24, 2021
pull bot pushed a commit to tank0226/keepassxc that referenced this pull request Apr 24, 2021
pull bot pushed a commit to iloeng/keepassxc that referenced this pull request Apr 24, 2021
pull bot pushed a commit to SammyEnigma/keepassxc that referenced this pull request Apr 24, 2021
pull bot pushed a commit to NeatNerdPrime/keepassxc that referenced this pull request Apr 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants