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

Minor fixes for 2.4.1 #2947

Merged
merged 4 commits into from
Apr 7, 2019
Merged

Minor fixes for 2.4.1 #2947

merged 4 commits into from
Apr 7, 2019

Conversation

droidmonkey
Copy link
Member

@droidmonkey droidmonkey commented Apr 5, 2019

Type of change

  • ✅ Bug fix (non-breaking change which fixes an issue)

Description and Context

Improved error messages when opening database

Add note to restart after changing language

Allow copying passwords directly from searching

Correct issues with apply button

  • Don't show apply button when creating new entries or groups (Fix New entries can be discarded if user only presses Apply #2191)
  • Don't mark entry/group as dirty when first creating a new one (prevents unnecessary discard dialog on cancel)
  • Properly enable/disable apply button when changes are made to entries and groups
  • Don't show discard change warning when locking database unless their are actual changes made

NOTE: Extra pages in the group edit widget are not watched for changes yet. Requires a major refactor.

Screenshots

image

image

Testing strategy

Added tests to ensure copy from search worked.

Checklist:

  • ✅ I have read the CONTRIBUTING document. [REQUIRED]
  • ✅ My code follows the code style of this project. [REQUIRED]
  • ✅ All new and existing tests passed. [REQUIRED]
  • ✅ I have compiled and verified my code with -DWITH_ASAN=ON. [REQUIRED]
  • ✅ My change requires a change to the documentation, and I have updated it accordingly.
  • ✅ I have added tests to cover my changes.

* Reduced wording and confusion
* Streamlined delivery format
* Fix #813
@droidmonkey droidmonkey added the ux label Apr 5, 2019
@droidmonkey droidmonkey added this to the v2.4.1 milestone Apr 5, 2019
@droidmonkey droidmonkey requested a review from phoerious April 5, 2019 14:23
@phoerious
Copy link
Member

Looks good to mew, but the search tests are failing.

@droidmonkey droidmonkey force-pushed the hotfix/consolidated-2.4.1 branch from 24e37f7 to a702aa3 Compare April 7, 2019 13:56
@droidmonkey droidmonkey force-pushed the hotfix/consolidated-2.4.1 branch 2 times, most recently from 3839449 to b349dd0 Compare April 7, 2019 15:46
* Reverts removal of previously implemented feature
* Fix #2630
* Make gui search tests more robust
* Don't show apply button when creating new entries or groups (Fix #2191)
* Don't mark entry/group as dirty when first creating a new one (prevents unnecessary discard dialog on cancel)
* Properly enable/disable apply button when changes are made to entries and groups
* Don't show discard change warning when locking database unless their are actual changes made

NOTE: Extra pages in the group edit widget are not watched for changes yet. Requires a major refactor.
@droidmonkey droidmonkey force-pushed the hotfix/consolidated-2.4.1 branch from b349dd0 to 0c5f802 Compare April 7, 2019 16:00
@droidmonkey droidmonkey merged commit 4b1258f into release/2.4.1 Apr 7, 2019
@droidmonkey droidmonkey deleted the hotfix/consolidated-2.4.1 branch April 7, 2019 23:12
droidmonkey added a commit that referenced this pull request Apr 12, 2019
- Fix database deletion when using unsafe saves to a different file system [#2889]
- Fix opening databases with legacy key files that contain '/' [#2872]
- Fix opening database files from the command line [#2919]
- Fix crash when editing master key [#2836]
- Fix multiple issues with apply button behavior [#2947]
- Fix issues on application startup (tab order, --pw-stdin, etc.) [#2830]
- Fix building without WITH_XC_KEESHARE
- Fix reference entry coloring on macOS dark mode [#2984]
- Hide window when performing entry auto-type on macOS [#2969]
- Improve UX of update checker; reduce checks to every 7 days [#2968]
- KeeShare improvements [#2946, #2978, #2824]
- Re-enable Ctrl+C to copy password from search box [#2947]
- Add KeePassXC-Browser integration for Brave browser [#2933]
- SSH Agent: Re-Add keys on database unlock [#2982]
- SSH Agent: Only remove keys on app exit if they are removed on lock [#2985]
- CLI: Add --no-password option [#2708]
- CLI: Improve database extraction to XML [#2698]
- CLI: Don't call mandb on build [#2774]
- CLI: Add debug info [#2714]
- Improve support for Snap theming [#2832]
- Add support for building on Haiku OS [#2859]
- Ctrl+PgDn now goes to the next tab and Ctrl+PgUp to the previous
- Fix compiling on GCC 5 / Xenial [#2990]
- Add .gitrev output to tarball for third-party builds [#2970]
- Add WITH_XC_UPDATECHECK compile flag to toggle the update checker [#2968]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants