Remove defaultProps (use default parameters)

[koteus]

This fixes React warnings: "Support for defaultProps will be removed from function components in a future major release. Use JavaScript default parameters instead."

Changelog

Please copy and paste new entries from CHANGELOG.md here.

Specific changes

Please list each individual specific change in this pull request.

[koteus]

This is a rebased version of my previous PR #134 I had to open a new one because I opened it from the main branch on my fork (my bad), and because another PR was merged (#137) I had trouble syncing my changes. Sorry for this compulim, and I hope this one will be moved forward and merged soon 🙏

[shgotlib]

Is there any progress about merging this one? I really need it

[rafalzawadzki]

Is there any progress about merging this one? I really need it

If you really need it, you can use the branch as dependency, eg.

"react-scroll-to-bottom": "github:koteus/react-scroll-to-bottom#fix/remove-default-props",

in your package.json

[shgotlib]

I tried to install the branch but then got the following error:

[vite] Internal server error: Failed to resolve entry for package "react-scroll-to-bottom". The package may have incorrect main/module/exports specified in its package.json

Do you know why?

[rafalzawadzki]

I tried to install the branch but then got the following error:

[vite] Internal server error: Failed to resolve entry for package "react-scroll-to-bottom". The package may have incorrect main/module/exports specified in its package.json

Do you know why?

hmm no, unfortunately I don't know why :/ gave a generic advice that works in most cases but seems the fork (or the library itself) may not work well with this workaround

[codeXLinkX]

@compulim is there anything we can help to get this reviewed and merged?

[josheverett]

If it's helpful to anyone out there, for a project I work on I finally got annoyed enough to just use patch-package to fix this.

Patch file: react-scroll-to-bottom+4.2.0.patch

Apologies for the size of it -- when my editor auto-formatted the files I was like ehhh whatevs.

[donghoon-song]

I need this :)

[zuhaib10]

i also need this : )

[lx-0]

I made a smaller version of @josheverett patch.

react-scroll-to-bottom+4.2.0.patch

1. Install patch-package: npm i patch-package --save-dev
2. copy react-scroll-to-bottom+4.2.0.patch to new root folder patches
3. add "postinstall": "patch-package" to scripts in package.json
4. run npm ci

[jdmnk]

Bumping this thread, because it would be really useful to merge this :) There is unfortunately no "nice" workaround for getting rid of the error, although there are working solutions in this thread.

[2-fly-4-ai]

Yes. The Merge. Send it.

[benrugg]

Bumping this PR again. I fully get how taxing it is to have a fulltime engineering job and then come home to a pull request. Given the community support on this one, I'd say it's ready to merge and publish! 👍

[willemmulder]

React 19 is out, and this is now very much needed. @compulim Not sure about your time, but would you be okay to merge it?