From 467ace2f14b511cf4a62f08436831c4bc713b387 Mon Sep 17 00:00:00 2001
From: Jackson Harper <jacksonh@gmail.com>
Date: Mon, 13 Mar 2023 13:29:23 +0800
Subject: [PATCH] Rename reading progress file now that its no longer a react
 hook

---
 .../components/templates/article/Article.tsx  |  4 +-
 .../lib/hooks/useReadingProgressAnchor.tsx    | 79 -------------------
 2 files changed, 2 insertions(+), 81 deletions(-)
 delete mode 100644 packages/web/lib/hooks/useReadingProgressAnchor.tsx

diff --git a/packages/web/components/templates/article/Article.tsx b/packages/web/components/templates/article/Article.tsx
index 2726f1fd5..8204bc7d1 100644
--- a/packages/web/components/templates/article/Article.tsx
+++ b/packages/web/components/templates/article/Article.tsx
@@ -2,12 +2,12 @@ import { Box } from '../../elements/LayoutPrimitives'
 import {
   getTopOmnivoreAnchorElement,
   parseDomTree,
-} from '../../../lib/hooks/useReadingProgressAnchor'
+} from '../../../lib/anchorElements'
 import {
   ScrollOffsetChangeset,
   useScrollWatcher,
 } from '../../../lib/hooks/useScrollWatcher'
-import { MutableRefObject, useEffect, useMemo, useRef, useState } from 'react'
+import { MutableRefObject, useEffect, useRef, useState } from 'react'
 import { Tweet } from 'react-twitter-widgets'
 import { render } from 'react-dom'
 import { isDarkTheme } from '../../../lib/themeUpdater'
diff --git a/packages/web/lib/hooks/useReadingProgressAnchor.tsx b/packages/web/lib/hooks/useReadingProgressAnchor.tsx
deleted file mode 100644
index 245c8b2b9..000000000
--- a/packages/web/lib/hooks/useReadingProgressAnchor.tsx
+++ /dev/null
@@ -1,79 +0,0 @@
-import { useEffect } from 'react'
-
-const ANCHOR_ELEMENTS_BLOCKED_ATTRIBUTES = [
-  'omnivore-highlight-id',
-  'data-twitter-tweet-id',
-  'data-instagram-id',
-]
-
-// We search in reverse so we can find the last element
-// that is visible on the page
-export const getTopOmnivoreAnchorElement = (
-  articleContentElement: HTMLElement
-): string | undefined => {
-  let topVisibleRect: Element | undefined = undefined
-  const anchors = Array.from(
-    document.querySelectorAll(`[data-omnivore-anchor-idx]`)
-  ).reverse()
-
-  for (const anchor of anchors) {
-    const rect = anchor.getBoundingClientRect()
-    if (rect.top >= 0 && rect.bottom <= articleContentElement.clientHeight) {
-      if (
-        topVisibleRect &&
-        topVisibleRect.getBoundingClientRect().top < rect.top
-      ) {
-        continue
-      }
-      topVisibleRect = anchor
-    }
-  }
-
-  return topVisibleRect?.getAttribute(`data-omnivore-anchor-idx`) ?? undefined
-}
-
-export const useReadingProgressAnchor = (
-  articleContentRef: React.MutableRefObject<HTMLDivElement | null>
-): void => {
-  useEffect(() => {
-    parseDomTree(articleContentRef.current)
-  }, [articleContentRef])
-}
-
-export function parseDomTree(
-  pageNode: HTMLDivElement | null
-): HTMLDivElement[] {
-  if (!pageNode || pageNode.childNodes.length == 0) {
-    return []
-  }
-
-  const nodesToVisitStack: [HTMLDivElement] = [pageNode]
-  const visitedNodeList = []
-
-  while (nodesToVisitStack.length > 0) {
-    const currentNode = nodesToVisitStack.pop()
-    if (
-      currentNode?.nodeType !== Node.ELEMENT_NODE ||
-      // Avoiding dynamic elements from being counted as anchor-allowed elements
-      ANCHOR_ELEMENTS_BLOCKED_ATTRIBUTES.some((attrib) =>
-        currentNode.hasAttribute(attrib)
-      )
-    ) {
-      continue
-    }
-    visitedNodeList.push(currentNode)
-    ;[].slice
-      .call(currentNode.childNodes)
-      .reverse()
-      .forEach(function (node) {
-        nodesToVisitStack.push(node)
-      })
-  }
-
-  visitedNodeList.shift()
-  visitedNodeList.forEach((node, index) => {
-    // start from index 1, index 0 reserved for anchor unknown.
-    node.setAttribute('data-omnivore-anchor-idx', (index + 1).toString())
-  })
-  return visitedNodeList
-}