Skip to content

Conversation

@abhishek1508
Copy link
Contributor

Description

When the user clicks on MapboxSpeedInfoView, the changes will notify the developer about it using NavigationViewListener#onSpeedInfoClicked(speedInfo: SpeedInfoValue?)

@codecov
Copy link

codecov bot commented Dec 21, 2022

Codecov Report

Merging #6770 (9814236) into main (b97c2f0) will decrease coverage by 0.01%.
The diff coverage is 73.68%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #6770      +/-   ##
============================================
- Coverage     72.51%   72.50%   -0.02%     
- Complexity     5541     5546       +5     
============================================
  Files           777      779       +2     
  Lines         30039    30060      +21     
  Branches       3547     3547              
============================================
+ Hits          21783    21794      +11     
- Misses         6832     6842      +10     
  Partials       1424     1424              
Impacted Files Coverage Δ
...avigation/dropin/infopanel/InfoPanelCoordinator.kt 70.90% <0.00%> (ø)
...ropin/speedlimit/SpeedInfoComponentContractImpl.kt 0.00% <0.00%> (ø)
...ation/ui/speedlimit/internal/SpeedInfoComponent.kt 75.00% <37.50%> (-17.31%) ⬇️
...ation/dropin/map/RouteLineComponentContractImpl.kt 40.00% <50.00%> (ø)
...avigation/dropin/speedlimit/SpeedInfoViewBinder.kt 94.11% <50.00%> (-5.89%) ⬇️
...vigation/ui/speedlimit/view/MapboxSpeedInfoView.kt 64.55% <50.00%> (-0.13%) ⬇️
...java/com/mapbox/navigation/dropin/ClickBehavior.kt 100.00% <100.00%> (ø)
.../navigation/dropin/infopanel/InfoPanelComponent.kt 92.59% <100.00%> (ø)
...box/navigation/dropin/maneuver/ManeuverBehavior.kt 100.00% <100.00%> (ø)
...n/dropin/maneuver/ManeuverComponentContractImpl.kt 100.00% <100.00%> (ø)
... and 8 more

@dzinad
Copy link
Contributor

dzinad commented Dec 22, 2022

I'm not sure why changelog-verification task thinks it's not a PR but you didn't update the changelog.

@abhishek1508 abhishek1508 force-pushed the ak-NAVAND-#1023-add-click-listener branch 2 times, most recently from bf07eed to f5a1adb Compare January 2, 2023 21:23
@abhishek1508 abhishek1508 force-pushed the ak-NAVAND-#1023-add-click-listener branch from f5a1adb to 9814236 Compare January 2, 2023 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants