Fix document not scrolling in iOS (all browsers)

[ZenulAbidin]

This patches a bug where users cannot scroll a document on iOS, because iOS does not allow iframes to be scrolled.

See https://davidwalsh.name/scroll-iframes-ios for details.

This is purely a cosmetic change, so a frontend test is not applicable for it.

Tested on iOS 12

[webzwo0i]

Thank you!
Did you test if the problem is also present in the timeslider?

[ZenulAbidin]

Thank you! Did you test if the problem is also present in the timeslider?

The issue doesn't appear to be present in the timeslider on any browser.

[JohnMcLear]

I imagine this will break some plugins..

Afaik some plugins reference #editorcontainer > #editorcontainerbox and wrapping this will cause that selector to no longer function.

I am not 100% certain but I imagine this PR is going to cause issues so I wont merge until I'm sure.

[SamTV12345]

I'm not sure if Dependabot also rebases non Dependabot pull requests 😅

[ZenulAbidin]

I'm not sure if Dependabot also rebases non Dependabot pull requests 😅

Why would it need to rebase this? It's only one commit.

[SamTV12345]

I'm not sure if Dependabot also rebases non Dependabot pull requests 😅

Why would it need to rebase this? It's only one commit.

There is a good explanation here about rebasing. In essence it puts your commit on top of the git commits. We want to test your commits against the latest develop branch so we can make sure that your changes work with the latest version and not with the version that you forked from 1/2 year ago. I'll let the pipelines run and after that I'll take a look if any plugin might be broken with this change. @JohnMcLear Could you point me in a direction. I'm not quite up to date which plugins there are and which might be broken after this.

[JohnMcLear]

Thisbadds an element as a parent before the editorcontainer.

Any CSS reference to foo > bar > editorcontainer could be broken by this afaik but this I'm only 50% confident.