[wp-trac] [WordPress Trac] #30646: oEmbed preview loading does not trigger editor scrolling
WordPress Trac
noreply at wordpress.org
Sun Feb 15 18:11:17 UTC 2015
#30646: oEmbed preview loading does not trigger editor scrolling
------------------------------+-------------------------
Reporter: celloexpressions | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 4.2
Component: TinyMCE | Version: 4.0
Severity: minor | Resolution:
Keywords: has-patch | Focuses: javascript
------------------------------+-------------------------
Comment (by 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.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/30646#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list