Browse Source

Move spinner add logic into same code pathway as spinner remove logic

James Allen 10 years ago
parent
commit
a196531a58

+ 7 - 6
services/web/public/coffee/ide/pdfng/directives/pdfViewer.coffee

@@ -290,11 +290,18 @@ define [
 						rescaleTimer = null
 						rescaleTimer = null
 					, 0
 					, 0
 
 
+				spinnerTimer = null
 				doRescale = (scale) ->
 				doRescale = (scale) ->
 					# console.log 'doRescale', scale
 					# console.log 'doRescale', scale
 					return unless scale?
 					return unless scale?
 					origposition = angular.copy scope.position
 					origposition = angular.copy scope.position
 					# console.log 'origposition', origposition
 					# console.log 'origposition', origposition
+					
+					if not spinnerTimer?
+						spinnerTimer = setTimeout () ->
+							spinner.add(element)
+							spinnerTimer = null
+						, 100
 					layoutReady.promise.then (parentSize) ->
 					layoutReady.promise.then (parentSize) ->
 						[h, w] = parentSize
 						[h, w] = parentSize
 						# console.log 'in promise', h, w
 						# console.log 'in promise', h, w
@@ -312,7 +319,6 @@ define [
 							scope.$emit 'pdf:error', error
 							scope.$emit 'pdf:error', error
 
 
 				elementTimer = null
 				elementTimer = null
-				spinnerTimer = null
 				updateLayout = () ->
 				updateLayout = () ->
 					# if element is zero-sized keep checking until it is ready
 					# if element is zero-sized keep checking until it is ready
 					# console.log 'checking element ready', element.height(), element.width()
 					# console.log 'checking element ready', element.height(), element.width()
@@ -329,11 +335,6 @@ define [
 						]
 						]
 						# console.log 'resolving layoutReady with', scope.parentSize
 						# console.log 'resolving layoutReady with', scope.parentSize
 						$timeout () ->
 						$timeout () ->
-							if not spinnerTimer?
-								spinnerTimer = setTimeout () ->
-									spinner.add(element)
-									spinnerTimer = null
-								, 100
 							layoutReady.resolve scope.parentSize
 							layoutReady.resolve scope.parentSize
 							scope.$emit 'flash-controls'
 							scope.$emit 'flash-controls'