From d01e2665b163bb5eb315a400f9dadcdf709181aa Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn?= <bjoern.korella@markenfilm-crossing.de>
Date: Fri, 4 Sep 2015 10:32:54 +0200
Subject: simplify show/hide functions; change function order; add close event
 to ESC

---
 js/viewer.js | 23 +++++++++++++----------
 1 file changed, 13 insertions(+), 10 deletions(-)

(limited to 'js/viewer.js')

diff --git a/js/viewer.js b/js/viewer.js
index 42b7fd9..5563980 100755
--- a/js/viewer.js
+++ b/js/viewer.js
@@ -6,24 +6,27 @@ var videoViewer = {
 		'<source type="%type%" src="%src%" />' +
 		'</video>',
 		show : function () {
+			// insert HTML
 			$('<div id="videoplayer_overlay" style="display:none;"><div id="videoplayer_outer_container"><div id="videoplayer_container"><div id="videoplayer"><a class="box-close" id="box-close" href="#"></a></div></div></div></div>').appendTo('body');
-
-			$('#videoplayer_overlay').fadeIn('fast',function(){
-				$('#videoplayer_container').fadeIn('fast');
-			});
-			$('#box-close').click(videoViewer.hidePlayer);
 			var playerView = videoViewer.UI.playerTemplate
 								.replace(/%type%/g, videoViewer.mime)
 								.replace(/%src%/g, videoViewer.location)
 			;
 			$(playerView).prependTo('#videoplayer');
+			// add event to close icon
+			$('#box-close').click(videoViewer.hidePlayer);
+			// add event to ESC key
+			$(document).keyup(function(e) {
+			  if (e.keyCode === 27) {
+			    videoViewer.hidePlayer();
+			  }
+			});
+			// show elements
+			$('#videoplayer_overlay').fadeIn('fast');
 		},
 		hide : function() {
-			$('#videoplayer_container').fadeOut('fast', function() {
-				$('#videoplayer_overlay').fadeOut('fast', function() {
-					$('#videoplayer_container').remove();
-					$('#videoplayer_overlay').remove();
-				});
+			$('#videoplayer_overlay').fadeOut('fast', function() {
+				$('#videoplayer_overlay').remove();
 			});
 		}
 	},
-- 
cgit v1.2.3