Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update TinyMCE to 4.5.9 #22879

Merged
merged 1 commit into from Nov 1, 2018
Merged

Update TinyMCE to 4.5.9 #22879

merged 1 commit into from Nov 1, 2018

Conversation

@brianteeman
Copy link
Member

@brianteeman brianteeman commented Oct 30, 2018

Version 4.5.9 (2018-08-02)
Fixed a bug where Edge 17 wouldn't be able to select images or tables.

Version 4.5.9 (2018-08-02)
  Fixed a bug where Edge 17 wouldn't be able to select images or tables.
@Quy
Copy link
Contributor

@Quy Quy commented Oct 31, 2018

I have tested this item successfully on 53321b9


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22879.

1 similar comment
@vc-development
Copy link

@vc-development vc-development commented Nov 1, 2018

I have tested this item successfully on 53321b9


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22879.

@Quy
Copy link
Contributor

@Quy Quy commented Nov 1, 2018

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22879.

@jsubri
Copy link
Contributor

@jsubri jsubri commented Nov 1, 2018

I have tested this item successfully on 53321b9

This is solving a long outstanding issue reported by end-users on Edge.
Screen shot below

Edited: broken English being so happy this is fixed.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22879.
@jsubri
Copy link
Contributor

@jsubri jsubri commented Nov 1, 2018

Now working with this patch
screen shot 2018-11-01 at 19 36 07


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22879.

@zero-24 zero-24 added this to the Joomla 3.9.1 milestone Nov 1, 2018
@zero-24 zero-24 merged commit 3c9c46e into joomla:staging Nov 1, 2018
5 checks passed
5 checks passed
Hound No violations found. Woof!
@joomla-cms-bot
JTracker/HumanTestResults Human Test Results: 3 Successful 0 Failed.
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/drone/pr the build was successful
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@zero-24
Copy link
Member

@zero-24 zero-24 commented Nov 1, 2018

Merged. Thanks!

@brianteeman brianteeman deleted the brianteeman:tinymce branch Nov 1, 2018
@brianteeman
Copy link
Member Author

@brianteeman brianteeman commented Nov 1, 2018

Thanks - another one bites the dust

@ssnobben
Copy link

@ssnobben ssnobben commented Nov 6, 2018

Ok thats fine.

I also see that WordPress 4.9.8 updated to TinyMCE 4.8.0 and that is 20 update versions https://www.tiny.cloud/docs/changelog/ after this version of Joomla. What stops Joomla to get to the 4.8 versions too?

@brianteeman
Copy link
Member Author

@brianteeman brianteeman commented Nov 6, 2018

Its not backwards compatible

@ssnobben
Copy link

@ssnobben ssnobben commented Nov 13, 2018

hmm ok yeah Joomla is behind Wordpress a lot now with TinyMCE and also soon with the Gutenberg editor https://frontenberg.tomjn.com/

@mbabker
Copy link
Contributor

@mbabker mbabker commented Nov 13, 2018

TinyMCE 4.6 dropped support for browsers that Joomla is still supporting; a review of that same changelog document would have shown that. Therefore, Joomla cannot upgrade the distributed TinyMCE editor without abandoning browser support.

Gutenberg is Automattic's attempt to use the open source project "founded" by their CEO to compete with other SaaS solutions by introducing an entirely new management workflow. It has nothing to do with upgrading editors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

8 participants