User tests: Successful: Unsuccessful:
Pull Request for Issue #46320
This update improves the Joomla CMS TinyMCE editor plugin configuration and behavior consistency so the editor will respect the settings in the plugin configuration. The following changes have been implemented:
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Administration Language & Strings Repository NPM Change SQL Installation Front End Plugins |
| Labels |
Added:
Language Change
NPM Resource Changed
PR-6.0-dev
|
||
| Category | Administration Language & Strings Repository NPM Change SQL Installation Front End Plugins | ⇒ | Administration Language & Strings Repository NPM Change SQL Installation Postgresql Front End Plugins |
There is a bug in tinymce
Resizing the editor’s height would add a fixed width value.
https://www.tiny.cloud/docs/tinymce/latest/changelog/
You should probably up date tinymce before debugging further
@brianteeman is this something we have to do in an other PR? I've never updated TinyMCE, so could use some help if that is necessary,
When should the values from the plugin settings be respected?
The priorities is following:
But because of #46320 (comment) the plugin never reach "plugin parameters"
So if no values are set in the form field the value of the tinymce config should be respected?
I'm now working on a solution so that when the form field editor is used, the width and height, if set, will be respected. If not defined, they will fall back to 100% width and 500px hight. The settings in the plugin will only reflect on the editor in the article edit.
Which is better? I guess 1...
So if no values are set in the form field the value of the tinymce config should be respected?
Yes, right. It already works like that, kind of.
To make it fully work it need to remove defaults from the editor field
joomla-cms/libraries/src/Form/Field/EditorField.php
Lines 208 to 209 in 508ac3d
But this can break editors sizing of existing installations.
@brianteeman is this something we have to do in an other PR? I've never updated TinyMCE, so could use some help if that is necessary,
You would need to check with the maintainers if they will accept an update to TinyMCE in a patch release. I only mentioned it because at least from their changelog it is relevant to this PR. I wouldnt want you to waste time fixing a bug if its already been fixed
| Title |
|
||||||
Your changes break possibility to set height per editor, when the form have multiple editors:
First field should be 200 by height and second field should be "plugin default" by height.