Item14323: Update to latest TinyMCE version

Because there was a major change in the TinyMCE APIs many versions ago, the version we use and ship has fallen well behind the curve. We need to update to a recent version of TinyMCE, and resolve any problems that creates.

This needs to be done before any other bugs are fixed, as fixing bugs against the old version is pretty pointless.

The plan is to throw out the current hacked version and build against a production, out of the box, pre-minimised TMCE that we simply drop into the plugin, without any customisation. All customisation should be done in the Foswiki plugins (TMCE plugins, jquery plugin and FW plugin). To do that I propose to add a full TMCE developer pack alongside the production version for dev (but not ship it).

Early issues noted are:

• the dropping of the advanced theme that we have been using. This can hopefully be replaced with modern.
• change to the TMCE init sequence which means the point at which we build the format menus is now before TMCE has initiated the formatter (so null pointer)

-- CrawfordCurrie - 21 Feb 2017

See also Item14288, which promises to create a common framework for plugging in editors with a shared toolbar. This framework should be able to feed directly off work done here.

-- Main.CrawfordCurrie - 21 Feb 2017 - 20:46

Reckon this is good to go now. Currently on branch Item14323, waiting for merge when end user testing is complete.

-- Main.CrawfordCurrie - 22 Mar 2017 - 20:13

Crawford, Here's the failing tests and results from git bisect

05fd01581b1d685ed063d8e2474718497e0c93b8 is the first bad commit
commit 05fd01581b1d685ed063d8e2474718497e0c93b8
Author: cdot <git@c-dot.co.uk>
Date:   Fri Mar 10 13:37:13 2017 +0000

    Item14323: convert unbulleted lists to a list type instead of a div. This uses some new classes which require a core change, so the fix won't be complete until that core change is made. However it should work even without the core change (it's a display problem only)

:040000 040000 cc7cadb8ea0c383e52c27c89e0a2f2667f2c3831 8d16f567d3beff2a21f95a0f87e8b3ae8dc425b0 M      WysiwygPlugin

And the failing tests:

----------------------------
---++ Module Failure summary
TranslatorTests has 2 unexpected results (of 2):
   * F: TranslatorTests::testTML2HTML_stickyInsideLiteral
   * F: TranslatorTests::testTML2HTML_verbatimInsideLiteralItem1980
----------------------------

-- GeorgeClark - 26 Mar 2017

The tests were wrong, probably reflecting a problem in the core rendering which has since been fixed.

-- Main.CrawfordCurrie - 26 Mar 2017 - 09:52

Reopening this. the TinyMCEPlugin topic needs updates.

• Lists of known issues needs cleanup
• Claims it is shipipng version 3.5.7
• Missing task entries from history.
  • Foswikitask:Item14323: Update to latest TinyMCE version
• Render errors in the Default Plugins list - errors about missing sections in INCLUDE statements.
• pub/System/TinyMCEPlugin/plugins/foswiki/img/colour.gif missing from MANIFEST

-- GeorgeClark - 20 Apr 2017

Reopening to get the latest version. Was 4.5.3, now (as of 30 Aug 2017 - Version 4.6.6)

-- GeorgeClark - 15 Sep 2017

4.6.7 came out a few days ago - reopening to update.

-- GeorgeClark - 27 Sep 2017

And now 4.7.3 is available.

-- GeorgeClark - 25 Nov 2017

Updated to 4.7.5

-- GeorgeClark - 26 Jan 2018

Commenting is disabled while not logged in