Skip to content

[PM-38964] Remove obsolete LockScreen config from AppX manifest - #21229

Merged
djsmith85 merged 4 commits into
bitwarden:mainfrom
hammadxcm:fix/16746-remove-obsolete-lockscreen-appx
Aug 24, 2026
Merged

djsmith85 merged 4 commits into
bitwarden:mainfrom
hammadxcm:fix/16746-remove-obsolete-lockscreen-appx

Conversation

@hammadxcm

Copy link
Copy Markdown
Contributor

🎟️ Tracking

Closes #16746

📔 Objective

The desktop client no longer supports live tiles or badge notifications on Windows 10/11, yet Bitwarden still shows up as a selectable "Lock screen status" app under Windows Settings → Personalization → Lock screen (ms-settings:lockscreen) when installed from the Microsoft Store.

This PR removes the obsolete configuration:

  • Removes the <uap:LockScreen Notification="badgeAndTileText" BadgeLogo="assets\BadgeLogo.png" /> element from apps/desktop/custom-appx-manifest.xml.
  • Deletes the now-unused apps/desktop/resources/appx/BadgeLogo.png asset, so electron-builder no longer injects an equivalent <uap:LockScreen> element into the manifest it generates (electron-builder adds that element automatically whenever a BadgeLogo.png asset is present).
  • Removes the corresponding entry from .github/whitelist-capital-letters.txt.

After this change Bitwarden no longer appears as a lock screen status app option.

Note: a previous PR with the same change (#18137) was approved by maintainers but closed by its author; this PR re-applies that change on top of the current main.

The desktop client no longer supports live tiles or badge notifications
on Windows 10/11, but the LockScreen declaration in the AppX manifest
still causes Bitwarden to appear as a selectable lock screen status app
in Windows Settings.

Remove the LockScreen element from the custom AppX manifest template and
delete the now-unused BadgeLogo asset so electron-builder no longer
injects the equivalent element into the generated manifest.
@hammadxcm
hammadxcm requested a review from a team as a code owner June 12, 2026 13:25
@hammadxcm
hammadxcm requested a review from dani-garcia June 12, 2026 13:25
@CLAassistant

CLAassistant commented Jun 12, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@bitwarden-bot

Copy link
Copy Markdown
Collaborator

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

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 Remove obsolete LockScreen config from AppX manifest [PM-38964] Remove obsolete LockScreen config from AppX manifest Jun 12, 2026
@bitwarden-bot bitwarden-bot added community-pr needs-qa Marks a PR as requiring QA approval labels Jun 12, 2026
@djsmith85
djsmith85 requested review from djsmith85 and removed request for dani-garcia June 12, 2026 17:04
iinuwa
iinuwa previously approved these changes Aug 20, 2026

@iinuwa iinuwa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@djsmith85, this looks good to me.

@djsmith85 djsmith85 added the desktop Desktop Application label Aug 22, 2026
djsmith85
djsmith85 previously approved these changes Aug 22, 2026

@djsmith85 djsmith85 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for your contribution @hammadxcm

Changes are good to go

@djsmith85 djsmith85 added t:bugfix Change Type - Bugfix and removed needs-qa Marks a PR as requiring QA approval labels Aug 22, 2026
@djsmith85
djsmith85 enabled auto-merge (squash) August 22, 2026 10:54
@djsmith85
djsmith85 dismissed stale reviews from iinuwa and themself via 0611a94 August 22, 2026 11:26
@djsmith85
djsmith85 self-requested a review August 22, 2026 11:26

@djsmith85 djsmith85 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

#22564 introduces a removal of the uap:LockScreen entry when found. As this PR removes the whole line and BadgeLogo the check can also be removed. Done with 0611a94

@djsmith85
djsmith85 requested a review from iinuwa August 22, 2026 11:28
@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@3c2e780). Learn more about missing BASE report.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #21229   +/-   ##
=======================================
  Coverage        ?   54.58%           
=======================================
  Files           ?     4309           
  Lines           ?   137619           
  Branches        ?    21796           
=======================================
  Hits            ?    75116           
  Misses          ?    57019           
  Partials        ?     5484           

☔ 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.

@iinuwa iinuwa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the fixup @djsmith85!

LGTM

@djsmith85
djsmith85 merged commit 450b54a into bitwarden:main Aug 24, 2026
118 of 122 checks passed
addisonbeck pushed a commit that referenced this pull request Aug 26, 2026
The desktop client no longer supports live tiles or badge notifications
on Windows 10/11, but the LockScreen declaration in the AppX manifest
still causes Bitwarden to appear as a selectable lock screen status app
in Windows Settings.

Remove the LockScreen element from the custom AppX manifest template and
delete the now-unused BadgeLogo asset so electron-builder no longer
injects the equivalent element into the generated manifest.

Co-authored-by: Daniel James Smith <2670567+djsmith85@users.noreply.github.com>
Richard102257 pushed a commit to Richard102257/bitwarden-clients that referenced this pull request Sep 15, 2026
The desktop client no longer supports live tiles or badge notifications
on Windows 10/11, but the LockScreen declaration in the AppX manifest
still causes Bitwarden to appear as a selectable lock screen status app
in Windows Settings.

Remove the LockScreen element from the custom AppX manifest template and
delete the now-unused BadgeLogo asset so electron-builder no longer
injects the equivalent element into the generated manifest.

Co-authored-by: Daniel James Smith <2670567+djsmith85@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-pr desktop Desktop Application t:bugfix Change Type - Bugfix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove obsolete LockScreen config from Microsoft store installation

5 participants