[PR #3148] [PM-7369] Show passkey icon on android when the item has a Fido2 credential #4051

Closed
opened 2025-11-26 23:40:03 -06:00 by GiteaMirror · 0 comments
Owner

Original Pull Request: https://github.com/bitwarden/android/pull/3148

State: closed
Merged: Yes


Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Android is not showing passkey icon when "show website icons" is disabled

Code changes

Changed IconGlyphExtensions to take into account if the cipher has Fido2Credential and return passkey icon in that scenario.

  • IconGlyphExtensions.cs:

Screenshots

Before After
Screenshot_1712768117 Screenshot_1712767855

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team
**Original Pull Request:** https://github.com/bitwarden/android/pull/3148 **State:** closed **Merged:** Yes --- ## Type of change - [x] Bug fix - [ ] New feature development - [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc) - [ ] Build/deploy pipeline (DevOps) - [ ] Other ## Objective Android is not showing passkey icon when "show website icons" is disabled ## Code changes Changed IconGlyphExtensions to take into account if the cipher has Fido2Credential and return passkey icon in that scenario. * **IconGlyphExtensions.cs:** ## Screenshots Before | After :-------------------------:|:-------------------------: ![Screenshot_1712768117](https://github.com/bitwarden/mobile/assets/2824952/0d83f975-62bd-4762-aa1c-99b330b20ee3) | ![Screenshot_1712767855](https://github.com/bitwarden/mobile/assets/2824952/ddab0c83-70d4-4644-9a79-18112ab2288b) ## Before you submit - Please check for formatting errors (`dotnet format --verify-no-changes`) (required) - Please add **unit tests** where it makes sense to do so (encouraged but not required) - If this change requires a **documentation update** - notify the documentation team - If this change has particular **deployment requirements** - notify the DevOps team
GiteaMirror added the pull-request label 2025-11-26 23:40:03 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/android#4051