Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issue for paths directive. #230

Closed
wants to merge 1 commit into from

Conversation

lukasz-zak
Copy link

Here is demo:
http://jsfiddle.net/49xVW/

Look on this what is in attribute paths and what map display.
Map read only default paths from $scope. It ignore attribute variables.

I also write about this in issue #229

Here is demo:
http://jsfiddle.net/49xVW/

Look on this what is in attribute paths and what map display.
Map read only default paths from `$scope`. It ignore attribute variables.
@tombatossals
Copy link
Owner

I'm revising all the directive attributes that have grown too much without testing, next will be the 'path' one, during this week.

Thanks for your report and your PR, keep you informed about this shortly.

tombatossals added a commit that referenced this pull request Jan 9, 2014
@tombatossals
Copy link
Owner

I finally managed to solve this error, it was not listening to the correct scope, so your reported behaviour occurred. Thanks so much for reporting @lukasz-zak !

@lukasz-zak
Copy link
Author

I'm waiting for new release.
Thanks for fixing.

@lukasz-zak lukasz-zak closed this Jan 9, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants