From 2fd1af29ec2facd3d20c2c1e9d4d397dccd47774 Mon Sep 17 00:00:00 2001
From: Piotr Sarnacki <drogus@gmail.com>
Date: Wed, 18 Jun 2014 15:36:23 +0200
Subject: [PATCH] Unfold lines before trying to scroll

(closes travis-ci/travis-ci#1501)
---
 assets/scripts/app/views/log.coffee | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/assets/scripts/app/views/log.coffee b/assets/scripts/app/views/log.coffee
index e7cc47db..61e9e3cd 100644
--- a/assets/scripts/app/views/log.coffee
+++ b/assets/scripts/app/views/log.coffee
@@ -46,12 +46,16 @@ Travis.reopen
 
     createEngine: ->
       console.log 'log view: create engine' if Log.DEBUG
-      @scroll = new Log.Scroll
+      @scroll = new Log.Scroll beforeScroll: =>
+        @unfoldHighlight()
       @engine = Log.create(limit: Log.LIMIT, listeners: [@scroll])
       @logFolder = new Travis.LogFolder(@$().find('#log'))
       @lineSelector = new Travis.LinesSelector(@$().find('#log'), @scroll, @logFolder)
       @observeParts()
 
+    unfoldHighlight: ->
+      @lineSelector.unfoldLines()
+
     observeParts: ->
       parts = @get('log.parts')
       parts.addArrayObserver(@, didChange: 'partsDidChange', willChange: 'noop')
@@ -85,7 +89,10 @@ Travis.reopen
 
     noop: -> # TODO required?
 
-Log.Scroll = ->
+Log.Scroll = (options) ->
+  options ||= {}
+  @beforeScroll = options.beforeScroll
+  this
 Log.Scroll.prototype = $.extend new Log.Listener,
   insert: (log, data, pos) ->
     @tryScroll() if @numbers
@@ -93,6 +100,8 @@ Log.Scroll.prototype = $.extend new Log.Listener,
 
   tryScroll: ->
     if element = $("#log p:visible.highlight:first")
+      if @beforeScroll
+        @beforeScroll()
       $('#main').scrollTop(0)
       $('html, body').scrollTop(element.offset()?.top - (window.innerHeight / 3)) # weird, html works in chrome, body in firefox