From 5b16988cdc7903fe8464281896a12df822ea07cb Mon Sep 17 00:00:00 2001
From: baku <baku@mozilla.com>
Date: Fri, 6 Jan 2017 15:31:22 +0100
Subject: [PATCH] Title and cursor attributes in popup UI

---
 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">