From c8012dc22ca5d8bc7e0008d3d5b40374996894d0 Mon Sep 17 00:00:00 2001
From: Brian <bjg@network-theory.co.uk>
Date: Tue, 20 Jan 2015 20:35:29 +0000
Subject: [PATCH] fix getVisiblePages in pdf viewer to allow optimisation

---
 .../coffee/ide/pdfng/directives/pdfViewer.coffee       | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/services/web/public/coffee/ide/pdfng/directives/pdfViewer.coffee b/services/web/public/coffee/ide/pdfng/directives/pdfViewer.coffee
index 645f033bd0..22795b0edc 100644
--- a/services/web/public/coffee/ide/pdfng/directives/pdfViewer.coffee
+++ b/services/web/public/coffee/ide/pdfng/directives/pdfViewer.coffee
@@ -247,14 +247,12 @@ define [
 				getVisiblePages = () ->
 					top = element[0].scrollTop;
 					bottom = top + element[0].clientHeight;
-					isVisible = (pageElement) ->
+					visiblePages = scope.pages.filter (page) ->
+						pageElement = page.element[0]
 						pageTop = pageElement.offsetTop
 						pageBottom = pageTop + pageElement.clientHeight
-						return pageTop < bottom and pageBottom > top
-					visiblePages = []
-					for page in scope.pages
-						page.visible = visible = isVisible(page.element[0])
-						visiblePages.push page if visible
+						page.visible = pageTop < bottom and pageBottom > top
+						return page.visible
 					return visiblePages
 
 				getExtraPages = (visiblePages) ->