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

Improve entry preview panel #7993

Merged
merged 1 commit into from
Jun 6, 2022
Merged

Improve entry preview panel #7993

merged 1 commit into from
Jun 6, 2022

Conversation

droidmonkey
Copy link
Member

Screenshots

image

Testing strategy

Tested manually

Type of change

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

@droidmonkey droidmonkey added bug user interface pr:backport pending Pull request yet to be backported to a previous release labels May 1, 2022
@droidmonkey droidmonkey added this to the v2.7.2 milestone May 1, 2022
@droidmonkey droidmonkey requested a review from phoerious May 1, 2022 19:49
@codecov-commenter
Copy link

codecov-commenter commented May 1, 2022

Codecov Report

Merging #7993 (e72c601) into develop (ba8f787) will increase coverage by 0.00%.
The diff coverage is 65.22%.

@@           Coverage Diff            @@
##           develop    #7993   +/-   ##
========================================
  Coverage    64.29%   64.29%           
========================================
  Files          339      339           
  Lines        43450    43455    +5     
========================================
+ Hits         27933    27938    +5     
  Misses       15517    15517           
Impacted Files Coverage Δ
src/gui/EntryPreviewWidget.cpp 87.57% <60.00%> (-0.11%) ⬇️
src/gui/DatabaseWidget.cpp 61.99% <100.00%> (ø)
src/core/FileWatcher.cpp 86.75% <0.00%> (+1.20%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba8f787...e72c601. Read the comment docs.

* Fix #7811 - Notes height no longer truncated
* Fix #7949 - Improve copying attribute value to clipboard in entry preview
* Fix #7898 - Prevent copying url when copy password selected after clicking url in preview pane
* Fix #7982 - Double clicking hidden attributes in preview pane copies the value instead of ●●●●●●
@varjolintu
Copy link
Member

At least on MacOS the menu item "View -> Show Preview Panel" always has text "Show Preview Panel" without any indication if it's enabled or not. Same applies to all other settings in the menu. Should the text change between Hide/Show when switching the option on/off?

@droidmonkey droidmonkey merged commit b1f4e12 into develop Jun 6, 2022
@droidmonkey droidmonkey deleted the fix/entry-preview branch June 6, 2022 11:20
@droidmonkey droidmonkey added pr:backported Pull request backported to previous release and removed pr:backport pending Pull request yet to be backported to a previous release labels Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug pr:backported Pull request backported to previous release user interface
Projects
None yet
5 participants