Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Waypoints covered by maneuver arrow #2865

Closed
kmadsen opened this issue Apr 29, 2020 · 8 comments
Closed

Waypoints covered by maneuver arrow #2865

kmadsen opened this issue Apr 29, 2020 · 8 comments
Labels
bug Defect to be fixed.

Comments

@kmadsen
Copy link
Contributor

kmadsen commented Apr 29, 2020

Mapbox Navigation SDK version: 1.0.0

Steps to trigger behavior

  1. Open the MultipleStopsActivity example
  2. Long press to create two or more stops
  3. Start navigation

Expected behavior

The maneuver arrow should not cover the stop

Actual behavior

screen

@kmadsen kmadsen added the bug Defect to be fixed. label Apr 29, 2020
@kmadsen kmadsen added this to the v1.0.0 milestone Apr 29, 2020
@Guardiola31337
Copy link
Contributor

@kmadsen
This is by design and the same thing happens in the legacy 👀

upcoming_maneuver_arrow_legacy

Look at #934 and #1053

The idea behind placing the upcoming maneuver arrow at the top was to highlight it as it's an element that prevails over other annotation giving the most context to drivers but would love to hear from the design team @designevokes @map-angela @noracalabrese @d-prukop @avi-c and see what do they think these days 😅

@kmadsen
Copy link
Contributor Author

kmadsen commented Apr 29, 2020

Ok weird, well if it's by design should I close?

For what it's worth, I think it would look better if the arrow pointed to the stop - rather than lay on top of it.

@Guardiola31337 Guardiola31337 removed this from the v1.0.0 milestone May 12, 2020
@kmadsen kmadsen closed this as completed May 15, 2020
@Guardiola31337
Copy link
Contributor

From #1053 (comment)

This way the upcoming maneuver arrow will be drawn above shields (as it was) but underneath annotations

Re-opening as OP and #2872 are still issues.

cc @cafesilencio @kmadsen @LukasPaczos

@Guardiola31337
Copy link
Contributor

Any feedback here @noracalabrese?

@Guardiola31337
Copy link
Contributor

Are we good to close here now @cafesilencio? Could you check when you have a chance and close if so?

@noracalabrese
Copy link

noracalabrese commented Aug 27, 2020 via email

@map-angela
Copy link

Yes, I created this https://github.com/mapbox/UI-UX-SDK/ to track the issues. Let me know if this works.

@cafesilencio
Copy link
Contributor

As Pablo mentioned this is currently working as designed so I"m closing this ticket until such time it's decided the behavior should change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Defect to be fixed.
Projects
None yet
Development

No branches or pull requests

5 participants