Labels |
Added:
?
|
the actual text is off topic - please open a separate issue for that - this issue is about the Note field itself. I just used the skins as an example of its use. It is used in many places
@infograf768 I’m pretty sure I had a pr that cleaned up this mess by moving the override template to the actual template of the site
@infograf768 feel free to resurrect:
#14510 and #14456
Both totally valid and imo needed changes. These PRs align tinyMCE to the rest of the overriding methods on Joomla
@dgrammatiko
Alas I would not know what to do with these.
@brianteeman
The issue looks related to renderfield.php which does not accept specific elements in the xml, allowing therefore to create new css classes or conditions to cope with this bad display.
Checked further:
<div>
.I know there other fields in core using the Note field type but could not find a way to display them.
To implement this, it would be useful to test these too.
There may be simpler solutions but could not find them.
@C-Lodder
Status | New | ⇒ | Discussion |
Category | ⇒ | Plugins |
Labels |
Added:
J4 Issue
|
The problem is that this field (and others) is not using a layout - instead all the markup is being generated (badly) in the library libraries\src\Form\Field\NoteField.php
@wilsonge has already pointed out that all fields need to use a layout and that will resolve everything for this note field as well.
I will try to work on this next week but if anyone else wants to do it then just go for it
Status | Discussion | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-05-29 13:47:29 |
Closed_By | ⇒ | brianteeman |
Status | Closed | ⇒ | New |
Closed_Date | 2020-05-29 13:47:29 | ⇒ | |
Closed_By | brianteeman | ⇒ |
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-12-23 21:41:46 |
Closed_By | ⇒ | brianteeman |
The correct path is
/media/vendor/tinymce/skins/
but placing a new skin there is killed when using npm ci.
As, normally, final users do not have to use npm ci when updating J, it may be possible to add the skin there and it would not be deleted. Not sure.
Also, tinymcebuilder is not designed to load the css from such a new skin. As well for init.es6.js. Maybe more in other files (theme.js, etc.).
@dgrammatiko
Shall we just get rid of the Note itself and only use lightgray or is it possible to correct all this?