Skip to content

[PM-34833] bug: Search improvements#6776

Merged
david-livefront merged 6 commits intobitwarden:mainfrom
gabrbrand:search-improvements
Apr 10, 2026
Merged

[PM-34833] bug: Search improvements#6776
david-livefront merged 6 commits intobitwarden:mainfrom
gabrbrand:search-improvements

Conversation

@gabrbrand
Copy link
Copy Markdown
Contributor

@gabrbrand gabrbrand commented Apr 9, 2026

📔 Objective

Improvements to the search experience:

  • Show search icon only when content is displayed. Hidden when content is loading, there is an error or there are no items.
  • Hide clear icon when search term is empty. (Animated)
  • Use a formatted search string instead of hard coding the order. e.g. in German the words are in a different order

📸 Screenshots

Screen_Recording_20260409_160958_Bitwarden.Dev.mp4

@gabrbrand gabrbrand requested review from a team and david-livefront as code owners April 9, 2026 14:28
@bitwarden-bot
Copy link
Copy Markdown

Thank you for your contribution! We've added this to our internal tracking system for review.
ID: PM-34833
Link: https://bitwarden.atlassian.net/browse/PM-34833

Details on our contribution process can be found here: https://contributing.bitwarden.com/contributing/pull-requests/community-pr-process.

@bitwarden-bot bitwarden-bot changed the title Search improvements [PM-34833] Search improvements Apr 9, 2026
@david-livefront david-livefront changed the title [PM-34833] Search improvements [PM-34833] bug: Search improvements Apr 9, 2026
@david-livefront david-livefront added the t:bug Change Type - Bug label Apr 9, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.32%. Comparing base (de47c50) to head (52b3a4c).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6776      +/-   ##
==========================================
+ Coverage   84.85%   85.32%   +0.46%     
==========================================
  Files         844      958     +114     
  Lines       59037    60912    +1875     
  Branches     8563     8653      +90     
==========================================
+ Hits        50097    51972    +1875     
+ Misses       5958     5920      -38     
- Partials     2982     3020      +38     
Flag Coverage Δ
app-data 17.51% <0.00%> (-0.02%) ⬇️
app-ui-auth-tools 20.70% <9.52%> (+0.28%) ⬆️
app-ui-platform 15.90% <47.61%> (+0.59%) ⬆️
app-ui-vault 26.56% <19.04%> (+0.67%) ⬆️
authenticator 6.54% <23.80%> (-0.03%) ⬇️
lib-core-network-bridge 4.29% <0.00%> (+0.04%) ⬆️
lib-data-ui 1.03% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

Logo
Checkmarx One – Scan Summary & Details4fb30338-7803-4d24-a19d-5b4d4bd12641

Great job! No new security vulnerabilities introduced in this pull request

@david-livefront david-livefront added this pull request to the merge queue Apr 10, 2026
Merged via the queue into bitwarden:main with commit 954ac5b Apr 10, 2026
36 of 41 checks passed
@gabrbrand gabrbrand deleted the search-improvements branch April 10, 2026 14:57
@david-livefront
Copy link
Copy Markdown
Collaborator

Thanks @gabrbrand

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

Labels

community-pr t:bug Change Type - Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants