The top bar of the modal should remain on the page and the content should scroll within the modal. By having the modal scroll off the page, the user no longer has access to the close button. I know you can click off the modal to close it, but that isn't always intuitive.