From 20229c11931a0c807d9ad5e3cb08031a397e5c51 Mon Sep 17 00:00:00 2001
From: Wonseop Kim <wonseop.kim@samsung.com>
Date: Mon, 28 Jan 2013 16:27:02 +0900
Subject: [PATCH] handler: bug fix (regarding enableHandler function )

Handler was fixed a bug that sometimes appeared when it enabled by enableHandler function.

Change-Id: If46dd062ad48755fbe1e80ecfdf12d844f7b2bd9
---
 .../common/jquery.mobile.tizen.scrollview.handler.less  |  5 +++++
 .../handler/js/jquery.tizen.scrollview.handler.js       | 17 +++++++++--------
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/src/themes/tizen/common/jquery.mobile.tizen.scrollview.handler.less b/src/themes/tizen/common/jquery.mobile.tizen.scrollview.handler.less
index e0ff36a..bdcb4d4 100644
--- a/src/themes/tizen/common/jquery.mobile.tizen.scrollview.handler.less
+++ b/src/themes/tizen/common/jquery.mobile.tizen.scrollview.handler.less
@@ -5,6 +5,11 @@
 .ui-handler {
 	position : absolute;
 	overflow : hidden;
+	opacity: 0;
+}
+
+.ui-handler-visible {
+  opacity: 1;
 }
 
 .ui-handler-direction-y {
diff --git a/src/widgets/handler/js/jquery.tizen.scrollview.handler.js b/src/widgets/handler/js/jquery.tizen.scrollview.handler.js
index 84afd6e..1f47734 100644
--- a/src/widgets/handler/js/jquery.tizen.scrollview.handler.js
+++ b/src/widgets/handler/js/jquery.tizen.scrollview.handler.js
@@ -226,9 +226,9 @@
 					moveTimer = undefined;
 				}
 
-				handlerThumb.addClass( "ui-handler-visible" )
-							.stop( true, true )
-							.fadeIn( 'fast' );
+				handler.addClass( "ui-handler-visible" );
+				handlerThumb.stop( true, true )
+							.fadeIn();
 			}).bind( "scrollupdate", function ( event, data ) {
 				if ( !scrollview.enableHandler() || viewLength < 0 || clipLength < handlerHeight ) {
 					return;
@@ -254,15 +254,16 @@
 				}
 				scrollview._handlerTimer = setTimeout( function () {
 					if ( scrollview._timerID === 0 && $view.moveData === null ) {
-						handlerThumb.removeClass( "ui-handler-visible" )
-									.stop( true, true )
+						handlerThumb.stop( true, true )
 									.css( "opacity", 1.0 )
-									.fadeOut( 'fast' );
+									.fadeOut( function () {
+										handler.removeClass( "ui-handler-visible" );
+									});
 						scrollview._handlerTimer = 0;
 					}
 				}, 1000 );
 			}).bind( "mousewheel", function ( event ) {
-				handlerThumb.removeClass( "ui-handler-visible" ).hide();
+				handler.removeClass( "ui-handler-visible" );
 				setHanderPostion( scrollview.getScrollPosition() );
 			});
 		};
@@ -284,7 +285,7 @@
 				$view.find( ".ui-scrollbar" ).hide();
 				$view.find( ".ui-handler" ).show();
 			} else {
-				$view.find( ".ui-handler" ).hide();
+				$view.find( ".ui-handler" ).removeClass( "ui-handler-visible" ).hide();
 				$view.find( ".ui-scrollbar" ).show();
 			}
 		},
-- 
2.7.4