Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#3217 closed defect (fixed)

ContentPane: pane on <tr>, setContent() on produces error in IE

Reported by: guest Owned by: mumme
Priority: high Milestone: 0.9
Component: Dojox Version: 0.4.1
Keywords: Cc:
Blocked By: Blocking:

Description

I have a Table row that I am using as a contentpane: <tr id="cp1" dojoType="ContentPane?"></tr>

I then use setContent in a user activated script to load the contents of the row. This works marvelously in Firefox but in IE I get an error. Dojo.debug says it is an unknown runtime error.

IE is notoriously hard to debug (no Firebug or anything for it) so I am not able to find out more information than this.

Attachments (1)

test.html (789 bytes) - added by guest 12 years ago.
Example script

Download all attachments as: .zip

Change History (8)

Changed 12 years ago by guest

Attachment: test.html added

Example script

comment:1 Changed 12 years ago by bill

Component: GeneralDijit
Owner: changed from anonymous to Adam Peller
Summary: ContentPane.setContent produces error in IEContentPane: pane on <tr>, setContent() on produces error in IE

I don't think we really want to support this on a <tr>. Although it is possible like this on widget templates (see the MenuItem? template for an example) so maybe we can leverage that code.

comment:2 Changed 12 years ago by guest

It turns out this is due to the way IE handles TR. It makes it read-only because they have their own Table Object Model method of doing it. See http://msdn2.microsoft.com/en-us/library/ms533897.aspx

and http://msdn2.microsoft.com/en-us/library/ms532998.aspx

It would be nice if dojo would detect IE and use the IE method of updating rather than just setting innerHTML.

comment:3 Changed 12 years ago by Adam Peller

Owner: changed from Adam Peller to mumme

comment:4 Changed 12 years ago by mumme

Component: DijitDojox
Milestone: 1.0
Status: newassigned

dijit.layout.ContentPane? won't support this, but I'l try to make it work in dojox ContentPane?

comment:5 Changed 12 years ago by mumme

(In [9803]) Initial checkin of a enhanced ContentPane?, with support for scripts, styles, path adjustments, much like in 0.4, but with some enhancements. Removed the java handler code though (nobody requested it). refs #3594, refs#2852, refs #2040, refs #3217, refs #3738

comment:6 Changed 12 years ago by mumme

Resolution: fixed
Status: assignedclosed

This should work now in dojox.layout.ContentPane?. Please reopen if not.

For the archives, the html must still be valid table html (your test.html is)

<table dojoType='dojox.layout.ContentPane'></table>

your html content must be
<tbody|thead>
   <tr>
     <td>cell 1</td>
     ...
   </tr>
   ...
</tbody|thead>

comment:7 Changed 12 years ago by mumme

Milestone: 1.00.9
Note: See TracTickets for help on using tickets.