Warning on setting up wrapper's height to 100%

Hi @artf I hope everything's going well.

This is more like a reminder or a warning message for an issue I've just had that it took me some time to figure out.

if the wrapper's height (modified by the selector #wrapper by default) is set to height: 100%; instead of min-height: 100%, the editor's tool container will take a non desirable behavior.

This happens because the scroll event will not be triggered by the window object but rather the wrapper element, this causes the onFrameScroll not to be fired in the CanvasView.

The OnFrameScroll is assigned in the renderBody function.

...
      body.append('<style>' + frameCss + '</style>');
      body.append(wrap.render()).append(cssc.render());
      body.append(this.getJsContainer());
      em.trigger('loaded');
      **this.frame.el.contentWindow.onscroll = this.onFrameScroll;**
      this.frame.updateOffset();
...

Should this warning be written somewhere in the code? Maybe in the editor configuration file where the baseCSS is set, along with the comments you have there.

What do you think?

Thanks!