From: <no...@so...> - 2001-01-09 11:49:42
|
Bug #128155, was updated on 2001-Jan-09 03:49 Here is a current snapshot of the bug. Project: DynAPI 2 Category: Core API Status: Open Resolution: None Bug Group: None Priority: 5 Submitted by: camhart Assigned to : nobody Summary: Overriding setHTML method in sub class of DynLayer problem Details: I've made a widget that is a sub class of DynLayer. This widget overrides the behaviour of setHTML so that a call to setHTML changes the HTML of a child layer of the widget, rather than the widget itself. This is causing the Layer creation to break in IE5 and Mozilla (but not NS4) presumably because when DynLayer calls setHTML during layer creation in createElement, it been replaced with my method. NS4 is not using setHTML in createElement. This is only a bug if you think people should be allowed to override the setHTML method when creating sub classes of dynlayer. However this did used to work in earlier versions of the API. The line that causes this to break is line 91 in dynlayer.js: dlyr.setHTML(dlyr.getInnerHTML(),false); For detailed info, follow this link: http://sourceforge.net/bugs/?func=detailbug&bug_id=128155&group_id=5757 |
From: <no...@so...> - 2001-01-12 19:15:01
|
Bug #128155, was updated on 2001-Jan-09 03:49 Here is a current snapshot of the bug. Project: DynAPI 2 Category: Core API Status: Closed Resolution: Wont Fix Bug Group: None Priority: 5 Submitted by: camhart Assigned to : dcpascal Summary: Overriding setHTML method in sub class of DynLayer problem Details: I've made a widget that is a sub class of DynLayer. This widget overrides the behaviour of setHTML so that a call to setHTML changes the HTML of a child layer of the widget, rather than the widget itself. This is causing the Layer creation to break in IE5 and Mozilla (but not NS4) presumably because when DynLayer calls setHTML during layer creation in createElement, it been replaced with my method. NS4 is not using setHTML in createElement. This is only a bug if you think people should be allowed to override the setHTML method when creating sub classes of dynlayer. However this did used to work in earlier versions of the API. The line that causes this to break is line 91 in dynlayer.js: dlyr.setHTML(dlyr.getInnerHTML(),false); Follow-Ups: Date: 2001-Jan-12 11:15 By: dcpascal Comment: Not a real bug, it's upto the widget developer to do a check for the this.created flag.. if it's not set, then call the original DynLayer's setHTML() ------------------------------------------------------- For detailed info, follow this link: http://sourceforge.net/bugs/?func=detailbug&bug_id=128155&group_id=5757 |
From: Michael P. <mp...@ph...> - 2001-01-09 12:40:38
|
<!doctype html public "-//w3c//dtd html 4.0 transitional//en"> <html> Can you give an example of your custom method. If you place a check for "this.created==true", it should stop it from executing during the initial creation stage. <p>no...@so... wrote: <blockquote TYPE=CITE>Bug #128155, was updated on 2001-Jan-09 03:49 <br>Here is a current snapshot of the bug. <p>Project: DynAPI 2 <br>Category: Core API <br>Status: Open <br>Resolution: None <br>Bug Group: None <br>Priority: 5 <br>Submitted by: camhart <br>Assigned to : nobody <br>Summary: Overriding setHTML method in sub class of DynLayer problem <p>Details: I've made a widget that is a sub class of DynLayer. This widget <br>overrides the behaviour of setHTML so that a call to setHTML changes the <br>HTML of a child layer of the widget, rather than the widget itself. This is <br>causing the Layer creation to break in IE5 and Mozilla (but not NS4) <br>presumably because when DynLayer calls setHTML during layer creation in <br>createElement, it been replaced with my method. NS4 is not using setHTML in <br>createElement. <p>This is only a bug if you think people should be allowed to override the <br>setHTML method when creating sub classes of dynlayer. <p>However this did used to work in earlier versions of the API. <p>The line that causes this to break is line 91 in dynlayer.js: <p>dlyr.setHTML(dlyr.getInnerHTML(),false); <p>For detailed info, follow this link: <br><a href="http://sourceforge.net/bugs/?func=detailbug&bug_id=128155&group_id=5757">http://sourceforge.net/bugs/?func=detailbug&bug_id=128155&group_id=5757</a> <p>_______________________________________________ <br>Dynapi-Dev mailing list <br>Dyn...@li... <br><a href="http://lists.sourceforge.net/mailman/listinfo/dynapi-dev">http://lists.sourceforge.net/mailman/listinfo/dynapi-dev</a></blockquote> <p>-- <br>Michael Pemberton <br>mp...@ph... <br>ICQ: 12107010 <br> </html> |
From: Eytan H. <ey...@tr...> - 2001-01-09 13:43:36
|
First of all I agree you should check for created. Also you might leave = an old reference to the old setHTML (call it oldsetHTML or some other = creative name) then if you ever need to use it (let's say that IE needs = it then just perofrm a check if not created call the original. 8an |