YUI 3

Ticket #2529236 (closed defect)

Reporter


George 		Opened: 08/23/10
Last modified: 09/7/10
Status: closed
Type: defect
Resolution: fixed

Owner


Dav Glass 		Target Release: 3.2.0
Priority: P1 (critical)
Summary: After about 5 repostions, Simple Sortable List example becomes unresponsive in IE
Description:

launch
http://yuibuild.corp.yahoo.com/yui3/lastSuccessful/examples/sortable/simple-sortable.html

Reposition items in the simple example about 5 times.
(i moved an item from the center of the list towards the top 3 times, dragged one off to the right of the list and released mouse to have it return to its original position, then moved an item from
the center of the list to the bottom)

results, after a number of repositions example becomes unresponsive, clicking on an item will not drag it or it takes a very long time to begin tracking the drag. On 3 occasions got:
"Internet Explorer has encountered a problem and needs to close. We are sorry for the inconvenience"
Type: defect Observed in Version: development master
Component: DragDrop Severity: S1 (critical)
Assigned To: Dav Glass Target Release: 3.2.0
Location: Library Code Priority: P1 (critical)
Tags: Relates To:
Browsers: IE 7.x
URL:
Test Information:

Change History

George

YUI Developer
Posted: 08/23/10
  • component changed from None to Sortable
  • owner changed from George to Dav Glass
  • status changed from accepted to assigned

Dav Glass
Posted: 08/23/10
  • location changed to Example
  • status changed from assigned to accepted

Dav Glass
Posted: 08/24/10
  • location changed from Example to Library Code
  • priority changed from P3 (normal) to P1 (critical)
  • severity changed from S3 (normal) to S1 (critical)

I can replicate this in IE7. If I move an item, then try to move it again. IE crashes..

Dav Glass
Posted: 08/24/10
  • component changed from Sortable to DragDrop
  • owner changed from Dav Glass to Dav Glass
  • status changed from accepted to assigned

This is actually a DD.Delegate issue..

Dav Glass
Posted: 08/24/10
  • status changed from assigned to accepted

Boiled it down to the ID fix Matt and I worked on Friday for DDProxy.

It seems that calling this (to fix the duplicate id issue):



      domNode.removeAttributeNode(domNode.getAttributeNode('id'));

The first time works, however, calling it a second time (even with a try catch wrapping it) causes IE7 to crash..

Dav Glass
Posted: 08/24/10
  • resolution changed to fixed
  • status changed from accepted to checkedin

Fixes #2529236 - Refs #2529217 - Reverted the fix put in to fix the inDoc check, which was fixed in #2529232
View Commit:

George

YUI Developer
Posted: 09/7/10
  • status changed from checkedin to closed

© 2006-2013 Yahoo! Inc. All rights reserved.
All code on this site is licensed under the BSD License unless stated otherwise.
About This Site · Security Contact Info