Skip to content

[PM-41909] Remove desktop UI settings dialog feature flag - #22402

Open
djsmith85 wants to merge 6 commits into
mainfrom
ps/pm-41909/remove-desktop-ui-settings-dialog-feature-flag
Open

[PM-41909] Remove desktop UI settings dialog feature flag#22402
djsmith85 wants to merge 6 commits into
mainfrom
ps/pm-41909/remove-desktop-ui-settings-dialog-feature-flag

Conversation

@djsmith85

Copy link
Copy Markdown
Contributor

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-41909

📔 Objective

Remove the feature flag and flagged code around the new desktop settings UI which was part of the Desktop UI Refresh work.

Changes

@djsmith85
djsmith85 requested a review from Hinton August 13, 2026 15:04
@djsmith85 djsmith85 added desktop Desktop Application feature-flag t:feature Change Type - Feature Development labels Aug 13, 2026
Hinton
Hinton previously approved these changes Aug 13, 2026

@Hinton Hinton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👋 Bye settings.

Hinton
Hinton previously approved these changes Aug 13, 2026
@codecov

codecov Bot commented Aug 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 54.01%. Comparing base (8711032) to head (1e1b775).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
apps/desktop/src/app/app.component.ts 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #22402      +/-   ##
==========================================
- Coverage   54.05%   54.01%   -0.04%     
==========================================
  Files        4307     4306       -1     
  Lines      136733   136465     -268     
  Branches    21609    21577      -32     
==========================================
- Hits        73910    73711     -199     
+ Misses      57430    57364      -66     
+ Partials     5393     5390       -3     

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

…m-41909/remove-desktop-ui-settings-dialog-feature-flag
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

desktop Desktop Application feature-flag t:feature Change Type - Feature Development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants