Missing content style for <pre>

[Garry Yao]

<pre> didn't get styled from this change, especially considering the white-space attribute, which helps to eliminate the overflow.

[Jakub Ś]

I was able to reproduce this problem in Opera, Firefox and IE (7-10) from CKEditor 4 beta.

To reproduce:

1. Go to replacebyclass sample
2. Click inside second paragraph (Amstrong spent about...)
3. Choose "Formatted" from Format Dropdown

[Olek Nowodziński]

Created branch cksource t/9686.

[Piotrek Koszuliński]

I think that all we need is:

white-space: pre-wrap; /* CSS 2.1 */
word-wrap: break-word; /* IE7 */

That seems to be enough for samples (see samples/sample.css), so should be enough for editor as well.

Also - we can think of setting tab size to 4, which is more normal then 8. However, that would introduce difference between how that looks on modern browsers and old ones. So much probably we shouldn't do that.

[Olek Nowodziński]

Replying to Reinmar:

I think that all we need is:

white-space: pre-wrap; /* CSS 2.1 */
word-wrap: break-word; /* IE7 */

That seems to be enough for samples (see samples/sample.css), so should be enough for editor as well.

Right. Pushed a commit for this.

Also - we can think of setting tab size to 4, which is more normal then 8. However, that would introduce difference between how that looks on modern browsers and old ones. So much probably we shouldn't do that.

A separate ticket.

[Piotrek Koszuliński]

I tested these styles on all supported browsers. Everything seems to be fine.

[Olek Nowodziński]

Masterized fix ​git:15c12e4.

Extracted tab size thread to #9776.