Opened 16 years ago
Closed 16 years ago
#2764 closed Bug (fixed)
Source area is having the wrong size in IE
Reported by: | Martin Kou | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Must have (possibly next milestone) | Milestone: | CKEditor 3.0 |
Component: | UI : Source View | Version: | SVN (FCKeditor) - Retired |
Keywords: | Confirmed IE Review+ | Cc: |
Description
To reproduce:
- Open the replacebyclass.html sample in IE6.
- Add a default 2x3 table there.
- Switch to source mode.
- The HTML source for the table is seemingly not there... but it's only because the source mode text area is too small and is having s scroll bar.
Attachments (5)
Change History (15)
comment:1 Changed 16 years ago by
Keywords: | Confirmed IE added |
---|
comment:2 Changed 16 years ago by
Priority: | Normal → High |
---|
comment:3 Changed 16 years ago by
Owner: | set to Garry Yao |
---|
Changed 16 years ago by
Attachment: | plugin.js.diff added |
---|
comment:4 Changed 16 years ago by
Keywords: | Review? added |
---|
comment:5 Changed 16 years ago by
Keywords: | Review- added; Review? removed |
---|
Review- because the patch path and the coding styles of this patch do not conform to our standards. I've talked to Garry about the coding standards we have.
The patch works otherwise.
comment:6 Changed 16 years ago by
Hmm...
- The automatic code formatter seems to have turned a lot of the tabs into spaces - that's bad.
- Same for the comments, it reformatted the comments to exceed the 80-character limit in the style guidelines.
- The 0xfeff character at line 7 makes the patch unusable with GNU patch or TortoiseSVN. I guess you'd better generate the patch with TortoiseSVN rather than with the Eclipse plugin.
Changed 16 years ago by
Attachment: | 2764_3.patch added |
---|
comment:7 Changed 16 years ago by
Keywords: | Review? added; Review- removed |
---|
comment:8 Changed 16 years ago by
Status: | new → assigned |
---|
comment:9 Changed 16 years ago by
Keywords: | Review+ added; Review? removed |
---|
Putting to high priority so it will be fixed before the beta release.