From af0480ffcf23faa14ff1b5267cb42bd1b8c4d02b Mon Sep 17 00:00:00 2001
From: matheusps <matheus.ps@icloud.com>
Date: Mon, 15 Apr 2019 13:42:54 -0300
Subject: [PATCH] Separate hook for getVisible slides per page

---
 react/components/Dots.js        | 36 +++++++++++++--------------------
 react/hooks/useSlidesPerPage.js | 25 +++++++++++++++++++++++
 2 files changed, 39 insertions(+), 22 deletions(-)
 create mode 100644 react/hooks/useSlidesPerPage.js

diff --git a/react/components/Dots.js b/react/components/Dots.js
index 72a0eb6..f044cdd 100644
--- a/react/components/Dots.js
+++ b/react/components/Dots.js
@@ -1,10 +1,9 @@
-import React, { useEffect, useState, useMemo } from 'react'
+import React, { useMemo } from 'react'
 import PropTypes from 'prop-types'
 import classnames from 'classnames'
-import debounce from 'debounce'
 
+import useSlidesPerPage from '../hooks/useSlidesPerPage'
 import { constants } from '../utils'
-import resolveSlidesNumber from '../utils/resolveSlidesNumber'
 import styles from './styles.css'
 
 const Dots = ({
@@ -23,41 +22,34 @@ const Dots = ({
   treePath,
   ...otherProps
 }) => {
-  const [state, setState] = useState(resolveSlidesNumber(perPage))
-
-  useEffect(() => {
-    const handleResize = debounce(
-      () => setState(resolveSlidesNumber(perPage)),
-      resizeDebounce
-    )
-    window.addEventListener('resize', handleResize)
-    return () => {
-      window.removeEventListener('resize', handleResize)
-    }
-  }, [])
+  const visibleSlides = useSlidesPerPage(resizeDebounce, perPage)
 
   const slideIndeces = useMemo(
     () =>
-      state
+      visibleSlides
         ? [
             ...Array(
-              showDotsPerPage ? Math.ceil(totalSlides / state) : totalSlides
+              showDotsPerPage
+                ? Math.ceil(totalSlides / visibleSlides)
+                : totalSlides
             ).keys(),
           ]
         : [],
-    [state]
+    [visibleSlides]
   )
 
   const selectedDot = useMemo(() => {
-    const realCurrentSlide = loop ? currentSlide - state : currentSlide
+    const realCurrentSlide = loop ? currentSlide - visibleSlides : currentSlide
     return showDotsPerPage
-      ? Math.floor(realCurrentSlide / state)
+      ? Math.floor(realCurrentSlide / visibleSlides)
       : realCurrentSlide
   }, [currentSlide])
 
   const handleDotClick = index => {
-    const slideToGo = showDotsPerPage ? index * state : index
-    onChangeSlide(loop ? slideToGo + state : slideToGo)
+    console.log('OnClick', visibleSlides)
+    const slideToGo = showDotsPerPage ? index * visibleSlides : index
+    onChangeSlide(loop ? slideToGo + visibleSlides : slideToGo)
+    console.log('At end', visibleSlides)
   }
 
   const classes = {
diff --git a/react/hooks/useSlidesPerPage.js b/react/hooks/useSlidesPerPage.js
new file mode 100644
index 0000000..0026980
--- /dev/null
+++ b/react/hooks/useSlidesPerPage.js
@@ -0,0 +1,25 @@
+import { useEffect, useState } from 'react'
+import debounce from 'debounce'
+
+import resolveSlidesNumber from '../utils/resolveSlidesNumber'
+
+const useSlidesPerPage = (resizeDebounce, perPage) => {
+  const [visibleSlides, setVisibleSlides] = useState(
+    resolveSlidesNumber(perPage)
+  )
+
+  useEffect(() => {
+    const handleResize = debounce(
+      () => setVisibleSlides(resolveSlidesNumber(perPage)),
+      resizeDebounce
+    )
+    window.addEventListener('resize', handleResize)
+    return () => {
+      window.removeEventListener('resize', handleResize)
+    }
+  }, [])
+
+  return visibleSlides
+}
+
+export default useSlidesPerPage