Ticket #2528284 (closed defect)

Reporter


Nate Cavanaugh
Opened: 09/26/09
Last modified: 03/31/10
Status: closed
Type: defect
Resolution: fixed

Owner


Matt Sweeney
Target Release: 3.1.0
Priority: P3 (normal)
Summary: NodeList.refresh does not work on NodeLists created via Node.queryAll
Description:

If you do:

var children = Y.get('#el').queryAll('.child')

and one of the positions of the element changes, calling children.refresh() has no effect.

This is due to the queryAll method passing in an array to Y.all() and returning that result. This means that the query for the NodeList is never saved.

One way around this would be instead of just storing the query, NodeList would also store the original context. This would allow refresh to refresh the actual conditions of the original query.

Type: defect Observed in Version: development master
Component: NodeList Severity: S3 (normal)
Assigned To: Matt Sweeney Target Release: 3.1.0
Location: Priority: P3 (normal)
Tags: Relates To:
Browsers: N/A
URL:
Test Information:

Change History

Matt Sweeney

YUI Developer

Posted: 09/26/09
  • priority changed to P3 (normal)
  • status changed from new to checkedin

Thanks for the report. I've checked in a fix for this.

George

YUI Developer

Posted: 03/31/10
  • milestone changed to 3.1.0

George

YUI Developer

Posted: 03/31/10
  • resolution changed to fixed
  • status changed from checkedin to closed