Remove usage of isIntersecting, doesn't handle threshold

Author: Daniel Schmidt

src/intersection.js

@@ -63,20 +63,15 @@ export function unobserve(element) {
 
 function onChange(changes) {
   changes.forEach(intersection => {
+    console.log(intersection)
     if (INSTANCE_MAP.has(intersection.target)) {
       const { callback, visible, threshold } = INSTANCE_MAP.get(
         intersection.target,
       )
-      let inView
-      if (typeof intersection.isIntersecting === 'boolean') {
-        // Use the new "isIntersecting" property if available.
-        inView = intersection.isIntersecting
-      } else {
-        // Trigger on 0 ratio only when not visible.
-        inView = visible
-          ? intersection.intersectionRatio > threshold
-          : intersection.intersectionRatio >= threshold
-      }
+      // Trigger on 0 ratio only when not visible.
+      const inView = visible
+        ? intersection.intersectionRatio > threshold
+        : intersection.intersectionRatio >= threshold
 
       INSTANCE_MAP.set(intersection.target, {
         callback,

stories/Observer.story.js

@@ -38,6 +38,15 @@ storiesOf('Intersection Observer', module)
       </Observer>
     </ScrollWrapper>
   ))
+  .add('With threshold 50%', () => (
+    <ScrollWrapper>
+      <Observer threshold={0.5} onChange={action('Child Observer inview')}>
+        {inView => (
+          <Header>{`Header is fully inside the viewport: ${inView}`}</Header>
+        )}
+      </Observer>
+    </ScrollWrapper>
+  ))
   .add('Trigger once', () => (
     <ScrollWrapper>
       <Observer