oEmbed preview loading does not trigger editor scrolling

[celloexpressions]

You can't scroll to see the full preview until you move the cursor or something.

[azaozz]

@iseulde not sure if we should call editor.nodeChanged() on iframe resize. The node hasn't really been changed :) Maybe fire a wpAdjustSize or editorSizeChanged event and listen for it from the wpautoresize plugin?

[iseulde]

Yeah, maybe that's better. Just used that because it's mainly used for UI updates.
​https://github.com/tinymce/tinymce/blob/master/js/tinymce/classes/Editor.js#L1198

[iseulde]

Hm, I still think we should use nodeChanged because if you start adding inline toolbars their position should update. And that event is used for UI updates.

[azaozz]

Right, nodeChanged is used for UI updates but also for selection/caret range updates (most notably in IE). My concern is that the iframe resize function is fired with mutationObserver that may trigger several times in a very short time while we are trying to use/update the same selection/caret range. This could result in race condition.

On the other hand comparing the iframe height to the iframe's body height before resizing reduces the number of times we fire nodeChanged, so it shouldn't fire excessively. Good way to test this is by embedding a "twit", fires only two times here.

Lets add it and see if there will be any problems in IE.

[azaozz]

In 31466:

TinyMCE wpView: fire nodeChanged when an embedded iframe is resized so we can adjust the editor height and other UI components. Props iseulde, fixes #30646.