From 4a35930e2dcd7032edaeaffd250d2361d63d0b40 Mon Sep 17 00:00:00 2001
From: "K. Sasa" <kznr.luk@gmail.com>
Date: Tue, 28 Jul 2020 02:22:56 +0900
Subject: [PATCH] Consistent: Replaced the clipboard icon with a clone icon to
 improve UX  (#1344)

* Replace clone item button fa-clone to fa-files-o

* Replace copy value button fa-clipboard to fa-clone
---
 .../components/action-buttons.component.html     |  2 +-
 .../password-generator-history.component.html    |  2 +-
 src/popup/vault/password-history.component.html  |  2 +-
 src/popup/vault/view.component.html              | 16 ++++++++--------
 4 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/popup/components/action-buttons.component.html b/src/popup/components/action-buttons.component.html
index 54039a3057..f6c225f51b 100644
--- a/src/popup/components/action-buttons.component.html
+++ b/src/popup/components/action-buttons.component.html
@@ -31,6 +31,6 @@
 <ng-container *ngIf="cipher.type === cipherType.SecureNote">
     <span class="row-btn" appStopClick appStopProp appA11yTitle="{{'copyNote' | i18n}}"
         (click)="copy(cipher, cipher.notes, 'note', 'Note')" [ngClass]="{disabled: !cipher.notes}">
-        <i class="fa fa-lg fa-clipboard" aria-hidden="true"></i>
+        <i class="fa fa-lg fa-clone" aria-hidden="true"></i>
     </span>
 </ng-container>
diff --git a/src/popup/generator/password-generator-history.component.html b/src/popup/generator/password-generator-history.component.html
index 48190a650a..f2d93bc27f 100644
--- a/src/popup/generator/password-generator-history.component.html
+++ b/src/popup/generator/password-generator-history.component.html
@@ -28,7 +28,7 @@
                 <div class="action-buttons">
                     <a class="row-btn" href="#" appStopClick appA11yTitle="{{'copyPassword' | i18n}}"
                         (click)="copy(h.password)">
-                        <i class="fa fa-lg fa-clipboard" aria-hidden="true"></i>
+                        <i class="fa fa-lg fa-clone" aria-hidden="true"></i>
                     </a>
                 </div>
             </div>
diff --git a/src/popup/vault/password-history.component.html b/src/popup/vault/password-history.component.html
index 5637dc9d22..6072d2613c 100644
--- a/src/popup/vault/password-history.component.html
+++ b/src/popup/vault/password-history.component.html
@@ -22,7 +22,7 @@
                 <div class="action-buttons">
                     <a class="row-btn" href="#" appStopClick appA11yTitle="{{'copyPassword' | i18n}}"
                         (click)="copy(h.password)">
-                        <i class="fa fa-lg fa-clipboard" aria-hidden="true"></i>
+                        <i class="fa fa-lg fa-clone" aria-hidden="true"></i>
                     </a>
                 </div>
             </div>
diff --git a/src/popup/vault/view.component.html b/src/popup/vault/view.component.html
index 946a7d7a5d..061ca996ac 100644
--- a/src/popup/vault/view.component.html
+++ b/src/popup/vault/view.component.html
@@ -31,7 +31,7 @@
                     <div class="action-buttons">
                         <a class="row-btn" href="#" appStopClick appA11yTitle="{{'copyUsername' | i18n}}"
                             (click)="copy(cipher.login.username, 'username', 'Username')">
-                            <i class="fa fa-lg fa-clipboard" aria-hidden="true"></i>
+                            <i class="fa fa-lg fa-clone" aria-hidden="true"></i>
                         </a>
                     </div>
                 </div>
@@ -62,7 +62,7 @@
                         <a class="row-btn" href="#" appStopClick appA11yTitle="{{'copyPassword' | i18n}}"
                             (click)="copy(cipher.login.password, 'password', 'Password')"
                             *ngIf="cipher.viewPassword">
-                            <i class="fa fa-lg fa-clipboard" aria-hidden="true"></i>
+                            <i class="fa fa-lg fa-clone" aria-hidden="true"></i>
                         </a>
                     </div>
                 </div>
@@ -86,7 +86,7 @@
                     <div class="action-buttons">
                         <a class="row-btn" href="#" appStopClick appA11yTitle="{{'copyVerificationCode' | i18n}}"
                             (click)="copy(totpCode, 'verificationCodeTotp', 'TOTP')">
-                            <i class="fa fa-lg fa-clipboard" aria-hidden="true"></i>
+                            <i class="fa fa-lg fa-clone" aria-hidden="true"></i>
                         </a>
                     </div>
                 </div>
@@ -105,7 +105,7 @@
                     <div class="action-buttons">
                         <a class="row-btn" href="#" appStopClick appA11yTitle="{{'copyNumber' | i18n}}"
                             (click)="copy(cipher.card.number, 'number', 'Number')">
-                            <i class="fa fa-lg fa-clipboard" aria-hidden="true"></i>
+                            <i class="fa fa-lg fa-clone" aria-hidden="true"></i>
                         </a>
                     </div>
                 </div>
@@ -131,7 +131,7 @@
                         </a>
                         <a class="row-btn" href="#" appStopClick appA11yTitle="{{'copySecurityCode' | i18n}}"
                             (click)="copy(cipher.card.code, 'securityCode', 'Security Code')">
-                            <i class="fa fa-lg fa-clipboard" aria-hidden="true"></i>
+                            <i class="fa fa-lg fa-clone" aria-hidden="true"></i>
                         </a>
                     </div>
                 </div>
@@ -199,7 +199,7 @@
                     </a>
                     <a class="row-btn" href="#" appStopClick appA11yTitle="{{'copyUri' | i18n}}"
                         (click)="copy(u.uri, u.isWebsite ? 'website' : 'uri', 'URI')">
-                        <i class="fa fa-lg fa-clipboard" aria-hidden="true"></i>
+                        <i class="fa fa-lg fa-clone" aria-hidden="true"></i>
                     </a>
                 </div>
             </div>
@@ -245,7 +245,7 @@
                     <a class="row-btn" href="#" appStopClick appA11yTitle="{{'copyValue' | i18n}}"
                         *ngIf="field.value && field.type !== fieldType.Boolean && !(field.type === fieldType.Hidden && !cipher.viewPassword)"
                         (click)="copy(field.value, 'value', field.type === fieldType.Hidden ? 'H_Field' : 'Field')">
-                        <i class="fa fa-lg fa-clipboard" aria-hidden="true"></i>
+                        <i class="fa fa-lg fa-clone" aria-hidden="true"></i>
                     </a>
                 </div>
             </div>
@@ -272,7 +272,7 @@
                 *ngIf="!cipher.organizationId && !cipher.isDeleted">
                 <div class="row-main text-primary">
                     <div class="icon text-primary" aria-hidden="true">
-                        <i class="fa fa-clone fa-lg fa-fw"></i>
+                        <i class="fa fa-files-o fa-lg fa-fw"></i>
                     </div>
                     <span>{{'cloneItem' | i18n}}</span>
                 </div>