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
Hide the remaining steps when user is on final step #3729
Conversation
No breaking changes detected in MapboxCoreNavigation |
No breaking changes detected in MapboxNavigation |
44189de
to
60fa266
Compare
No breaking changes detected in MapboxCoreNavigation |
No breaking changes detected in MapboxNavigation |
60fa266
to
14d96e9
Compare
No breaking changes detected in MapboxCoreNavigation |
Breaking Changes in MapboxNavigationBreaking API Changes
|
Type of change: | key.doc.result_discussion |
---|---|
From: | [["Para": optionally, a customized NSAttributedString that will be presented instead of the default, or if nil, the default behavior will be used.]] |
To: | [["Para": Optionally, a customized NSAttributedString that will be presented instead of the default, or if nil, the default behavior will be used.]] |
primaryLabel(_:willPresent:as:)
in InstructionsCardContainerViewDelegate
Type of change: | key.doc.result_discussion |
---|---|
From: | [["Para": optionally, a customized NSAttributedString that will be presented instead of the default, or if nil, the default behavior will be used.]] |
To: | [["Para": Optionally, a customized NSAttributedString that will be presented instead of the default, or if nil, the default behavior will be used.]] |
LanesView
- removed method:
update(for:)
inLanesView
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great to add a CHANGELOG line
14d96e9
to
df9b28b
Compare
No breaking changes detected in MapboxCoreNavigation |
No breaking changes detected in MapboxNavigation |
No breaking changes detected in MapboxCoreNavigation |
No breaking changes detected in MapboxNavigation |
5e459f0
to
76a3fd7
Compare
No breaking changes detected in MapboxCoreNavigation |
No breaking changes detected in MapboxNavigation |
Description
This Pr is to fix #2015 to hide the remaining steps when user is on final step.
Implementation
When user arrives on final step,
TopBannerViewController
would dismiss thestepsViewController
and change theinstructionsBannerView.showStepIndicator
tofalse
. And the step indicator of the instruction banner would be hidden, the swiping and tapping gestures wouldn't work.And if the mobile is in the previewing mode, the previewing process would stop. If the mobile is displaying the remaining steps in pull-down menu, the pull-down table would be dismissed.
Screenshots or Gifs
When in final step, the indicator view would be hidden, swiping and taping gestures wouldn't trigger the remaining steps displaying.