Use toString instead of toFixed

juliencrn · Feb 8, 2025 · 7b74d4c · 7b74d4c
1 parent 75f571b
commit 7b74d4c
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/packages/usehooks-ts/src/useIntersectionObserver/useIntersectionObserver.demo.tsx b/packages/usehooks-ts/src/useIntersectionObserver/useIntersectionObserver.demo.tsx
@@ -28,7 +28,7 @@ export default function Component() {
   return (
     <>
       {Array.from({ length: 5 }).map((_, index) => (
-        <Section key={index + 1} title={(index + 1).toFixed()} />
+        <Section key={index + 1} title={(index + 1).toString()} />
       ))}
     </>
   )

diff --git a/packages/usehooks-ts/src/useScrollLock/useScrollLock.test.ts b/packages/usehooks-ts/src/useScrollLock/useScrollLock.test.ts
@@ -109,7 +109,7 @@ describe('useScrollLock()', () => {
     const scrollbarWidth = window.innerWidth - document.body.scrollWidth
 
     expect(document.body.style.paddingRight).toBe(
-      `${scrollbarWidth.toFixed()}px`,
+      `${scrollbarWidth.toString()}px`,
     )
     unmount()
     expect(document.body.style.paddingRight).toBe('')

diff --git a/packages/usehooks-ts/src/useScrollLock/useScrollLock.ts b/packages/usehooks-ts/src/useScrollLock/useScrollLock.ts
@@ -90,7 +90,7 @@ export function useScrollLock(
           0
 
         const scrollbarWidth = offsetWidth - target.current.scrollWidth
-        target.current.style.paddingRight = `${(scrollbarWidth + currentPaddingRight).toFixed()}px`
+        target.current.style.paddingRight = `${(scrollbarWidth + currentPaddingRight).toString()}px`
       }
 
       // Lock the scroll