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

Polylines/Polygons not responding to visible/editable changes #1255

Closed
cmorford opened this issue Mar 30, 2015 · 8 comments
Closed

Polylines/Polygons not responding to visible/editable changes #1255

cmorford opened this issue Mar 30, 2015 · 8 comments
Labels
bug
Milestone
2.1.6

Comments

@cmorford
Copy link

This bug appears to have been fixed in Issue #1059, but has regressed in version 2.0.17 forward.

I modified the plnkr from the previous issue to include one polyline and one polygon to toggle visibility.

http://plnkr.co/edit/Ar6kuqf7IQO1HSjoCZ7M?p=preview
@nmccready
Copy link
Contributor

Confirmed

@nmccready nmccready added the bug label Mar 30, 2015
@nmccready
Copy link
Contributor

Feel like writing a spec for this? That way it is caught in the future?

@nmccready nmccready added this to the 2.0.20 milestone Mar 30, 2015
@cmorford
Copy link
Author

I'm just getting started into mean stack dev/coffeescript so may take me a little bit to get up to speed.

@nmccready
Copy link
Contributor

Np just asking for help as I am getting more active on other projects.

@ajones-miovision
Copy link

I've also noticed that changes to the path property (setting it to a new array) doesn't seem to get picked up.

@nmccready nmccready modified the milestones: 2.0.20, 2.1.6 Aug 16, 2015
@nmccready
Copy link
Contributor

Updated plnkr here http://plnkr.co/edit/Ar6kuqf7IQO1HSjoCZ7M?p=preview

nmccready added a commit to nmccready/angular-google-maps that referenced this issue Aug 17, 2015
@nmccready
fix(polylines polygons attribute watching): updates not get set
4274d5a 
correctly fixes issue angular-ui#1255
@nmccready nmccready added the PR'd label Aug 17, 2015
nmccready added a commit that referenced this issue Aug 18, 2015
@nmccready
Merge pull request #1441 from nmccready/issue_1436_poly_fit
1f35166 
fixes issue #1436 & #1255
@nmccready nmccready removed the PR'd label Aug 25, 2015
@markmssd
Copy link

I am using the latest version (v2.3.2), and this issue seems to have reappeared for the editable option.

@nmccready
Copy link
Contributor

Please re-open a new issue linking this one.

@angular-ui angular-ui locked and limited conversation to collaborators Apr 21, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
2.1.6
Development

No branches or pull requests
4 participants
@nmccready @markmssd @cmorford @ajones-miovision