Opened 8 years ago
Closed 8 years ago
#9686 closed Bug (fixed)
Missing content style for <pre>
Reported by: | Garry Yao | Owned by: | Olek Nowodziński |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.0.1 |
Component: | Documentation & Samples | Version: | 4.0 Beta |
Keywords: | Cc: |
Description
<pre> didn't get styled from this change, especially considering the white-space attribute, which helps to eliminate the overflow.
Change History (7)
comment:1 Changed 8 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 8 years ago by
Owner: | set to Olek Nowodziński |
---|---|
Status: | confirmed → assigned |
comment:4 follow-up: 5 Changed 8 years ago by
Status: | review → review_failed |
---|
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.
comment:5 Changed 8 years ago by
Status: | review_failed → review |
---|
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.
comment:6 Changed 8 years ago by
Status: | review → review_passed |
---|
I tested these styles on all supported browsers. Everything seems to be fine.
comment:7 Changed 8 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Masterized fix git:15c12e4.
Extracted tab size thread to #9776.
I was able to reproduce this problem in Opera, Firefox and IE (7-10) from CKEditor 4 beta.
To reproduce:
Amstrong spent about...
)