Fixes scroll for modals in the Admin UI

[saul-mtz]

Description of changes

Fixes the scroll problem with modals in the Admin UI, when the height of the modal form is higher than the one of the screen.

Related issues (if any)

#3712, #3822 and #4025

fixes #3822
fixes #4025

Testing

Create a new item for a List with many fields, if the modal form is higher than the visible screen then that modal must be scrollable and the "Create" button visible.

• Please confirm npm run test-all ran successfully.

[VinayaSathyanarayana]

Please merge this at the earliest

[morenoh149]

@keystonejs/leads please merge as minor.

[JedWatson]

Thanks for the fix - it will be included in the next release. If anyone would mind updated related issues, that would be greatly appreciated.

[1337cookie]

Can I get this via npm? I have a continuous deployment running with 4.0.0-beta.8 so any changes via my project's git get overwritten.

[kamontat]

Include this to next release please.

[stennie]

@1337cookie @kamontat This change has been committed to master and will be included in the next 4.0.0 release. It is not included in 4.0.0-beta.8, which is currently the latest beta published on npm.

See the 4.0.0 Milestone for issues currently being triaged for release.

Please us on the Keystone Slack in #keystone-4-release if you'd like to discuss or flag any blocking/critical issues for a 4.0.0 release.

Regards,
Stennie

[stennie]

FYI, a fix for this issue is included in Keystone 4.0.0-rc.0 which is now available via npm. Release notes: https://github.com/keystonejs/keystone/releases/tag/v4.0.0-rc.0.

Regards,
Stennie