Repository navigation
Conversation
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.
|
Thank you for your contribution! We've added this to our internal tracking system for review. Details on our contribution process can be found here: https://contributing.bitwarden.com/contributing/pull-requests/community-pr-process. |
iinuwa
left a comment
There was a problem hiding this comment.
@djsmith85, this looks good to me.
djsmith85
left a comment
There was a problem hiding this comment.
Thank you for your contribution @hammadxcm
Changes are good to go
Codecov Report✅ All modified and coverable lines are covered by tests. 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. 🚀 New features to boost your workflow:
|
iinuwa
left a comment
There was a problem hiding this comment.
Thanks for the fixup @djsmith85!
LGTM
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>
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>
🎟️ 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:
<uap:LockScreen Notification="badgeAndTileText" BadgeLogo="assets\BadgeLogo.png" />element fromapps/desktop/custom-appx-manifest.xml.apps/desktop/resources/appx/BadgeLogo.pngasset, so electron-builder no longer injects an equivalent<uap:LockScreen>element into the manifest it generates (electron-builder adds that element automatically whenever aBadgeLogo.pngasset is present)..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.