Added isBodyScrollable prop

[sushant-j]

This prop adds a no-scroll class on body when set to FALSE.

P.S. Used a quick hacky way, manipulating DOM directly from render function.
I realize this is probably not the ideal way to do this.
Would like if you could suggest an alternate, ideal way.

[egoist]

Should update ./themes/default.css as well

[egoist]

hmm I guess this won't work since .no-scroll on body won't be removed when this component is destroyed? we need to turn this component into non-functional component and use lifecycle hooks.

[sushant-j]

I ran it locally and it works.
When I close the modal (click-outside), the .no-scroll class is removed from the body.

Is there some other case I am missing here?

Also, the tests on circleci are failing, due to document being undefined. How can we pass this test?

[egoist]

When I close the modal (click-outside), the .no-scroll class is removed from the body. Is there some other case I am missing here?

when the component is destroyed.

you closed the modal but the component is still alive.

[sushant-j]

when the component is destroyed.

In what scenario would it be destroyed rather than just changed?