From 155e8b86396e44c3d79e57cc7873ff3837c2a6d7 Mon Sep 17 00:00:00 2001
From: Andrea Marchesini <baku@ippolita.net>
Date: Fri, 6 Jan 2017 15:34:44 +0100
Subject: [PATCH] Title and cursor attributes in popup UI (#35)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

lgtm 👍
---
 webextension/css/popup.css | 1 +
 webextension/js/popup.js   | 6 +++++-
 webextension/popup.html    | 2 +-
 3 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/webextension/css/popup.css b/webextension/css/popup.css
index 7e1d806..f81f463 100644
--- a/webextension/css/popup.css
+++ b/webextension/css/popup.css
@@ -26,6 +26,7 @@ table {
 table.unstriped tbody tr {
   border-bottom: 1px solid #f1f1f1;
   background-color: #fefefe;
+  cursor: pointer;
 }
 
 .userContext-icon {
diff --git a/webextension/js/popup.js b/webextension/js/popup.js
index e5cb33b..0978b12 100644
--- a/webextension/js/popup.js
+++ b/webextension/js/popup.js
@@ -59,10 +59,14 @@ browser.runtime.sendMessage({method: 'query'}).then(identities=> {
       </td>
       <td>${identity.name}</td>
       <td class="newtab">
-        <img src="/img/container-add.svg" class="icon newtab-icon" />
+        <img
+          title="Open a new ${identity.name} container tab"
+          src="/img/container-add.svg"
+          class="icon newtab-icon" />
       </td>
       <td class="hideorshow" >
         <img
+          title="Hide or show ${identity.name} container tabs"
           data-identity-cookie-store-id="${identity.cookieStoreId}"
           id="${identity.cookieStoreId}-hideorshow-icon"
           class="icon hideorshow-icon"
diff --git a/webextension/popup.html b/webextension/popup.html
index 9edcc6f..4b69a48 100644
--- a/webextension/popup.html
+++ b/webextension/popup.html
@@ -14,7 +14,7 @@
       <h4>Containers</h4>
     </div>
     <div class="small-2 columns header-column">
-      <a id="sort-containers-link"><img alt="Sort Containers" class="icon" src="/img/container-sort.svg"></a>
+      <a id="sort-containers-link" title="Sort Containers"><img alt="Sort Containers" class="icon" src="/img/container-sort.svg"></a>
     </div>
   </div>
   <table class="unstriped hover">