Skip to content
New issue

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

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

Already on GitHub? # to your account

Support tearing off tags menu #11652

Merged
merged 2 commits into from
Feb 23, 2025
Merged

Support tearing off tags menu #11652

merged 2 commits into from
Feb 23, 2025

Conversation

droidmonkey
Copy link
Member

  • Closes Tags multiselect right click context menu #11649 - tags menu can be torn off to set and unset tags without having to dive into the context menu every time.

  • Tags menu will hide when database is locked or view is switched away from the main database view (eg, settings)

Screenshots

See #11649

Testing strategy

Tested on windows manually

Type of change

  • ✅ New feature (change that adds functionality)

@droidmonkey droidmonkey added feature: Tags pr: new feature Pull request that adds a new feature labels Jan 12, 2025
@droidmonkey droidmonkey added this to the v2.7.10 milestone Jan 12, 2025
Copy link

codecov bot commented Jan 12, 2025

Codecov Report

Attention: Patch coverage is 3.84615% with 25 lines in your changes missing coverage. Please review.

Project coverage is 63.61%. Comparing base (41b6247) to head (e233b74).
Report is 4 commits behind head on develop.

Files with missing lines Patch % Lines
src/gui/MainWindow.cpp 3.85% 25 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #11652      +/-   ##
===========================================
- Coverage    63.66%   63.61%   -0.05%     
===========================================
  Files          367      367              
  Lines        38868    38886      +18     
===========================================
- Hits         24745    24737       -8     
- Misses       14123    14149      +26     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@xboxones1
Copy link
Contributor

@droidmonkey, In Linux, clicking on a tag from the context menu results in a segmentation fault.

@varjolintu
Copy link
Member

varjolintu commented Jan 13, 2025

I'm also having Exception: EXC_BAD_ACCESS (code=1, address=0x9) with macOS:

* Closes #11649 - tags menu can be torn off to set and unset tags without having to dive into the context menu every time.

* Tags menu will hide when database is locked or view is switched away from the main database view (eg, settings)
@droidmonkey droidmonkey merged commit e4bb80b into develop Feb 23, 2025
11 checks passed
@droidmonkey droidmonkey deleted the feature/tags-tearoff branch February 23, 2025 01:44
droidmonkey added a commit that referenced this pull request Feb 23, 2025
* Support tearing off tags menu
* Closes #11649 - tags menu can be torn off to set and unset tags without having to dive into the context menu every time.
* Tags menu will hide when database is locked or view is switched away from the main database view (eg, settings)
pull bot pushed a commit to blog2i2j/keepassxreboot.._..keepassxc that referenced this pull request Feb 23, 2025
* Support tearing off tags menu
* Closes keepassxreboot#11649 - tags menu can be torn off to set and unset tags without having to dive into the context menu every time.
* Tags menu will hide when database is locked or view is switched away from the main database view (eg, settings)
jopamo pushed a commit to jopamo/keepassxc that referenced this pull request Mar 1, 2025
* Support tearing off tags menu
* Closes keepassxreboot#11649 - tags menu can be torn off to set and unset tags without having to dive into the context menu every time.
* Tags menu will hide when database is locked or view is switched away from the main database view (eg, settings)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
feature: Tags pr: new feature Pull request that adds a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tags multiselect right click context menu
4 participants