Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Map feedback view controller #6001

Closed
1ec5 opened this issue Aug 13, 2016 · 2 comments
Closed

Map feedback view controller #6001

1ec5 opened this issue Aug 13, 2016 · 2 comments
Labels
archived Archived because of inactivity feature iOS Mapbox Maps SDK for iOS MapKit parity For feature parity with MapKit on iOS or macOS

Comments

@1ec5
Copy link
Contributor

1ec5 commented Aug 13, 2016

The “Improve This Map” item in the attribution action sheet opens the Map Feedback tool in the default Web browser. This Web application is difficult to use on a mobile device. Meanwhile, the SDK has a lot more context about the current view that could lead to more actionable feedback.

The SDK should vend a view controller that allows the user to fill out all the same information that the existing Map Feedback tool collects, but using native controls. Additionally, it could allow the user to select the problematic map feature from a map view using feature querying and send along the feature’s OpenStreetMap ID. Upon sending the feedback, the user would automatically be taken back to the map. This would be an improvement over the current workflow, which requires the user to manually return to the Mapbox-powered application.

This functionality would have to live in a view controller because MGLMapView could be arbitrarily small and incapable of hosting a complex form layout. We’d expose a method in MGLMapViewDelegate to give the map view’s host view controller the opportunity to choose between segueing to the feedback view controller or opening the existing tool in the default browser (useful for maps contained in embedded view controllers or other non-obvious configurations).

I’m keeping this ticket limited to iOS for the moment, because the view controller consideration is an important one on iOS. (On macOS, for instance, it’d trivial for a view to open a sheet or modal window large enough to host the entire feedback experience.)

/cc @mapbox/mobile

@zugaldia
Copy link
Member

The Android equivalent is tracked in #6013.

@1ec5 1ec5 added the MapKit parity For feature parity with MapKit on iOS or macOS label Apr 25, 2017
@stale stale bot added the archived Archived because of inactivity label Nov 8, 2018
@stale
Copy link

stale bot commented Nov 27, 2018

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Nov 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
archived Archived because of inactivity feature iOS Mapbox Maps SDK for iOS MapKit parity For feature parity with MapKit on iOS or macOS
Projects
None yet
Development

No branches or pull requests

2 participants