Bump react-remove-scroll in reach/dialog to v2.5.5

[iamhosseindhv]

🐛 Bug report

This isn't directly related to reach-ui, rather a dependency of it.

Current Behavior

There are various bug fixed in the latest release of react-remove-scroll. Namely allowing range inputs to be used inside a dialog (link). At the moment, we have to dangerouslyBypassScrollLock and install the latest version of react-remove-scroll to benefit from the latest bug fixes.

Expected behavior

Reach dialog to use the latest version of react-remove-scroll which contains various bug fixes.

Your environment

Software	Name(s)	Version
Reach Package	dialog	0.16.2
React		17.0.2
Browser		Chrome
Assistive tech		n/a
Node		n/a
npm/yarn		8.1.2
Operating System		MacOS

[nemdo]

In addition, is it possible to add the property removeScrollBar? Just for this reason, I have to dangerouslyBypassScrollLock and install react-remove-scroll myself.

Let me know, thank you.

[camdecoster]

I'd like to make this request as well. There's a bug in the current version of react-remove-scroll where the scrollbar width is added twice when the scrollbar is no longer displayed (such as when a Dialog is open).

[nemdo]

I'd like to make this request as well. There's a bug in the current version of react-remove-scroll where the scrollbar width is added twice when the scrollbar is no longer displayed (such as when a Dialog is open).

Again, this problem is only present when removeScrollBar={true}. For those who don't mind leaving the scrollBar (only visually, scrolling is still locked), it would be nice to add the property to @reach/dialog!