Reusing TinyMCE instances

[ellatrix]

If we're going to have multiple TinyMCE instances (one or more for each block), it might be good to try to reuse one for all of the fields. The instance could be attached to the field that is currently focussed, undo levels could be disabled etc. Creating new blocks would also be a bit faster.

[androb]

@spocke @Afraithe maybe they should be using our inline mode?

[ellatrix]

We are using the inline mode :)

[androb]

Ha :) sorry. I thought we handled this sort of reuse of editors automagically. It is possible that it is something we did in Textbox.io and haven't yet got into TinyMCE.
#abovemypaygrade

[jasmussen]

Is this more or less addressed with the Editable component?

[ellatrix]

@jasmussen No, they're all separate instances. See also #919 (comment).

[androb]

@iseulde - is this still an issue? It would be good to understand a bit more about what you are looking for here. Feel free to ping me on Slack.

[ellatrix]

The idea was to only have one instance of TinyMCE and when the selected Editable changes, move the instance to the new Editable, discard undo levels etc. to avoid having tons of instances around. This may be something we can explore at a later stage to improve performance. Not sure how much it would improve that though.

[ellatrix]

This may still be an interesting experiment to try to speed things up. I'm noticing some really big delays when enter is pressed on iOS and a new instance needs to be created. Theoretically this will create one instance per setting variant (most blocks will have the same setting, variants include list and table), and when a new field is focussed it will attach to that field (reset target node, purge undo levels, trigger nodeChange etc.). When enter is pressed, the field and editor will be available immediately. This may also be beneficial for big posts where now an instance needs to be created for each editable field. When we reuse instance, we can just load the content with contentEditable and that's it.

[karmatosed]

As we approach the close of phase one, lets close this for now. We can always look at this in a later stage if we do want and reopen.