Skip to content
This repository has been archived by the owner on Feb 22, 2018. It is now read-only.

Clean up NodeCursor and remove nodlist from it, its just node #644

Closed
mhevery opened this issue Feb 27, 2014 · 1 comment
Closed

Clean up NodeCursor and remove nodlist from it, its just node #644

mhevery opened this issue Feb 27, 2014 · 1 comment
Milestone
v0.9.9

Comments

@mhevery
Copy link
Contributor

mhevery commented Feb 27, 2014

NodeCursor has method

cursorSize() => 1;

which is left over from having directives span multiple elements. This feature is no longer needed because we will use <template> tag for this purpose. The NodeCursor code can be cleaned up.
@mhevery mhevery added this to the v1.0 milestone Feb 27, 2014
@vicb vicb self-assigned this Mar 3, 2014
@vicb vicb modified the milestones: v0.9.9, v1.0 Mar 3, 2014
@vicb vicb removed their assignment Mar 3, 2014
@vicb
Copy link
Contributor

vicb commented Mar 3, 2014

working on it

vicb added a commit to vicb/angular.dart that referenced this issue Mar 3, 2014
@vicb
fix(NodeCursor): Removes nodeList() in favor of current
e8aa234 
Closes dart-archive#644
@vicb vicb mentioned this issue Mar 3, 2014
Closed
@vicb vicb closed this as completed in aaae1d6 Mar 6, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Milestone
v0.9.9
Development

Successfully merging a pull request may close this issue.

fix(NodeCursor): Removes nodeList() in favor of current vicb/angular.dart
2 participants
@mhevery @vicb