From d2bb8ef503901b05a34a7d24ddd32c8a5b7524fc Mon Sep 17 00:00:00 2001
From: Andrey Nering <andrey.nering@gmail.com>
Date: Tue, 17 Jan 2017 23:18:55 -0200
Subject: Notifications: trying to get a better layout (#660)

* i18n button titles

* Improvements on notification page layout

* Notification count badge fixes

* Make table <tr> clickable

* Fix octicon aligment

* Fix use of AppSubUrl
---
 public/css/index.css   | 11 +++++++----
 public/js/index.js     |  5 +++++
 public/less/_user.less | 12 ++++++++----
 3 files changed, 20 insertions(+), 8 deletions(-)

(limited to 'public')

diff --git a/public/css/index.css b/public/css/index.css
index c569209e49..7c84cf8517 100644
--- a/public/css/index.css
+++ b/public/css/index.css
@@ -2712,12 +2712,15 @@ footer .ui.language .menu {
   float: left;
   margin-left: 7px;
 }
-.user.notification .buttons-panel button {
-  padding: 3px;
-}
-.user.notification .buttons-panel form {
+.user.notification table form {
   display: inline-block;
 }
+.user.notification table button {
+  padding: 3px 3px 3px 5px;
+}
+.user.notification table tr {
+  cursor: pointer;
+}
 .user.notification .octicon-issue-opened,
 .user.notification .octicon-git-pull-request {
   color: #21ba45;
diff --git a/public/js/index.js b/public/js/index.js
index bd22442e5f..8674993261 100644
--- a/public/js/index.js
+++ b/public/js/index.js
@@ -1243,6 +1243,11 @@ $(document).ready(function () {
         $($(this).data('target')).slideToggle(100);
     });
 
+    // make table <tr> element clickable like a link
+    $('tr[data-href]').click(function(event) {
+        window.location = $(this).data('href');
+    });
+
     // Highlight JS
     if (typeof hljs != 'undefined') {
         hljs.initHighlightingOnLoad();
diff --git a/public/less/_user.less b/public/less/_user.less
index 38b73f7853..d63b9b3de9 100644
--- a/public/less/_user.less
+++ b/public/less/_user.less
@@ -85,13 +85,17 @@
             margin-left: 7px;
         }
 
-        .buttons-panel {
+        table {
+            form {
+                display: inline-block;
+            }
+
             button {
-                padding: 3px;
+                padding: 3px 3px 3px 5px;
             }
 
-            form {
-                display: inline-block;
+            tr {
+                cursor: pointer;
             }
         }
 
-- 
cgit v1.2.3