From 03e615bcc9255e18aefa40eab37ad5c27632b4cb Mon Sep 17 00:00:00 2001
From: Felix Niklas <mrflix@gmail.com>
Date: Tue, 20 Jan 2015 14:31:05 +0100
Subject: [PATCH] add a shadow on overflowing notifications popover

and fix the title in chrome vs FF
---
 .../javascripts/app/lib/bootstrap/tooltip.js       |  5 +++--
 app/assets/stylesheets/zammad.css.scss             | 14 ++++++++++++--
 2 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/app/assets/javascripts/app/lib/bootstrap/tooltip.js b/app/assets/javascripts/app/lib/bootstrap/tooltip.js
index 6fd404906..06277ce4c 100644
--- a/app/assets/javascripts/app/lib/bootstrap/tooltip.js
+++ b/app/assets/javascripts/app/lib/bootstrap/tooltip.js
@@ -259,8 +259,9 @@
     }, offset), 0)
 
     var maxHeight = $('#app').height() - 18
-    if(height > maxHeight)
-      $tip.height(maxHeight)
+    if(height > maxHeight){
+      $tip.height(maxHeight).addClass('is-overflowing')
+    }
 
     $tip.addClass('in')
 
diff --git a/app/assets/stylesheets/zammad.css.scss b/app/assets/stylesheets/zammad.css.scss
index 930c4a8b6..88eb92e81 100644
--- a/app/assets/stylesheets/zammad.css.scss
+++ b/app/assets/stylesheets/zammad.css.scss
@@ -2725,13 +2725,23 @@ footer {
     &.is-visible {
       @extend .vertical;
     }
+
+    .popover-content {
+      @extend .flex;
+    }
+
+    &.is-overflowing .popover-notificationsHeader {
+      box-shadow: 
+        0 1px hsla(240,4%,95%,.5),
+        0 2px hsla(240,4%,95%,.2);
+    }
   }
 
   .popover-notificationsHeader {
     @extend .horizontal;
     @extend .end;
-    @extend .popover-title;
-    padding-bottom: 29px;
+    padding-bottom: 22px;
+    margin: 21px 17px 0;
     
     .popover-title {
       @extend h1;