Skip to content

Document guidance cards, junction views#2627

Merged
1ec5 merged 1 commit into
masterfrom
1ec5-cards-docs-2519
Sep 15, 2020
Merged

Document guidance cards, junction views#2627
1ec5 merged 1 commit into
masterfrom
1ec5-cards-docs-2519

Conversation

@1ec5
Copy link
Copy Markdown
Contributor

@1ec5 1ec5 commented Sep 14, 2020

Added class documentation comments for the minimum set of types necessary to consider the guidance cards feature from #2149 and #2296 to be publicly available. With these changes, we can finally add a blurb to the changelog announcing the introduction of this feature, even though it has been in the codebase for quite some time. This PR also adds some documentation about junction views for styling purposes.

Guidance cards, represented by the InstructionsCardViewController class, is an alternative presentation of maneuver instructions that is intended to be better visually integrated with the map than the traditional presentation of banners flush with the edges of the screen, represented by the default TopBannerViewController class. To enable guidance cards, an application would pass an instance of InstructionsCardViewController into NavigationOptions(styles:navigationService:voiceController:topBanner:bottomBanner:) when creating the NavigationViewController. Using guidance cards in a custom turn-by-turn navigation UI requires some additional setup and delegate method implementations.

Fixes most of #2519.

/cc @mapbox/navigation-ios @avi-c @d-prukop

Comment thread MapboxNavigation/TopBannerViewController.swift
@1ec5 1ec5 force-pushed the 1ec5-cards-docs-2519 branch from d6e3389 to 041b620 Compare September 15, 2020 01:57
@1ec5 1ec5 merged commit db74f46 into master Sep 15, 2020
@1ec5 1ec5 deleted the 1ec5-cards-docs-2519 branch September 15, 2020 02:13
@1ec5
Copy link
Copy Markdown
Contributor Author

1ec5 commented Sep 17, 2020

Cherry-picked into the release-v1.0 branch as db74f46.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants