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

Bring back view package #4094

Closed
tobrun opened this issue Feb 24, 2016 · 2 comments · Fixed by #4095
Closed

Bring back view package #4094

tobrun opened this issue Feb 24, 2016 · 2 comments · Fixed by #4095
Assignees
Labels
Android Mapbox Maps SDK for Android

Comments

@tobrun
Copy link
Member

tobrun commented Feb 24, 2016

In same line as fix for #3755, I would bet better to not use package visibility too much.
Going to try reintroducing the view package.

The following views should be included:

  • CompassView
  • UserLocationView
@tobrun tobrun added this to the android-v4.0.0 milestone Feb 24, 2016
@tobrun tobrun added the Android Mapbox Maps SDK for Android label Feb 24, 2016
@tobrun
Copy link
Member Author

tobrun commented Feb 24, 2016

Thinking this a bit more through, I'm going to make subpackage within the maps package and name the package widgets instead of views. Mainly because the package below already contains a View (MapView), and this will introduce the concept of overlain widgets as I was thinking about for #3276.

@tobrun
Copy link
Member Author

tobrun commented Feb 24, 2016

To be able to remove package visibility issue, I was obliged exposing this in MapboxMap:

screen shot 2016-02-24 at 11 39 22

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 a pull request may close this issue.

3 participants