List indentation is not working

[Frederico Caldeira Knabben]

1. Load this HTML:

<ul>
	<li>Line 1</li>
	<li>Line 2</li>
</ul>

2. CTRL+A to select all.
3. Click the "Decrease Indentation" button.

The list will not be removed, and the items contents will be now inside <p> elements.

We have the same issue when using "Increase Indentation" on step 3, having the indentation being applied to the new <p> instead of the <ul>.

[Frederico Caldeira Knabben]

It's R+ for me, but it looks like you forgot putting it on review.

Be sure to open a ticket for Firebug before committing, as we should not have such "_firebugConsole" checks in the code.

[Sa'ar Zac Elias]

Replying to fredck:

Be sure to open a ticket for Firebug before committing, as we should not have such "_firebugConsole" checks in the code.

The issue was already reported by Garry as ​issue 2960 and has been fixed in Firebug 1.6a9.

[Tobiasz Cudnik]

Indeed i forgot to put it on a review, sorry. I've removed Firebug related code in second patch and updated bookmarks to new prefix (cke).

[Sa'ar Zac Elias]

I can still reproduce the bug in FF.

[Tobiasz Cudnik]

You need to disable firebug prior to version 1.6 to have it working.

[Tobiasz Cudnik]

Fixed with [5842].