[BD-46] Add fixed increment scroll behavior to useOverflowScroll

[UvgenGen]

Description

Added fixed increment scroll behavior to useOverflowScroll

• Add fixed increment scroll behavior to useOverflowScroll #1950

Merge Checklist

• If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
• Does your change adhere to the documented style conventions?
• Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
• Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
• Were your changes tested in the example app?
• Is there adequate test coverage for your changes?
• Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

• Verify your changes were released to NPM at the expected version.
• If you'd like, share your contribution in #show-and-tell.
• 🎉 🙌 Celebrate! Thanks for your contribution.

[openedx-webhooks]

Thanks for the pull request, @UvgenGen!

When this pull request is ready, tag your edX technical lead.

[netlify]

✅ Deploy Preview for paragon-openedx ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	6f587a0
🔍 Latest deploy log	https://app.netlify.com/sites/paragon-openedx/deploys/6479b871c231d700085f05d4
😎 Deploy Preview	https://deploy-preview-2151--paragon-openedx.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[adamstankiewicz]

Overall, this is looking good. A few other comments:

• Linting errors (npm run lint) will need to be resolved.
• Include tests for new functionality.
• Document and include example(s) of new functionality in documentation website.

[adamstankiewicz]

[suggestion] Would it be helpful to potentially treat the offset value as a distinct argument than the offset type (pixel or percent). For example, something like:

useOverflowScrollActions({
  offset: 25, // number or string
  offsetType: 'fixed', // "percentage" or "fixed" (default)
});

This way, this logic wouldn't need to have conditionals based on px or % in a string itself.

Curious to hear your thoughts :)

[codecov]

Codecov Report

Patch coverage: 90.62% and project coverage change: -0.02 ⚠️

Comparison is base (088eab4) 91.38% compared to head (6f587a0) 91.36%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2151      +/-   ##
==========================================
- Coverage   91.38%   91.36%   -0.02%     
==========================================
  Files         234      234              
  Lines        4120     4148      +28     
  Branches      982      997      +15     
==========================================
+ Hits         3765     3790      +25     
- Misses        348      351       +3     
  Partials        7        7              

Impacted Files	Coverage Δ
src/OverflowScroll/OverflowScroll.jsx	100.00% <ø> (ø)
...rc/OverflowScroll/data/useOverflowScrollActions.js	91.66% <90.00%> (-2.46%)	⬇️
src/OverflowScroll/data/useOverflowScroll.js	63.82% <100.00%> (+1.60%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[openedx-webhooks]

@UvgenGen 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

[edx-semantic-release]

🎉 This PR is included in version 20.44.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀