? ? Pending
Pull Request for # 29181

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
23 May 2020

Fixes #29181.

Summary of Changes

Fixes size units not working in TinyMCE.

Testing Instructions

Edit TinyMCE plugin.
Set HTML Width to 50%.
View a page containing TinyMCE editor.

Expected result

Editor is 50% wide.

Actual result

Editor is 100% wide.

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 23 May 2020
avatar SharkyKZ SharkyKZ - change - 23 May 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 May 2020
Category Front End Plugins
avatar SharkyKZ SharkyKZ - change - 23 May 2020
Rel_Number 29181
Relation Type Pull Request for
Labels Added: ?
avatar C-Lodder C-Lodder - test_item - 24 May 2020 - Tested successfully
avatar C-Lodder
C-Lodder - comment - 24 May 2020

I have tested this item successfully on 2dea258


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

avatar adj9
adj9 - comment - 24 May 2020

@SharkyKZ Is the test run on the backend?

How do I reduce the percentage? TinyMCE is not in the plugin list.


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

avatar C-Lodder
C-Lodder - comment - 24 May 2020

@adj9 Unless you've uninstalled TinyMCE, it will be in your plugins list.

avatar adj9
adj9 - comment - 24 May 2020
  • Apply a PR
  • Change the value of HTML Width to 375

The plugin screen changes
image

avatar toivo
toivo - comment - 24 May 2020

@SharkyKZ Started to test in Nightly Build of 24 May, which does not seem to be the right test environment because of the following inconsistencies in the plugin Editor - TinyMCE:

  • HTML Width is pixels, not percentage, in the Advanced tab
  • the Advanced tab displays language strings, for example PLG_TINY_FIELD_HTMLWIDTH_DESC
  • the Plugin tab displays PHP warning:
    Warning: htmlspecialchars() expects parameter 1 to be string, array given in C:\www\joomla4\layouts\joomla\form\field\text.php on line 93
    This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29204.
avatar toivo toivo - test_item - 24 May 2020 - Tested unsuccessfully
avatar toivo
toivo - comment - 24 May 2020

I have tested this item ? unsuccessfully on 2dea258

Started to test in Nightly Build of 24 May but stopped because HTML Width is pixels, language string untranslated and PHP warning.


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

avatar adj9
adj9 - comment - 24 May 2020

I adding a feature.
By setting the default parameters, I disable the PR, return to the plugin and have:
image

avatar SharkyKZ
SharkyKZ - comment - 24 May 2020

@toivo @adj9 this PR is for staging but you are testing it on 4.0.

avatar toivo
toivo - comment - 24 May 2020

@SharkyKZ - Sorry, need to check [4.0] first...


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

avatar Quy Quy - test_item - 25 May 2020 - Tested successfully
avatar Quy
Quy - comment - 25 May 2020

I have tested this item successfully on 2dea258


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

avatar Quy Quy - change - 25 May 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 25 May 2020

RTC


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

avatar HLeithner HLeithner - change - 26 May 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-05-26 12:39:13
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 26 May 2020
avatar HLeithner HLeithner - merge - 26 May 2020
avatar HLeithner
HLeithner - comment - 26 May 2020

Thanks

Add a Comment

Login with GitHub to post a comment