Update file rss-feed-preview.js

assets/js/rss-feed-preview.js

@@ -66,17 +66,17 @@
     return result;
   }
 
-  function checkFeed(url, callback) {
+  function checkFeed(url, onSuccess, onError) {
-    $.ajax({
+    return $.ajax({
       url: CORS_PROXY + url,
       type: 'GET',
       dataType: 'text',
       success: function (data) {
         var items = parseRSS(data);
-        callback(items);
+        onSuccess(items);
       },
       error: function () {
-        callback(null);
+        onError();
       }
     });
   }
@@ -130,17 +130,20 @@
     var cache = {};
     var currentLink = null;
     var timeout = null;
+    var currentRequest = null;
+    var currentRequestId = 0;
+    $('main table tbody').on('mouseenter mousemove mouseleave', 'tr td a', function (e) {
 
-    $('main table tbody tr td a').each(function () {
+      if (e.type === 'mouseenter') {
         var $link = $(this);
-
-      $link.on('mouseenter', function (e) {
-        currentLink = this;
         var siteName = $link.text();
         var url = $link.attr('data-feed');
         if (!url)
           return;
 
+        currentLink = $link[0];
+        var requestId = ++currentRequestId;
+
         $previewEl.html('<p>Checking for RSS/Atom feed...</p>').show();
         positionPreview(e);
 
@@ -148,38 +151,51 @@
           clearTimeout(timeout);
         timeout = setTimeout(function () {
           if (cache[url]) {
+            if (currentLink === $link[0] && requestId === currentRequestId) {
               renderFeedItems(cache[url], siteName);
               positionPreview(e);
+            }
             return;
           }
 
-          if (url) {
+          currentRequest = checkFeed(
-            checkFeed(url, function (items) {
+            url,
-              if (currentLink === $link[0] && items) {
+            function (items) {
+              if (requestId !== currentRequestId || currentLink !== $link[0])
+                return;
+
+              if (items && items.length) {
                 cache[url] = items;
                 renderFeedItems(items, siteName);
-                positionPreview(e);
               } else {
-                $previewEl.hide();
+                $previewEl.html('<p>No feed items found.</p>');
               }
-            });
+
-          } else {
+              positionPreview(e);
-            $previewEl.hide();
+            },
+            function () {
+              if (requestId !== currentRequestId || currentLink !== $link[0])
+                return;
+              $previewEl.html('<p>Failed to load feed.</p>');
+              positionPreview(e);
             }
+          );
         }, 300);
-      });
+      } else if (e.type === 'mousemove') {
-
-      $link.on('mousemove', function (e) {
         if ($previewEl.is(':visible'))
           positionPreview(e);
-      });
+      } else if (e.type === 'mouseleave') {
-
-      $link.on('mouseleave', function () {
         clearTimeout(timeout);
         timeout = null;
         currentLink = null;
+
+        if (currentRequest) {
+          currentRequest.abort();
+          currentRequest = null;
+        }
+
         $previewEl.hide();
-      });
+      }
     });
 
     $(document).on('click', function (e) {
@@ -189,6 +205,7 @@
     });
   }
 
+
   if (document.readyState === 'complete' || document.readyState === 'interactive') {
     init();
   } else {