Recent Topics
  1. b2evolution CMS Support Forums
  2. Archives
  3. Older bugs
  4. Bugs in versions 1.9.x
  5. Ordered lists break validation

Ordered lists break validation

1 Apr 19, 2007 16:14    

Using the quicktag button for OL produces <br> tags after all but the last </li> tag. This is with auto-P on by the way, just in case that's part of the solution.

This is what I have in a blog post: 

<ul>
<li>Unordered Lists suck to make.</li>
<li>They are not friendly.</li>
<li>But they can be styled!</li>
</ul>

<ol>
<li>Ordered Lists suck to make.</li>
<li>They are not friendly.</li>
<li>But they can be styled!</li>
</ol>

Viewing the source reveals this: 

<ul>
<li>Unordered Lists suck to make.</li>
<li>They are not friendly.</li>
<li>But they can be styled!</li>
</ul>

<ol>
<li>Ordered Lists suck to make.</li><br>
<li>They are not friendly.</li><br>
<li>But they can be styled!</li>
</ol>

BTW I can make it work properly by putting my everything for the ordered list on one line in the blog post. Like this: 

<ol><li>Ordered Lists suck to make.</li><li>They are not friendly.</li><li>But they can be styled!</li></ol>

---------------

Unrelated: why would b2evolution produce <br> when it should be <br />?

Back to top

2 Apr 20, 2007 10:37

Crack open auto-p and remove the ol from this bit ( approx line 40 ) and see if that cures your problem :-

	var $br_allowed_in = array(
		// Block level:
		'address', 'center', 'dl', 'dir', 'div', 'fieldset', 'form', 'h1', 'h2', 'h3', 'h4', 'h5', 'h6', 'hr', 'isindex', 'menu', 'noframes', 'noscript', 'ol', 'p', 'pre',

The plugin actually adds <br /> ?

	function autobr( $text, $replace_last = true )
	{
		if( ! $this->use_auto_br )
		{ // don't make <br />'s
			return $text;
		}

		return preg_replace( '~(?<!<br />)\n'.( $replace_last ? '' : '(?!\z)' ).'~i', "<br />\n", $text );
	}

¥

Back to top

3 Apr 22, 2007 09:28

I'll be darned! Removing ol from that list worked, but for sure the problem was having a <br> tag - not a <br /> tag. There are a handful of files that have "<br>" in them, but none of them seemed like they would kick in when rendering happened.

Anyway that solves the problem so hooray!

Back to top


Form is loading...