From d0dab49bc2ec70d21eb3b640f8219253374a6f17 Mon Sep 17 00:00:00 2001
From: Adrian <adrian.dev@kousz.ch>
Date: Sat, 21 Jul 2018 23:24:11 +0200
Subject: [PATCH] Click on user name fills in the password

---
 data/panel/panel.html      |  6 ++----
 data/panel/panel.scss      | 16 +++-------------
 data/panel/passwordList.js |  4 ++--
 locale/en-US.properties    |  1 +
 manifest.json              |  2 +-
 5 files changed, 9 insertions(+), 20 deletions(-)

diff --git a/data/panel/panel.html b/data/panel/panel.html
index 369778b..e3e8c31 100644
--- a/data/panel/panel.html
+++ b/data/panel/panel.html
@@ -85,12 +85,10 @@ <h1 data-l10n-id="migration_title"></h1>
       <div id="password-template" hidden>
         <div class="password-container">
           <a href="#" class="password-menu-link iconic-link" data-l10n-id="password_menu"></a>
-          <a href="#" class="to-document-link iconic-link" data-l10n-id="to_document"></a>
-          <a href="#" class="to-clipboard-link iconic-link" data-l10n-id="to_clipboard"></a>
-          <span class="user-name-container">
+          <div class="user-name-container to-document-link">
             <span class="user-name"></span>
             <span class="password-revision"></span>
-          </span>
+          </div>
         </div>
       </div>
 
diff --git a/data/panel/panel.scss b/data/panel/panel.scss
index 3ec1b3d..d429352 100644
--- a/data/panel/panel.scss
+++ b/data/panel/panel.scss
@@ -229,11 +229,6 @@ a
   display: flex;
   flex-direction: row;
   align-items: center;
-
-  &:hover
-  {
-    background-color: $hover-background;
-  }
 }
 
 .user-name-container
@@ -280,14 +275,10 @@ a
   background-color: 0.7 * $background-color + 0.3 * $text-color;
 }
 
-.to-document-link
+.to-document-link:hover
 {
-  background-image: url(../images/ios-color-wand.svg);
-}
-
-.to-clipboard-link
-{
-  background-image: url(../images/archive.svg);
+  background-color: $hover-background;
+  cursor: pointer;
 }
 
 .pwshow-link
@@ -376,7 +367,6 @@ a
   }
 }
 
-:root.webclient .to-document-link,
 :root.webclient #menu-to-document
 {
   display: none;
diff --git a/data/panel/passwordList.js b/data/panel/passwordList.js
index 7c8996e..befa2c1 100644
--- a/data/panel/passwordList.js
+++ b/data/panel/passwordList.js
@@ -198,7 +198,8 @@ function showPasswords()
       let tooltip;
       if (password.type == "generated2" || password.type == "generated")
       {
-        tooltip = i18n.getMessage("password_type_" + password.type);
+        tooltip = i18n.getMessage("to_document_tooltip");
+        tooltip += "\n" + i18n.getMessage("password_type_" + password.type);
         if (password.type == "generated")
           tooltip += "\n" + i18n.getMessage("password_type_generated_replace");
 
@@ -224,7 +225,6 @@ function showPasswords()
       let entry = template.cloneNode(true);
       setCommandHandler(entry.querySelector(".password-menu-link"), toggleMenu.bind(null, password, entry));
       setCommandHandler(entry.querySelector(".to-document-link"), fillInPassword.bind(null, password));
-      setCommandHandler(entry.querySelector(".to-clipboard-link"), copyToClipboard.bind(null, password));
 
       if (password.type == "generated")
       {
diff --git a/locale/en-US.properties b/locale/en-US.properties
index 6bde9cf..b6aac44 100644
--- a/locale/en-US.properties
+++ b/locale/en-US.properties
@@ -82,6 +82,7 @@ password_ready_message = Your password is ready, click again anywhere to copy it
 passwords_label = Passwords:
 password_menu = All actions
 to_document = Fill in
+to_document_tooltip = Click to fill in password
 to_clipboard = Copy to clipboard
 pwshow = Show password
 add_notes = Add notes
diff --git a/manifest.json b/manifest.json
index 4efc855..175d9f3 100644
--- a/manifest.json
+++ b/manifest.json
@@ -8,7 +8,7 @@
   "description": "__MSG_description__",
   "author": "Adrium",
   "homepage_url": "https://github.com/adrium/easypass.git",
-  "version": "0.10",
+  "version": "0.11",
   "permissions": [
     "tabs",
     "http://*/*",