Ticket #2152 (closed defect: fixed)

Opened 3 months ago

Last modified 2 months ago

Embedded: document.onunload and document.onbeforeunload do not fire in IE6/7 when inside CustomLayout

Reported by: -Sami- Owned by: Jouni Koivuviita
Priority: minor Milestone: User Interface Library 5.2.11
Component: gwt-adapter-client Version: 5.2.10
Keywords: Cc:
Known Issue description:
Hours estimate: Deadline (dd.mm.yyyy):
Known Issue version (since): Known Issue title:
Hours done: Depends to:
Affects documentation: no
Known Issue workaround:
Affects release notes: no Contract: Efecte

Description

The problem described in #1500 still happens if the embedded component is inside a CustomLayout?. After some debugging it seems that problem is because ICustomLayout overrides onDetach() and doesn't call doDetachChildren(). So the onDetach() of the embedded gets never called.

(Created a new ticket instead of reopening the original one because this is kind of special case.)

Attachments

Ticket2152.java (1.0 kB) - added by -Sami- 3 months ago.
Test case for the ticket
test.html (26 bytes) - added by -Sami- 3 months ago.
html for the custom layout
a.html (158 bytes) - added by -Sami- 3 months ago.
Source for the embedded component

Change History

Changed 3 months ago by -Sami-

Test case for the ticket

Changed 3 months ago by -Sami-

html for the custom layout

Changed 3 months ago by -Sami-

Source for the embedded component

Changed 3 months ago by Mauno Haukila

document.onunload doesn't work with IE correctly, you have to use document.beforeunload event.

Changed 3 months ago by -Sami-

  • summary changed from Embedded: document.onunload does not fire in IE6/7 when inside CustomLayout to Embedded: document.onunload and document.onbeforeunload does not fire in IE6/7 when inside CustomLayout

Well, that isn't fired either. (Updated the summary)

Changed 3 months ago by -Sami-

  • summary changed from Embedded: document.onunload and document.onbeforeunload does not fire in IE6/7 when inside CustomLayout to Embedded: document.onunload and document.onbeforeunload do not fire in IE6/7 when inside CustomLayout

Changed 3 months ago by Joonas Lehtinen

  • priority changed from undefined to minor

Changed 2 months ago by Joonas Lehtinen

  • owner changed from ticketmaster to Jouni Koivuviita
  • status changed from new to assigned

Changed 2 months ago by Jouni Koivuviita

  • milestone changed from User Interface Library 5 stable branch backlog to User Interface Library 5.2.11

Changed 2 months ago by Jouni Koivuviita

  • status changed from assigned to closed
  • resolution set to fixed

Fix for 5.2 branch in [5731].

Fix for 5.3 branch in [5733].

Note: See TracTickets for help on using tickets.