Opened 13 years ago
Closed 12 years ago
#5840 closed defect (fixed)
ContentPane onUnload event is called on programatic tab creation
Reported by: | guest | Owned by: | bill |
---|---|---|---|
Priority: | high | Milestone: | 1.3 |
Component: | Dijit | Version: | 1.0 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description (last modified by )
this problem happens only when I specify the href for the ContentPane?.
In ContentPane?.js file, _downloadExternalContent function calls the unload event handler.
_downloadExternalContent: function(){ this._onUnloadHandler(); ...
I don't know why unload event handler is there.
Test page attached.
cheers
Attachments (4)
Change History (12)
Changed 13 years ago by
Attachment: | tabs_test.html added |
---|
Changed 13 years ago by
Changed 13 years ago by
comment:1 Changed 13 years ago by
Component: | General → Dijit |
---|---|
Owner: | changed from anonymous to mumme |
comment:2 Changed 13 years ago by
Well, i am doing an action and some clean-up onUnload. In my case it was easy to do a work-around checking if an element exist before performing the action and trying to delete it. so my application works but dojo calling its onUnload event is wrong.
Do you have a test case?
I attached a minimal example to show the bug. or what do you mean by "test case"?
regards,
prajwala & eduardo
comment:4 Changed 13 years ago by
Milestone: | → 1.1 |
---|
comment:5 Changed 13 years ago by
Milestone: | 1.1 → 1.2 |
---|
Move all milestone 1.1 tickets to 1.2, except for reopened tickets and tickets opened after 1.1RC1 was released.
comment:6 Changed 12 years ago by
Milestone: | 1.2 → 1.3 |
---|---|
Owner: | changed from mumme to Sam Foster |
Sam can you take a look at these? Hopefully these are fixed by your refactor or you can fix them
comment:7 Changed 12 years ago by
Description: | modified (diff) |
---|---|
Owner: | changed from Sam Foster to bill |
Status: | new → assigned |
I've been mucking in ContentPane, might as well fix this. I think it should just check the isLoaded flag and not call the onUnload handler() if it isn't set.
comment:8 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
(In [15957]) Fixes #5840: don't call ContentPane? onUnload event on initial load of href (unless the ContentPane? initially had some content) !strict
just guessing, but it's probably trying to trigger unload on any content which may be getting replaced. Does this result in a problem for you? Do you have a test case?