-
Notifications
You must be signed in to change notification settings - Fork 5
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
Merge arcgis web #42
Merge arcgis web #42
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that there are minimal changes for the iOs and Android implementations.
@@ -100,20 +74,6 @@ extension FieldJsonExtension on Field { | |||
}; | |||
} | |||
|
|||
extension ArcGisMapAttributesJsonExtension on ArcGisMapAttributes { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not needed anymore because Attributes has now a Map
parameter.
/// Examples of polygon areas | ||
final List<List<LatLng>> redPolygon = [ | ||
<List<double>>[ | ||
const List<List<List<double>>> redPolygon = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed it back to List<double>
.
Let's see in a following PR if it makes sense for every platform to change it back to LatLng
Merge the newer features of arcgis web
Already merged:
Web functionality bug:
iOs functionality bug:
Android functionality bug: