User tests: Successful: Unsuccessful:
Pull Request for Issue #11020 .
This patch deactivates the TinyMCE filtering, if the editor has "No filtering" in the global configuration (Tab Text filter)
<span class="fa fa-phone fa-3x sr-contact"></span> <p>123-456-6789</p>
to a articleStatus | New | ⇒ | Pending |
Labels |
Added:
?
|
Rel_Number | 0 | ⇒ | 11020 |
Relation Type | ⇒ | Pull Request for |
Category | ⇒ | Plugins |
That's (sadly) a standard behavior in the tinymce. If you toggle the editor (to html) insert some text, toggle it back and look again, it also added the <p>
See also: https://www.tinymce.com/docs/configure/content-filtering/#forced_root_block
This PR has received new commits.
CC: @brianteeman
I have tested this item
@brianteeman if you turn off <p>
for new lines in the tinyMCE Plugin it doesn´t add the additional p
I see this fix as important to be merged because otherwise everything is striped by default?
@bembelimen either i don't get the PR or you interchanged true and false. Not working for me.
Added some var_dumps and the output of verify_html for your reference:
Okay, sorry my fault. Should read the instruction properly (forgot switching to joomla filter in plugin settings).
I have tested this item
Works fine, thank you.
Status | Pending | ⇒ | Ready to Commit |
Labels |
Added:
?
|
Milestone |
Added: |
@joomla-cms-bot you record i need to call you twice on one issue for that label thing :P
I have tested this item? unsuccessfully on f85ad0e
I can confirm the issue as reported
But after applying the patch and toggling the editor and pasting the following code
<span class="fa fa-phone fa-3x sr-contact"></span> <p>123-456-6789</p>
When I toggle the editor back on and then off again it shows
<p><span class="fa fa-phone fa-3x sr-contact"></span></p>
<p>123-456-6789</p>
Note the extra < p >
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11022.