- \#5: Fixed an issue that caused the plugin to possibly send an unne…

.eslintrc

@@ -17,6 +17,7 @@
     "doneLoadingEvent": true,
     "loadTriggerEvents": true,
     "requestErrorCode": true,
-    "parseResultErrorCode": true
+    "parseResultErrorCode": true,
+    "ignoreEvent": true
   }
 }

CHANGELOG.md

@@ -4,6 +4,11 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [2.0.3] - 2018-03-27
+### Fixed
+- \#5: Fixed an issue that caused the plugin to possibly send an unnecessary request if the `$.mobile.hideUrlBar` JQM setting was set to `true`.
+- Fixed incorrect url to the plugin within the `sample.html`.
+
 ## [2.0.2] - 2018-03-09
 ### Fixed
 - \#3: Corrected an issue that caused the plugin not to load items on initialization if the `filter` option was set to `true` 

dist/jquery.mobile.lazyloader.js

@@ -58,7 +58,14 @@ var $window = $( window ),
    * The error code that is provided in the event when parsing of a response fails
    * @type {number}
    */
-  parseResultErrorCode = 2;
+  parseResultErrorCode = 2,
+
+  /**
+   * Indicates the event should be ignored. The value is determined by $.mobile.hideUrlBar and when
+   * it is true a programmatic scroll is done.
+   * @type {boolean}
+   */
+  ignoreEvent = $.mobile.hideUrlBar;
 
 // Source: src/filterable.js
 var filterableWidget = $.mobile.filterable,
@@ -103,6 +110,15 @@ if ( filterableWidget ) {
 }
 
 // Source: src/lazyloader.js
+// Check if the first scroll event should be ignored
+if ( ignoreEvent ) {
+
+  // It should be ignored so listen to the first scroll event to toggle the boolean
+  $window.one( "scrollstart", function() {
+    ignoreEvent = false;
+  } );
+}
+
 $.widget( "mobile." + widgetName, $.mobile.listview, {
   options: {
 
@@ -256,7 +272,7 @@ $.widget( "mobile." + widgetName, $.mobile.listview, {
 
         // Check if the page scroll location is close to the bottom
         if ( self.element.height() - options.threshold <=
-             $window.scrollTop() + $window.height() ) {
+          $window.scrollTop() + $window.height() ) {
 
           // Get the progress element
           $( options.$progress ).show();
@@ -405,8 +421,9 @@ $.widget( "mobile." + widgetName, $.mobile.listview, {
   _handleEvent: function() {
     var self = this;
 
-    // Check if the listview is visible and an event has not triggered a load already
-    if ( !self._eventTriggered && self.element.is( ":visible" ) ) {
+    // Check if the listview is visible and an event has not triggered a load already or if it
+    // should be ignored
+    if ( !self._eventTriggered && !ignoreEvent && self.element.is( ":visible" ) ) {
 
       // Block other events from triggering a load
       self._eventTriggered = true;

samples/sample.html

@@ -7,7 +7,7 @@
     <script src="http://code.jquery.com/jquery-2.2.4.min.js"></script>
     <script src="http://code.jquery.com/mobile/1.4.5/jquery.mobile-1.4.5.min.js"></script>
     <script src="https://cdnjs.cloudflare.com/ajax/libs/ICanHaz.js/0.10.3/ICanHaz.min.js"></script>
-    <script src="../dist/jquery.mobile.lazyloader-2.0.0.js"></script>
+    <script src="../dist/jquery.mobile.lazyloader.js"></script>
     <script src="sample.js"></script>
     <meta name="viewport" content="width=device-width, initial-scale=1">
     <script id="user" type="text/html">

src/_vars.js

@@ -53,4 +53,11 @@ var $window = $( window ),
    * The error code that is provided in the event when parsing of a response fails
    * @type {number}
    */
-  parseResultErrorCode = 2;
+  parseResultErrorCode = 2,
+
+  /**
+   * Indicates the event should be ignored. The value is determined by $.mobile.hideUrlBar and when
+   * it is true a programmatic scroll is done.
+   * @type {boolean}
+   */
+  ignoreEvent = $.mobile.hideUrlBar;

src/lazyloader.js

@@ -1,5 +1,14 @@
 "use strict";
 
+// Check if the first scroll event should be ignored
+if ( ignoreEvent ) {
+
+  // It should be ignored so listen to the first scroll event to toggle the boolean
+  $window.one( "scrollstart", function() {
+    ignoreEvent = false;
+  } );
+}
+
 $.widget( "mobile." + widgetName, $.mobile.listview, {
   options: {
 
@@ -153,7 +162,7 @@ $.widget( "mobile." + widgetName, $.mobile.listview, {
 
         // Check if the page scroll location is close to the bottom
         if ( self.element.height() - options.threshold <=
-             $window.scrollTop() + $window.height() ) {
+          $window.scrollTop() + $window.height() ) {
 
           // Get the progress element
           $( options.$progress ).show();
@@ -302,8 +311,9 @@ $.widget( "mobile." + widgetName, $.mobile.listview, {
   _handleEvent: function() {
     var self = this;
 
-    // Check if the listview is visible and an event has not triggered a load already
-    if ( !self._eventTriggered && self.element.is( ":visible" ) ) {
+    // Check if the listview is visible and an event has not triggered a load already or if it
+    // should be ignored
+    if ( !self._eventTriggered && !ignoreEvent && self.element.is( ":visible" ) ) {
 
       // Block other events from triggering a load
       self._eventTriggered = true;