Use Android View instead of runtime styling for way name

Author: danesfeder

codecov.yml

@@ -27,4 +27,8 @@ comment:
 
 ignore:
   - "**/ManeuversStyleKit.java"
-  - "**/LaneStyleKit.java"
+  - "**/LaneStyleKit.java"
+  - "**/LaneStyleKit.java"
+  - "**/NavigationViewInstanceState.java"
+  - "**/NavigationMapboxMapInstanceState.java"
+  - "**/NavigationConstants.java"

libandroid-navigation-ui/src/main/java/com/mapbox/services/android/navigation/ui/v5/NavigationContract.java

@@ -1,6 +1,7 @@
 package com.mapbox.services.android.navigation.ui.v5;
 
 import android.location.Location;
+import android.support.annotation.NonNull;
 
 import com.mapbox.api.directions.v5.models.DirectionsRoute;
 import com.mapbox.geojson.Point;
@@ -18,6 +19,8 @@ interface View {
 
     void updateWaynameVisibility(boolean isVisible);
 
+    void updateWaynameView(@NonNull String wayName);
+
     void updateCameraTrackingMode(@NavigationCamera.TrackingMode int trackingMode);
 
     void resetCameraPosition();

libandroid-navigation-ui/src/main/java/com/mapbox/services/android/navigation/ui/v5/NavigationPresenter.java

@@ -1,6 +1,7 @@
 package com.mapbox.services.android.navigation.ui.v5;
 
 import android.location.Location;
+import android.support.annotation.NonNull;
 import android.support.design.widget.BottomSheetBehavior;
 
 import com.mapbox.api.directions.v5.models.DirectionsRoute;
@@ -77,6 +78,15 @@ void onInstructionListVisibilityChanged(boolean visible) {
     }
   }
 
+  void onWayNameChanged(@NonNull String wayName) {
+    if (wayName.isEmpty()) {
+      view.updateWaynameVisibility(false);
+      return;
+    }
+    view.updateWaynameView(wayName);
+    view.updateWaynameVisibility(true);
+  }
+
   void onRouteOverviewClick() {
     view.updateWaynameVisibility(false);
     view.updateCameraRouteOverview();

libandroid-navigation-ui/src/main/java/com/mapbox/services/android/navigation/ui/v5/NavigationView.java

@@ -1,7 +1,6 @@
 package com.mapbox.services.android.navigation.ui.v5;
 
 import android.app.Activity;
-import android.app.Fragment;
 import android.arch.lifecycle.LifecycleObserver;
 import android.arch.lifecycle.LifecycleOwner;
 import android.arch.lifecycle.ViewModelProviders;
@@ -32,6 +31,7 @@
 import com.mapbox.services.android.navigation.ui.v5.instruction.NavigationAlertView;
 import com.mapbox.services.android.navigation.ui.v5.map.NavigationMapboxMap;
 import com.mapbox.services.android.navigation.ui.v5.map.NavigationMapboxMapInstanceState;
+import com.mapbox.services.android.navigation.ui.v5.map.WayNameView;
 import com.mapbox.services.android.navigation.ui.v5.summary.SummaryBottomSheet;
 import com.mapbox.services.android.navigation.v5.location.replay.ReplayRouteLocationEngine;
 import com.mapbox.services.android.navigation.v5.navigation.MapboxNavigation;
@@ -74,6 +74,7 @@ public class NavigationView extends CoordinatorLayout implements LifecycleObserv
   private BottomSheetBehavior summaryBehavior;
   private ImageButton cancelBtn;
   private RecenterButton recenterBtn;
+  private WayNameView wayNameView;
   private ImageButton routeOverviewBtn;
 
   private NavigationPresenter navigationPresenter;
@@ -140,8 +141,10 @@ public boolean onBackPressed() {
    */
   public void onSaveInstanceState(Bundle outState) {
     int bottomSheetBehaviorState = summaryBehavior == null ? INVALID_STATE : summaryBehavior.getState();
-    NavigationViewInstanceState navigationViewInstanceState = new NavigationViewInstanceState(bottomSheetBehaviorState,
-      recenterBtn.getVisibility(), instructionView.isShowingInstructionList());
+    boolean isWayNameVisible = wayNameView.getVisibility() == VISIBLE;
+    NavigationViewInstanceState navigationViewInstanceState = new NavigationViewInstanceState(
+      bottomSheetBehaviorState, recenterBtn.getVisibility(), instructionView.isShowingInstructionList(),
+      isWayNameVisible, wayNameView.retrieveWayNameText());
     String instanceKey = getContext().getString(R.string.navigation_view_instance_state);
     outState.putParcelable(instanceKey, navigationViewInstanceState);
     outState.putBoolean(getContext().getString(R.string.navigation_running), navigationViewModel.isRunning());
@@ -160,6 +163,8 @@ public void onRestoreInstanceState(Bundle savedInstanceState) {
     String instanceKey = getContext().getString(R.string.navigation_view_instance_state);
     NavigationViewInstanceState navigationViewInstanceState = savedInstanceState.getParcelable(instanceKey);
     recenterBtn.setVisibility(navigationViewInstanceState.getRecenterButtonVisibility());
+    wayNameView.setVisibility(navigationViewInstanceState.isWayNameVisible() ? VISIBLE : INVISIBLE);
+    wayNameView.updateWayNameText(navigationViewInstanceState.getWayNameText());
     resetBottomSheetState(navigationViewInstanceState.getBottomSheetBehaviorState());
     updateInstructionListState(navigationViewInstanceState.isInstructionViewVisible());
     mapInstanceState = savedInstanceState.getParcelable(MAP_INSTANCE_STATE_KEY);
@@ -171,7 +176,8 @@ public void onRestoreInstanceState(Bundle savedInstanceState) {
    * <p>
    * In an {@link Activity} this should be in {@link Activity#onDestroy()}.
    * <p>
-   * In a {@link android.app.Fragment}, this should be in {@link Fragment#onDestroyView()}.
+   * In a {@link android.support.v4.app.Fragment}, this should
+   * be in {@link android.support.v4.app.Fragment#onDestroyView()}.
    */
   public void onDestroy() {
     shutdown();
@@ -211,6 +217,7 @@ public void onStop() {
   @Override
   public void onMapReady(MapboxMap mapboxMap) {
     initializeNavigationMap(mapView, mapboxMap);
+    initializeWayNameListener();
     onNavigationReadyCallback.onNavigationReady(navigationViewModel.isRunning());
     isMapInitialized = true;
   }
@@ -268,17 +275,6 @@ public void drawRoute(DirectionsRoute directionsRoute) {
     }
   }
 
-  private void initializeNavigationMap(MapView mapView, MapboxMap map) {
-    navigationMap = new NavigationMapboxMap(mapView, map);
-    if (mapInstanceState != null) {
-      navigationMap.restoreFrom(mapInstanceState);
-      return;
-    }
-    if (initialMapCameraPosition != null) {
-      map.setCameraPosition(initialMapCameraPosition);
-    }
-  }
-
   @Override
   public void addMarker(Point position) {
     if (navigationMap != null) {
@@ -292,23 +288,33 @@ public void clearMarkers() {
     }
   }
 
+  /**
+   * Updates the text of the way name view below the
+   * navigation icon.
+   * <p>
+   * If you'd like to use this method without being overridden by the default way names
+   * values we provide, please disabled auto-query with
+   * {@link NavigationMapboxMap#updateWaynameQueryMap(boolean)}.
+   *
+   * @param wayname to update the view
+   */
   public void updateWaynameView(String wayname) {
-    if (navigationMap != null) {
-      navigationMap.updateWaynameView(wayname);
-    }
+    wayNameView.updateWayNameText(wayname);
   }
 
+  /**
+   * Updates the visibility of the way name view that is show below
+   * the navigation icon.
+   * <p>
+   * If you'd like to use this method without being overridden by the default visibility values
+   * values we provide, please disabled auto-query with
+   * {@link NavigationMapboxMap#updateWaynameQueryMap(boolean)}.
+   *
+   * @param isVisible true to show, false to hide
+   */
   @Override
   public void updateWaynameVisibility(boolean isVisible) {
-    if (navigationMap != null) {
-      navigationMap.updateWaynameVisibility(isVisible);
-    }
-  }
-
-  public void updateWaynameQueryMap(boolean isEnabled) {
-    if (navigationMap != null) {
-      navigationMap.updateWaynameQueryMap(isEnabled);
-    }
+    wayNameView.updateVisibility(isVisible);
   }
 
   @Override
@@ -497,6 +503,7 @@ private void bind() {
     summaryBottomSheet = findViewById(R.id.summaryBottomSheet);
     cancelBtn = findViewById(R.id.cancelBtn);
     recenterBtn = findViewById(R.id.recenterBtn);
+    wayNameView = findViewById(R.id.wayNameView);
     routeOverviewBtn = findViewById(R.id.routeOverviewBtn);
   }
 
@@ -525,6 +532,22 @@ private void initializeInstructionListListener() {
       navigationViewEventDispatcher));
   }
 
+  private void initializeNavigationMap(MapView mapView, MapboxMap map) {
+    navigationMap = new NavigationMapboxMap(mapView, map);
+    if (mapInstanceState != null) {
+      navigationMap.restoreFrom(mapInstanceState);
+      return;
+    }
+    if (initialMapCameraPosition != null) {
+      map.setCameraPosition(initialMapCameraPosition);
+    }
+  }
+
+  private void initializeWayNameListener() {
+    NavigationViewWayNameListener wayNameListener = new NavigationViewWayNameListener(navigationPresenter);
+    navigationMap.addOnWayNameChangedListener(wayNameListener);
+  }
+
   private void updateSavedInstanceStateMapStyle(@Nullable Bundle savedInstanceState) {
     if (savedInstanceState != null) {
       String mapStyleUrl = ThemeSwitcher.retrieveMapStyle(getContext());

libandroid-navigation-ui/src/main/java/com/mapbox/services/android/navigation/ui/v5/NavigationViewInstanceState.java

@@ -4,15 +4,20 @@
 import android.os.Parcelable;
 
 class NavigationViewInstanceState implements Parcelable {
+
   private int bottomSheetBehaviorState;
   private int recenterButtonVisibility;
   private boolean instructionViewVisible;
+  private boolean isWayNameVisible;
+  private String wayNameText;
 
   NavigationViewInstanceState(int bottomSheetBehaviorState, int recenterButtonVisibility,
-                              boolean instructionViewVisible) {
+                              boolean instructionViewVisible, boolean isWayNameVisible, String wayNameText) {
     this.bottomSheetBehaviorState = bottomSheetBehaviorState;
     this.recenterButtonVisibility = recenterButtonVisibility;
     this.instructionViewVisible = instructionViewVisible;
+    this.isWayNameVisible = isWayNameVisible;
+    this.wayNameText = wayNameText;
   }
 
   int getBottomSheetBehaviorState() {
@@ -27,35 +32,45 @@ boolean isInstructionViewVisible() {
     return instructionViewVisible;
   }
 
-  private NavigationViewInstanceState(Parcel parcel) {
-    bottomSheetBehaviorState = parcel.readInt();
-    recenterButtonVisibility = parcel.readInt();
-    instructionViewVisible = parcel.readByte() != 0x00;
+  boolean isWayNameVisible() {
+    return isWayNameVisible;
   }
 
-  @Override
-  public int describeContents() {
-    return 0;
+  String getWayNameText() {
+    return wayNameText;
+  }
+
+  private NavigationViewInstanceState(Parcel in) {
+    bottomSheetBehaviorState = in.readInt();
+    recenterButtonVisibility = in.readInt();
+    instructionViewVisible = in.readByte() != 0;
+    isWayNameVisible = in.readByte() != 0;
+    wayNameText = in.readString();
   }
 
   @Override
   public void writeToParcel(Parcel dest, int flags) {
     dest.writeInt(bottomSheetBehaviorState);
     dest.writeInt(recenterButtonVisibility);
-    dest.writeByte((byte) (instructionViewVisible ? 0x01 : 0x00));
+    dest.writeByte((byte) (instructionViewVisible ? 1 : 0));
+    dest.writeByte((byte) (isWayNameVisible ? 1 : 0));
+    dest.writeString(wayNameText);
   }
 
-  public static final Parcelable.Creator<NavigationViewInstanceState> CREATOR =
-    new Parcelable.Creator<NavigationViewInstanceState>() {
+  @Override
+  public int describeContents() {
+    return 0;
+  }
 
-      @Override
-      public NavigationViewInstanceState createFromParcel(Parcel source) {
-        return new NavigationViewInstanceState(source);
-      }
+  public static final Creator<NavigationViewInstanceState> CREATOR = new Creator<NavigationViewInstanceState>() {
+    @Override
+    public NavigationViewInstanceState createFromParcel(Parcel in) {
+      return new NavigationViewInstanceState(in);
+    }
 
-      @Override
-      public NavigationViewInstanceState[] newArray(int size) {
-        return new NavigationViewInstanceState[size];
-      }
-    };
+    @Override
+    public NavigationViewInstanceState[] newArray(int size) {
+      return new NavigationViewInstanceState[size];
+    }
+  };
 }

libandroid-navigation-ui/src/main/java/com/mapbox/services/android/navigation/ui/v5/NavigationViewWayNameListener.java

@@ -0,0 +1,19 @@
+package com.mapbox.services.android.navigation.ui.v5;
+
+import android.support.annotation.NonNull;
+
+import com.mapbox.services.android.navigation.ui.v5.map.OnWayNameChangedListener;
+
+class NavigationViewWayNameListener implements OnWayNameChangedListener {
+
+  private final NavigationPresenter presenter;
+
+  NavigationViewWayNameListener(NavigationPresenter presenter) {
+    this.presenter = presenter;
+  }
+
+  @Override
+  public void onWayNameChanged(@NonNull String wayName) {
+    presenter.onWayNameChanged(wayName);
+  }
+}

libandroid-navigation-ui/src/main/java/com/mapbox/services/android/navigation/ui/v5/map/MapLayerInteractor.java

@@ -1,7 +1,5 @@
 package com.mapbox.services.android.navigation.ui.v5.map;
 
-import android.graphics.Bitmap;
-
 import com.mapbox.mapboxsdk.maps.MapboxMap;
 import com.mapbox.mapboxsdk.style.layers.Layer;
 import com.mapbox.mapboxsdk.style.layers.LineLayer;
@@ -21,18 +19,10 @@ class MapLayerInteractor {
     this.mapboxMap = mapboxMap;
   }
 
-  void addLayer(Layer layer) {
-    mapboxMap.addLayer(layer);
-  }
-
   Layer retrieveLayerFromId(String layerId) {
     return mapboxMap.getLayerAs(layerId);
   }
 
-  void addLayerImage(String imageName, Bitmap image) {
-    mapboxMap.addImage(imageName, image);
-  }
-
   void updateLayerVisibility(boolean isVisible, String layerIdentifier) {
     // TODO add sourceIdentifier logic when https://github.com/mapbox/mapbox-gl-native/issues/12691 lands
     List<Layer> layers = mapboxMap.getLayers();