Draw slight left/right lanes

mapbox · Apr 2, 2018 · 3e3f710 · 3e3f710
1 parent 5829a14
commit 3e3f710
Show file tree

Hide file tree

Showing 3 changed files with 54 additions and 10 deletions.
diff --git a/MapboxNavigation/LaneView.swift b/MapboxNavigation/LaneView.swift
@@ -39,26 +39,42 @@ open class LaneView: UIView {
             if lane.indications.isSuperset(of: [.straightAhead, .sharpRight]) || lane.indications.isSuperset(of: [.straightAhead, .right]) || lane.indications.isSuperset(of: [.straightAhead, .slightRight]) {
                 flipLane = false
                 if !isValid {
-                    LanesStyleKit.drawLane_straight_right(primaryColor: appropriatePrimaryColor)
+                    if lane.indications == .slightRight {
+                        LanesStyleKit.drawLane_slight_right(primaryColor: appropriatePrimaryColor)
+                    } else {
+                        LanesStyleKit.drawLane_straight_right(primaryColor: appropriatePrimaryColor)
+                    }
                     alpha = invalidAlpha
                 } else if maneuverDirection == .straightAhead {
                     LanesStyleKit.drawLane_straight_only(primaryColor: appropriatePrimaryColor, secondaryColor: secondaryColor)
                 } else if maneuverDirection == .sharpLeft || maneuverDirection == .left || maneuverDirection == .slightLeft {
-                    LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
+                    if lane.indications == .slightLeft {
+                        LanesStyleKit.drawLane_slight_right(primaryColor: appropriatePrimaryColor)
+                    } else {
+                        LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
+                    }
                     flipLane = true
                 } else {
                     LanesStyleKit.drawLane_right_only(primaryColor: appropriatePrimaryColor, secondaryColor: secondaryColor)
                 }
             } else if lane.indications.isSuperset(of: [.straightAhead, .sharpLeft]) || lane.indications.isSuperset(of: [.straightAhead, .left]) || lane.indications.isSuperset(of: [.straightAhead, .slightLeft]) {
                 flipLane = true
                 if !isValid {
-                    LanesStyleKit.drawLane_straight_right(primaryColor: appropriatePrimaryColor)
+                    if lane.indications == .slightLeft {
+                        LanesStyleKit.drawLane_slight_right(primaryColor: appropriatePrimaryColor)
+                    } else {
+                        LanesStyleKit.drawLane_straight_right(primaryColor: appropriatePrimaryColor)
+                    }
+
                     alpha = invalidAlpha
                 } else if maneuverDirection == .straightAhead {
                     LanesStyleKit.drawLane_straight_only(primaryColor: appropriatePrimaryColor, secondaryColor: secondaryColor)
-                } else if maneuverDirection == .sharpRight || maneuverDirection == .right || maneuverDirection == .slightRight {
+                } else if maneuverDirection == .sharpRight || maneuverDirection == .right {
                     LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
                     flipLane = false
+                } else if maneuverDirection == .slightRight {
+                    LanesStyleKit.drawLane_slight_right(primaryColor: appropriatePrimaryColor)
+                    flipLane = false
                 } else {
                     LanesStyleKit.drawLane_right_only(primaryColor: appropriatePrimaryColor, secondaryColor: secondaryColor)
                 }
@@ -67,20 +83,31 @@ open class LaneView: UIView {
                 // Account for a configuation where there is no straight lane
                 // but there are at least 2 indications.
                 // In this situation, just draw a left/right arrow
-                if maneuverDirection == .sharpRight || maneuverDirection == .right || maneuverDirection == .slightRight {
+                if maneuverDirection == .sharpRight || maneuverDirection == .right {
                     LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
                     flipLane = false
+                } else if maneuverDirection == .slightRight {
+                    LanesStyleKit.drawLane_slight_right(primaryColor: appropriatePrimaryColor)
+                    flipLane = false
                 } else {
                     LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
                     flipLane = true
                 }
                 alpha = isValid ? 1 : invalidAlpha
             } else if lane.indications.isSuperset(of: [.sharpRight]) || lane.indications.isSuperset(of: [.right]) || lane.indications.isSuperset(of: [.slightRight]) {
-                LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
+                if lane.indications == .slightRight {
+                    LanesStyleKit.drawLane_slight_right(primaryColor: appropriatePrimaryColor)
+                } else {
+                    LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
+                }
                 flipLane = false
                 alpha = isValid ? 1 : invalidAlpha
             } else if lane.indications.isSuperset(of: [.sharpLeft]) || lane.indications.isSuperset(of: [.left]) || lane.indications.isSuperset(of: [.slightLeft]) {
-                LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
+                if lane.indications == .slightLeft {
+                    LanesStyleKit.drawLane_slight_right(primaryColor: appropriatePrimaryColor)
+                } else {
+                    LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
+                }
                 flipLane = true
                 alpha = isValid ? 1 : invalidAlpha
             } else if lane.indications.isSuperset(of: [.straightAhead]) {
@@ -95,10 +122,18 @@ open class LaneView: UIView {
                 // If the lane indication is `none` and the maneuver modifier has a turn in it,
                 // show the turn in the lane image.
                 if maneuverDirection == .sharpRight || maneuverDirection == .right || maneuverDirection == .slightRight {
-                    LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
+                    if maneuverDirection == .slightRight {
+                        LanesStyleKit.drawLane_slight_right(primaryColor: appropriatePrimaryColor)
+                    } else {
+                        LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
+                    }
                     flipLane = false
                 } else if maneuverDirection == .sharpLeft || maneuverDirection == .left || maneuverDirection == .slightLeft {
-                    LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
+                    if maneuverDirection == .slightLeft {
+                        LanesStyleKit.drawLane_slight_right(primaryColor: appropriatePrimaryColor)
+                    } else {
+                        LanesStyleKit.drawLane_right_h(primaryColor: appropriatePrimaryColor)
+                    }
                     flipLane = true
                 } else {
                     LanesStyleKit.drawLane_straight(primaryColor: appropriatePrimaryColor)

diff --git a/MapboxNavigationTests/LaneTests.swift b/MapboxNavigationTests/LaneTests.swift
@@ -1,6 +1,6 @@
 import XCTest
 import FBSnapshotTestCase
-import MapboxDirections
+@testable import MapboxDirections
 @testable import MapboxNavigation
 @testable import MapboxCoreNavigation
 
@@ -43,4 +43,13 @@ class LaneTests: FBSnapshotTestCase {
     func testRightNone() {
         assertLanes(step: steps[1])
     }
+
+    func testSlightRight() {
+        let view = LaneView(frame: CGRect(origin: .zero, size: CGSize(width: 30, height: 30)))
+        view.backgroundColor = .white
+        view.lane = Lane(indications: [.slightRight])
+        view.maneuverDirection = .slightRight
+        view.isValid = true
+        FBSnapshotVerifyView(view)
+    }
 }
diff --git a/...Images_64/MapboxNavigationTests.LaneTests/testSlightRight_iPhone11_2_0x0@3x.png b/...Images_64/MapboxNavigationTests.LaneTests/testSlightRight_iPhone11_2_0x0@3x.png