From 123e628c19146745d78788a403eb5a683e78642a Mon Sep 17 00:00:00 2001
From: Felix Niklas <mrflix@gmail.com>
Date: Tue, 16 Dec 2014 15:35:49 +0100
Subject: [PATCH] make scrollHeader fully dissapear

by using scroll offset of the scroll parent (previously the headers bottom border was still visible because it only scrolled up 62px instead its full height of 64px)
---
 .../app/controllers/ticket_zoom.js.coffee       | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/app/assets/javascripts/app/controllers/ticket_zoom.js.coffee b/app/assets/javascripts/app/controllers/ticket_zoom.js.coffee
index a2f6415b5..abbfbb4de 100644
--- a/app/assets/javascripts/app/controllers/ticket_zoom.js.coffee
+++ b/app/assets/javascripts/app/controllers/ticket_zoom.js.coffee
@@ -160,23 +160,22 @@ class App.TicketZoom extends App.Controller
     @main.unbind('scroll', @positionPageHeaderUpdate)
 
   positionPageHeaderUpdate: =>
-    pageHeader       = @scrollPageHeader.height()
+    headerHeight     = @scrollPageHeader.outerHeight()
     mainScrollHeigth = @main.prop('scrollHeight')
     mainHeigth       = @main.height()
 
     # if page header is possible to use, show page header
     top = 0
-    if mainScrollHeigth > mainHeigth + pageHeader
-      offset = @ticketZoom.offset()
-      if offset.top >= 0
-        top = offset.top
+    if mainScrollHeigth > mainHeigth + headerHeight
+      scroll = @main.scrollTop()
+      if scroll <= headerHeight
+        top = (scroll - headerHeight)
 
     # if page header is not possible to use - mainScrollHeigth to low - hide page header
     else
-      top = pageHeader
-
-    #console.log('TOP', top, @ticket.id, new Date)
-    @scrollPageHeader.css('transform', "translateY(-#{top}px)")
+      top = -headerHeight
+    
+    @scrollPageHeader.css('transform', "translateY(#{top}px)")
 
   render: (force) =>