Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[android] Tidy up annotations API #2651

Merged
merged 1 commit into from
Oct 17, 2015
Merged

Conversation

ljbade
Copy link
Contributor

@ljbade ljbade commented Oct 17, 2015

Make unimplemented properties package private.
Make annotations object properties immutable.
Make InfoWindow and Circle classes package private.
Mininal version of #2546 suitable for v2.1.0 release.

@ljbade ljbade added the Android Mapbox Maps SDK for Android label Oct 17, 2015
@ljbade ljbade added this to the android-v2.1.0 milestone Oct 17, 2015
public long getId() {
return id;
}

public boolean isVisible() {
boolean isVisible() {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why remove public?

@ljbade
Copy link
Contributor Author

ljbade commented Oct 17, 2015

@bleege Core GL does not currently implement the visible or isVisible properties.
setAlpha on an existing Polyline/Polygon object will not update the marker already on the map and will confuse developers. It is only needed for the builder pattern in PolylineOptions/PolygonOptions.

Also I just remembered that Marker does not support alpha at all in Core GL. Will remove from MarkerOptions.

@ljbade
Copy link
Contributor Author

ljbade commented Oct 17, 2015

Also made methods private instead of package private where I could.

Make unimplemented properties package private.
Make annotations object properties immutable.
Make InfoWindow and Circle classes package private.
Remove no longer needed casts from Options classes.
Added a few missing get/add methods to annotations classes for consistency.
Mininal version of #2546 suitable for v2.1.0 release.
@ljbade ljbade merged commit 1c5bab1 into release-android-v2.1.0 Oct 17, 2015
@ljbade ljbade deleted the 2546-min-v2.1.0 branch October 17, 2015 01:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android Mapbox Maps SDK for Android
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants