?
avatar brianteeman
brianteeman
27 Mar 2017

Steps to reproduce the issue

screenshotr10-16-24

in the advanced options tab for tinymce are two options for HTML Height and Width
As can be seen from the screenshot above the tooltip refers to them only working in Advanced and Extended mode. But we no longer have either of those modes.

avatar brianteeman brianteeman - open - 27 Mar 2017
avatar joomla-cms-bot joomla-cms-bot - change - 27 Mar 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 27 Mar 2017
avatar brianteeman
brianteeman - comment - 28 Mar 2017

If someone can work out when they are used i can update the strings

avatar joomdonation
joomdonation - comment - 28 Mar 2017

As I understand, It supposes to be used as Default Height and Default Width of the editor when no width and height is provided on editor form field type https://docs.joomla.org/Editor_form_field_type

However, due to a bug in the current code, that Width and Height is always use even the width and height provided in field definition (that bug should be fixed)

avatar brianteeman
brianteeman - comment - 28 Mar 2017

My comment is that it states

Only applies in Advanced and Extended mode.

We dont have those modes anymore.

avatar schnuti
schnuti - comment - 30 Mar 2017

I had a short look and I would say those parameters should be removed.
width and height are set in the editor form field with defaults: width= 100%, height=500 (px)
That is used in the article text edit.

Edit:
If a developer is using direct call he should fill the needed values
$editor->display('name'....

The values set in the plugin could be used as default in this case, but how do you explain this setting to the user?

Are 0 valid for width and height in some case? I would guess so and then you can't use any default in the display method.

#14835
#14033

If you really want to customize the values you can use a system plugin - method onBeforeRender()

avatar franz-wohlkoenig franz-wohlkoenig - change - 30 Mar 2017
Category com_plugins
avatar franz-wohlkoenig franz-wohlkoenig - change - 2 Apr 2017
Priority Medium Low
Status New Discussion
avatar brianteeman
brianteeman - comment - 3 Apr 2017

Closed see #15059

avatar brianteeman brianteeman - change - 3 Apr 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-04-03 22:16:29
Closed_By brianteeman
avatar brianteeman brianteeman - close - 3 Apr 2017

Add a Comment

Login with GitHub to post a comment