Several lines' formatted data will be merged to one line when we apply Numbers/Bullets

[Satya Minnekanti]

To reproduce the defect

1. Open ajax sample and type some 4 lines of text( eg: test 1 test 2 test 3 each on a diff line)

2. Select all the lines and apply Formatting paragraph format option to them.

3. Now click on Numbers/Bullets Icon

Expected Result:we should see that Numbers/Bullets is applied to the lines

Actual Result: All the four lines of data are merged in to a single line.

[Garry Yao]

Select all the lines and apply Formatting paragraph format option to them.

Do you mean 'Formatted' (pre) format option?

[Satya Minnekanti]

yes that is the option i meant

[Garry Yao]

Converting pre-formatted content to list should behave just like apply bock style - with line break preserved, but it's not of high priority, since at least we have the right result by first apply "Paragraph" on it before convert to list.

[Satya Minnekanti]

Yes it should behave like block style, but doesn't at the moment.

I don't understand the justification for treating it as low priority, considering that it doesn't work as the feature is meant to work.

[Frederico Caldeira Knabben]

The <pre> block is a special case in the editor as it deals with raw text, unlike other blocks, where line breaks are treated differently. It behaves differently in the editor. For example, if you switch from <pre> to <p> with a collapsed selection, the entire <pre> is converted, not only the line with the caret.

We should have the same behavior here, having the entire <pre> being moved inside an <li>, just like we're doing now with the headings tags. So, in the ticket tc, we would end up with a single <ul><li><pre>...</pre></li></ul> result.

I would understand a different behavior only if it would be possible to have lists inside <pre>, which is not true,

[Garry Yao]

...the entire <pre> is converted, not only the line with the caret.

Ok, it's a reasonable proposal as it matches the browser's native create list command also.

[Sa'ar Zac Elias]

Same patch, updated to the current revision

[Tobiasz Cudnik]

Fixed with [5875].