Opened 14 years ago
Closed 14 years ago
#6791 closed Bug (fixed)
[IE7] Editor does not show in IE 7 when textarea name="description"
| Reported by: | Wes | Owned by: | Sa'ar Zac Elias |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.5.1 |
| Component: | General | Version: | |
| Keywords: | IE | Cc: |
Description
IE 7 ONLY
<textarea name="description">content here</textarea>
The rich editor will not initialize in IE 7 when the textarea's name is set to description.
Attachments (5)
Change History (18)
comment:1 Changed 14 years ago by
comment:2 Changed 14 years ago by
| Summary: | Editor does not show in IE 7 when textarea name="description" → [IE7] Editor does not show in IE 7 when textarea name="description" |
|---|
comment:3 Changed 14 years ago by
| Keywords: | IE7 Internet Explorer initialize show removed |
|---|---|
| Milestone: | → CKEditor 3.5.1 |
| Status: | new → confirmed |
I was about to open a ticket for it, as a follow up to this page:
http://extrafuture.com/2010/12/07/ckeditor-and-ie7/
The problem is related to the presence of a <meta name="description"> tag. In fact, any textarea with name matching meta tag names will be troublesome.
I've attached a TC for it. The problem can be confirmed in IE8 Compatibility mode as well.
I think we have dealt with this issue in v2 and the solution should be quite simple. It's just a matter of porting it to v3.
comment:4 Changed 14 years ago by
| Owner: | set to Tobiasz Cudnik |
|---|---|
| Status: | confirmed → assigned |
Changed 14 years ago by
| Attachment: | 6791.patch added |
|---|
comment:5 Changed 14 years ago by
| Status: | assigned → review |
|---|
comment:6 Changed 14 years ago by
| Status: | review → review_failed |
|---|
- Patch breaks the editor in pages where there's no element with an id of "description".
Changed 14 years ago by
| Attachment: | 6791_2.patch added |
|---|
comment:7 Changed 14 years ago by
| Status: | review_failed → review |
|---|
comment:8 Changed 14 years ago by
| Status: | review → review_failed |
|---|
Just to be sure, how about
element.tagName.toLowerCase() in CKEDITOR.dtd.head
Please always add back-ref comments to the ticket.
Changed 14 years ago by
| Attachment: | 6791_3.patch added |
|---|
comment:9 Changed 14 years ago by
| Status: | review_failed → review |
|---|
DTD approach seems to be more universal, attaching updated patch.
comment:10 Changed 14 years ago by
| Status: | review → review_failed |
|---|
Let's inline DTD tag names for now, that would not change the footprint of ckeditor_basic.js
Changed 14 years ago by
| Attachment: | 6791_4.patch added |
|---|
comment:11 Changed 14 years ago by
| Owner: | changed from Tobiasz Cudnik to Sa'ar Zac Elias |
|---|---|
| Status: | review_failed → review |
comment:12 Changed 14 years ago by
| Status: | review → review_passed |
|---|
comment:13 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [6301].
spent the last 4 hours trying to figure this out. hope it helps. There are a ton of related bug reports I found.