Not a bug Editor's list functionality doesn't behave as expected after editor content replaced with XF.replaceEditorContent

[MilkyMeda]

Affected version

2.1.7

My JS handler when insterting HTML into editor:

if (data.htmlText) {
                this.editor.ed.undo.saveStep();
                XF.replaceEditorContent(this.$form, data.htmlText);
                this.editor.ed.undo.saveStep();
                this.editor.focus();
            }

After replacement, this bug occurs:

Fixed - Applying unordered list don't really behave properly

I copied the following raw text in the editor and only that: Corrected bugs: Phrase xfa_dedf_params missing in changelog Wrong font selected in default editor font selection list after save ErrorException: Undefined index: font -...

xenforo.com

I render the posted BBCode as follows:

$htmlText = \XF::app()->bbCode()->render();

 

[Chris D]

Can you provide a more detailed reproduction case?

What is the value of dara.htmlText that is problematic? What is output? What are you expecting?

 

[MilkyMeda]

The output is what the BBCode renderer returns. Example:

<div class="bbWrapper">Example line 1<br />↵Example line 2</div>

Now try to add unordered list to the second line. It will add it to the first line.

 

[Chris D]

So what is the value of the text you are inserting into the editor?

 

[MilkyMeda]

Before sending it to the BBCode renderer? This:

htmlText: "<div class="bbWrapper">Line1<br />↵Line2</div>"
originalText: "Line1↵Line2"

Notice the character ↵ is a line break.

 

[Chris D]

I think this is mostly expected.

The primary issue here is that we don't actually use <br /> tags to denote a line break. We have the editor configured to wrap lines in <p> tags.

If it was inserted as <p>Line 1</p><p>Line 2</p> it would work as expected.

The editor would behave differently, I think, if we had the BR mode enabled.

The conversion from HTML to BB code produces:

Line 1
Line 2

Converting that back to HTML demonstrates the expected format.

In essence, the previous bug you pointed to was because we weren't converting to that expected format on paste. We were pasting in the <br /> line breaks as is, but now the behaviour is that we convert them to be enclosed in <p> tags.

You will see a bunch of this handling in XF.editor.editorInit() but it is specific to our paste handling and so you may want to handle it differently. If you're able to, it might be worth converting to BB code and then back again. It's a slightly roundabout way of doing it, but it produces the right output.

 

[MilkyMeda]

Removing the wrapper alone fixed it. It didn't need any conversion.

$htmlText = preg_replace('#<div class="bbWrapper">(.*)</div>#uis', '$1', $htmlText);

The editor doesn't like inner divs

 

[Mike]

Given that you mention how you're rendering the BB code, I think the problem is just that you're using the wrong renderer. You should be using the "editorHtml" renderer if you're trying to render for the RTE.