[PR #1935] [MERGED] Updating icons for Linked and Boolean fields #3233

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

📋 Pull Request Information

Original PR: https://github.com/bitwarden/android/pull/1935
Author: @CarleyDiaz-Bitwarden-zz
Created: 6/2/2022
Status: Merged
Merged: 6/2/2022
Merged by: @cd-bitwarden

Base: masterHead: PS-584-IOS-Icons


📝 Commits (1)

  • 00c8998 Updating icons for Linked and Boolean fields

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 src/App/Pages/Vault/ViewPageViewModel.cs (+2 -2)

📄 Description

Update to icons used for Boolean and Linked to use real icons

Type of change

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

Objective

Icons on Mobile devices aren't displaying properly for Linked and Boolean type

Code changes

ViewPageViewModel.cs
Updated to use Bitwarden defined icons for Boolean and linked types

Screenshots

Screen Shot 2022-06-02 at 1 21 30 PM

Before you submit

  • I have added unit tests where it makes sense to do so (encouraged but not required)
  • This change requires a documentation update (notify the documentation team)
  • This change has particular deployment requirements (notify the DevOps team)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/bitwarden/android/pull/1935 **Author:** [@CarleyDiaz-Bitwarden-zz](https://github.com/CarleyDiaz-Bitwarden-zz) **Created:** 6/2/2022 **Status:** ✅ Merged **Merged:** 6/2/2022 **Merged by:** [@cd-bitwarden](https://github.com/cd-bitwarden) **Base:** `master` ← **Head:** `PS-584-IOS-Icons` --- ### 📝 Commits (1) - [`00c8998`](https://github.com/bitwarden/android/commit/00c8998e33f1f3cfbe24003d62685053d63f7d78) Updating icons for Linked and Boolean fields ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/App/Pages/Vault/ViewPageViewModel.cs` (+2 -2) </details> ### 📄 Description Update to icons used for Boolean and Linked to use real icons ## Type of change - [x] Bug fix - [ ] New feature development - [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc) - [ ] Build/deploy pipeline (DevOps) - [ ] Other ## Objective Icons on Mobile devices aren't displaying properly for Linked and Boolean type ## Code changes ViewPageViewModel.cs Updated to use Bitwarden defined icons for Boolean and linked types ## Screenshots ![Screen Shot 2022-06-02 at 1 21 30 PM](https://user-images.githubusercontent.com/103955722/171690138-7b235592-34bc-4f73-a692-bea6cba2d56c.png) ## Before you submit - [ ] I have added **unit tests** where it makes sense to do so (encouraged but not required) - [ ] This change requires a **documentation update** (notify the documentation team) - [ ] This change has particular **deployment requirements** (notify the DevOps team) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2025-11-26 23:30:02 -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#3233