handler: bug fix (handler activation)

Handler=true work when scrollview area longer than the length of the handler.
Resolves #N_SE-22336

Change-Id: I2467ceb038a871b3744b2eb3a1fab0ab236e7f41

diff --git a/demos/tizen-winsets/widgets/handler.html b/demos/tizen-winsets/widgets/handler.html
index 085e87c..eb04708 100755 (executable)
--- a/demos/tizen-winsets/widgets/handler.html
+++ b/demos/tizen-winsets/widgets/handler.html
@@ -41,78 +41,6 @@
                        <li><a href="#">Rod Tarker</a></li>
                        <li data-role="list-divider">S</li>
                        <li><a href="#">Sawyer Wakefield</a></li>
-                       <li data-role="list-divider">A</li>
-                       <li><a href="#">Adam Kinkaid</a></li>
-                       <li><a href="#">Alex Wickerham</a></li>
-                       <li><a href="#">Avery Johnson</a></li>
-                       <li data-role="list-divider">B</li>
-                       <li><a href="#">Bob Cabot</a></li>
-                       <li data-role="list-divider">C</li>
-                       <li><a href="#">Caleb Booth</a></li>
-                       <li><a href="#">Christopher Adams</a></li>
-                       <li><a href="#">Culver James</a></li>
-                       <li data-role="list-divider">D</li>
-                       <li><a href="#">David Walsh</a></li>
-                       <li><a href="#">Drake Alfred</a></li>
-                       <li data-role="list-divider">E</li>
-                       <li><a href="#">Elizabeth Bacon</a></li>
-                       <li><a href="#">Emery Parker</a></li>
-                       <li><a href="#">Enid Voldon</a></li>
-                       <li data-role="list-divider">F</li>
-                       <li><a href="#">Francis Wall</a></li>
-                       <li data-role="list-divider">G</li>
-                       <li><a href="#">Graham Smith</a></li>
-                       <li><a href="#">Greta Peete</a></li>
-                       <li data-role="list-divider">H</li>
-                       <li><a href="#">Harvey Walls</a></li>
-                       <li data-role="list-divider">M</li>
-                       <li><a href="#">Mike Farnsworth</a></li>
-                       <li><a href="#">Murray Vanderbuilt</a></li>
-                       <li data-role="list-divider">N</li>
-                       <li><a href="#">Nathan Williams</a></li>
-                       <li data-role="list-divider">P</li>
-                       <li><a href="#">Paul Baker</a></li>
-                       <li><a href="#">Pete Mason</a></li>
-                       <li data-role="list-divider">R</li>
-                       <li><a href="#">Rod Tarker</a></li>
-                       <li data-role="list-divider">S</li>
-                       <li><a href="#">Sawyer Wakefield</a></li>
-                       <li data-role="list-divider">A</li>
-                       <li><a href="#">Adam Kinkaid</a></li>
-                       <li><a href="#">Alex Wickerham</a></li>
-                       <li><a href="#">Avery Johnson</a></li>
-                       <li data-role="list-divider">B</li>
-                       <li><a href="#">Bob Cabot</a></li>
-                       <li data-role="list-divider">C</li>
-                       <li><a href="#">Caleb Booth</a></li>
-                       <li><a href="#">Christopher Adams</a></li>
-                       <li><a href="#">Culver James</a></li>
-                       <li data-role="list-divider">D</li>
-                       <li><a href="#">David Walsh</a></li>
-                       <li><a href="#">Drake Alfred</a></li>
-                       <li data-role="list-divider">E</li>
-                       <li><a href="#">Elizabeth Bacon</a></li>
-                       <li><a href="#">Emery Parker</a></li>
-                       <li><a href="#">Enid Voldon</a></li>
-                       <li data-role="list-divider">F</li>
-                       <li><a href="#">Francis Wall</a></li>
-                       <li data-role="list-divider">G</li>
-                       <li><a href="#">Graham Smith</a></li>
-                       <li><a href="#">Greta Peete</a></li>
-                       <li data-role="list-divider">H</li>
-                       <li><a href="#">Harvey Walls</a></li>
-                       <li data-role="list-divider">M</li>
-                       <li><a href="#">Mike Farnsworth</a></li>
-                       <li><a href="#">Murray Vanderbuilt</a></li>
-                       <li data-role="list-divider">N</li>
-                       <li><a href="#">Nathan Williams</a></li>
-                       <li data-role="list-divider">P</li>
-                       <li><a href="#">Paul Baker</a></li>
-                       <li><a href="#">Pete Mason</a></li>
-                       <li data-role="list-divider">R</li>
-                       <li><a href="#">Rod Tarker</a></li>
-                       <li data-role="list-divider">S</li>
-                       <li><a href="#">Sawyer Wakefield</a></li>
                </ul>
        </div><!-- /content -->
        <div data-role="footer">

diff --git a/src/widgets/handler/js/jquery.tizen.scrollview.handler.js b/src/widgets/handler/js/jquery.tizen.scrollview.handler.js
index d7bc2d1..c6b33da 100755 (executable)
--- a/src/widgets/handler/js/jquery.tizen.scrollview.handler.js
+++ b/src/widgets/handler/js/jquery.tizen.scrollview.handler.js
@@ -214,7 +214,7 @@
                                        moveTimer = undefined;
                                }
 
-                               if ( viewLength < 0 || trackLength < handlerHeight ) {
+                               if ( viewLength < 0 || clipLength < handlerHeight ) {
                                        return;
                                }
 
@@ -222,7 +222,7 @@
                                                        .stop( true, true )
                                                        .fadeIn( 'fast' );
                        }).bind( "scrollupdate", function ( event, data ) {
-                               if ( !scrollview.enableHandler() || viewLength < 0 || trackLength < handlerHeight ) {
+                               if ( !scrollview.enableHandler() || viewLength < 0 || clipLength < handlerHeight ) {
                                        return;
                                }