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

Add support for UISwipeActionsConfiguration #167

Merged
merged 3 commits into from
Jul 3, 2019

Conversation

ronaldsmartin
Copy link
Contributor

Changes in this pull request

This adds tableview support for the UISwipeActionsConfiguration API, which was introduced in iOS 11. This should be used to replace usage of the UITableViewRowAction API, which will be deprecated in iOS 13.

Issue fixed: none

Checklist

  • All tests pass. Demo project builds and runs.
  • I added tests, an experiment, or detailed why my change isn't tested.
  • I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
  • I have reviewed the contributing guide

@ronaldsmartin ronaldsmartin added this to the Next milestone Jun 26, 2019
@ghost ghost assigned jessesquires Jun 26, 2019
Copy link
Contributor

@jessesquires jessesquires left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We've already dropped support for iOS 10, so I'd say we just convert over to the new API and remove the availables

@ronaldsmartin
Copy link
Contributor Author

ronaldsmartin commented Jun 26, 2019 via email

@ronaldsmartin
Copy link
Contributor Author

I also need to add default implementations so that these props are all Optional!

Copy link
Contributor

@jessesquires jessesquires left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 💯

@ronaldsmartin ronaldsmartin merged commit 73d0221 into master Jul 3, 2019
@ronaldsmartin ronaldsmartin deleted the uiswipeactionsconfiguration-support branch July 3, 2019 20:05
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