fix: do not use useLayoutEffect if not on client

[nelsliu9121]

Fixes #6

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/react-component/overflow/6DAjFbqGmRNgHL6t2KSms3pisp5j
✅ Preview: https://overflow-git-fork-nelsliu9121-master-react-component.vercel.app

[codecov]

Codecov Report

Merging #16 (cc27e69) into master (1bccbe8) will decrease coverage by 0.49%.
The diff coverage is 83.33%.

@@             Coverage Diff             @@
##            master      #16      +/-   ##
===========================================
- Coverage   100.00%   99.50%   -0.50%     
===========================================
  Files            4        5       +1     
  Lines          200      204       +4     
  Branches        62       64       +2     
===========================================
+ Hits           200      203       +3     
- Misses           0        1       +1     

Impacted Files	Coverage Δ
src/hooks/useIsomorphicLayoutEffect.ts	75.00% <75.00%> (ø)
src/Overflow.tsx	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1bccbe8...cc27e69. Read the comment docs.

[afc163]

Please add test case.

[nelsliu9121]

I am not certain if this is test-able while the JSDom is enabled in Jest. 😞

The same thing happened in react-component/select where the line is ignored by test coverage. See this line.

I could be wrong with this, but I do not have enough experience in testing useLayoutEffect, and I would love to have someone to come in and fix it. 😃

[ahallora]

I would ignore it like we do on select and get this PR merged. 🙏 @afc163 WDYT? 😄

[sb-lee]

Are you reviewing this PR? @zombieJ 🙏 🙏 🙏 🙏 🙏 🙏
I want to know why not merge.

cc. @afc163

[nelsliu9121]

Should I add the comment which will tell the test coverage to ignore the line, just like how react-component/select did?

[ahallora]

@afc163 / @zombieJ - I would happily help you with this, but I simply can't fathom it. Do you have 2 minutes to take a look? 🙏