Design Crashes! - Li button issue - Editor not filtering improper HTML usage.

[Pebbleontheshore]

When I create a topic or reply for a topic, if I click "li" button without "ul" or "ol" and submit the whole design crashes. When the participant user unknowingly do this this will be a problem. Then I disabled fancy editor and also tried <li>test</li> same thing happens. The design crashes. It shows if the end user gives improper or illogical html syntax within the content box the design breaks.

[Pebbleontheshore]

Li button issue design crash!

[johnjamesjacoby]

Unfortunately, this is indicative of a larger problem in WordPress's force_balance_tags() function, which is that there is no routine or code for repairing invalid markup if parent tags are missing but child tags are present.

It's definitely annoying. The visual editor did originally bypass this problem, but introduced others related to posting code snippets.

Going to move this into Future Release, so we can keep an eye on this, and iterate on how this works over time.

Thanks for the report!

[netweb]

Via Robkk in #2967:

If a user creates an html list without any surrounding tags defining the order to display the list in a topic or reply, the layout of the topic will break. Users in the support forum on bbpress.org do this all the time.

To test just post something like this in a topic or reply in your test forum.

<li></li>
<li></li>
<li></li>

[netweb]

#2967 was marked as a duplicate

[Track77]

As a temporary solution i using this code.

add_filter('bbp_get_reply_content', 'fix_li_tags');
function fix_li_tags( $content) {
    return "<ul>$content</ul>";
}

[netweb]

Also creating a post with the following content breaks the layout on w.org and bbpress.org support forums

<li>
<ol>
first
</ol>
<ol>
second
</ol>
</li>

See ​https://wordpress.org/support/topic/code-blocks-ordered-lists/

And discussion ​https://wordpress.slack.com/archives/forums/p1488244520914118

[netweb]

Related ​#meta20

[SergeyBiryukov]

There's an ​experimental patch on ​#meta20 that could be used as a base for potential fix.