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

Replace UIActionSheet, UIAlertView with UIAlertController #8371

Closed
1ec5 opened this issue Mar 13, 2017 · 1 comment
Closed

Replace UIActionSheet, UIAlertView with UIAlertController #8371

1ec5 opened this issue Mar 13, 2017 · 1 comment
Labels
iOS Mapbox Maps SDK for iOS refactor
Milestone
ios-v3.5.1

Comments

@1ec5
Copy link
Contributor

1ec5 commented Mar 13, 2017 •
edited
Loading

Now that we’ve dropped iOS 7 support, we can replace usage of the deprecated UIActionSheet and UIAlertView classes with UIAlertController. (Removing UIActionSheet and UIAlertView usage would also become important if we were to ever support tvOS, since neither class is available on that platform.) It’s going to be a bit strange that a view presents a view controller, but hopefully it won’t be much different than what we’re already doing in these situations.

/cc @friedbunny @frederoni @incanus
@1ec5 1ec5 added iOS Mapbox Maps SDK for iOS refactor labels Mar 13, 2017
@1ec5 1ec5 changed the title Replace UIActionSheet with UIAlertController Replace UIActionSheet, UIAlertView with UIAlertController Mar 13, 2017
@1ec5 1ec5 mentioned this issue Mar 13, 2017
Merged
@1ec5 1ec5 added this to the ios-v3.5.1 milestone Mar 13, 2017
@1ec5
Copy link
Contributor Author

1ec5 commented Apr 2, 2017

Fixed in #8373 on the release-ios-v3.5.0-android-v5.0.0 branch.

@1ec5 1ec5 closed this as completed Apr 2, 2017
@1ec5 1ec5 mentioned this issue Apr 25, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
iOS Mapbox Maps SDK for iOS refactor
Projects
None yet
Milestone
ios-v3.5.1
Development

No branches or pull requests
1 participant
@1ec5