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

Directions List Dropdown #415

Merged
merged 10 commits into from
Oct 27, 2017
Merged

Directions List Dropdown #415

merged 10 commits into from
Oct 27, 2017

Conversation

danesfeder
Copy link
Contributor

@danesfeder danesfeder commented Oct 24, 2017

  • Clicking the top banner will reveal the list of directions rather than an option from the bottomsheet

TODO:

  • Mirror banner view for each ViewHolder
  • Handle landscape
  • Handle back press
  • Clean up / docs

@danesfeder danesfeder added the feature New feature request. label Oct 24, 2017
@danesfeder danesfeder self-assigned this Oct 24, 2017
@danesfeder
Copy link
Contributor Author

@ericrwolfe w/o updated banner views

ezgif com-video-to-gif

@danesfeder danesfeder added this to the v0.7.0 milestone Oct 24, 2017
@cammace
Copy link

cammace commented Oct 24, 2017

Love it, for the direction item line breaker next to the 0.2mi, can/should we make this a lighter gray? Has a designer looked over this or is this entire directions sheet designed by you @danesfeder?

screen shot 2017-10-24 at 6 43 37 pm

@Danny-James
Copy link

@danesfeder will this also include my request? #403

@danesfeder
Copy link
Contributor Author

@Danny-James yeah we will handle back press in this PR (see TODO)

@danesfeder danesfeder merged commit 884af21 into master Oct 27, 2017
@danesfeder danesfeder deleted the dan-banner-dropdown branch October 27, 2017 22:56
Grsmto pushed a commit to Grsmto/mapbox-navigation-android that referenced this pull request Oct 30, 2017
…android into fix-build

* 'master' of https://github.com/mapbox/mapbox-navigation-android:
  Fixes issue with bearing values not matching the amount of coordinates passed with the new route request (mapbox#435)
  Feedback for Drop-In UI (mapbox#383)
  Directions List Dropdown (mapbox#415)
  fixed location object in snapToRoute (mapbox#434)
  add location engine class name as parameter of the different nav events builder methods (mapbox#401)
  Dynamic reroute tolerance method (mapbox#428)
  Replace maneuver arrows and lanes (mapbox#362)

# Conflicts:
#	libandroid-navigation/src/main/java/com/mapbox/services/android/navigation/v5/navigation/NavigationMetricsWrapper.java
@danesfeder danesfeder mentioned this pull request Nov 13, 2017
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants