fix(pat inject): autoload-visible only when in viewport.

Fix autload-visible trigger to only load the injection if the element is in the viewport.
As before, this check is done after 100ms.
This prevents loading items when we quickly scoll across them, like with href-section-jumps.

Author: thet

src/pat/inject/inject.js

@@ -992,24 +992,22 @@ const inject = {
                 if (!$el.is(":visible")) {
                     return false;
                 }
+                if (!utils.elementInViewport($el[0])) {
+                    return false;
+                }
                 // check if the target element still exists. Otherwise halt and catch fire
                 const target = (
                     $el.data("pat-inject")[0].target || cfgs[0].defaultSelector
                 ).replace(/::element/, "");
                 if (target && target !== "self" && $(target).length === 0) {
                     return false;
                 }
-                const reltop =
-                        $el.safeOffset().top - $scrollable.safeOffset().top - 1000,
-                    doTrigger = reltop <= $scrollable.innerHeight();
-                if (doTrigger) {
-                    // checkVisibility was possibly installed as a scroll
-                    // handler and has now served its purpose -> remove
-                    $($scrollable[0]).off("scroll", checkVisibility);
-                    $(window).off("resize.pat-autoload", checkVisibility);
-                    return trigger();
-                }
-                return false;
+
+                // checkVisibility was possibly installed as a scroll
+                // handler and has now served its purpose -> remove
+                $($scrollable[0]).off("scroll", checkVisibility);
+                $(window).off("resize.pat-autoload", checkVisibility);
+                return trigger();
             }, 100);
             if (checkVisibility()) {
                 return true;