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

[SlackTextViewController] Updated to (almost) v1.9.6 to fix iOS 11 bugs #246

Merged
merged 2 commits into from
Oct 19, 2017

Conversation

mattleibow
Copy link
Collaborator

@mattleibow mattleibow commented Oct 19, 2017

This addresses #244

This build works off a specific commit due to the version not being out yet, and this is a critical fix.

@softwaremills
Copy link

Thank you very much, sir!

@mattleibow
Copy link
Collaborator Author

Because the native version is not yet released, this will be a preview release. As soon as the official version goes out, we will release the new version with any additional fixes/features.

@bholmes bholmes merged commit 22b1edb into master Oct 19, 2017
@mattleibow mattleibow deleted the updates/slack branch October 19, 2017 21:07
@softwaremills
Copy link

That seems eminently reasonable.

@bholmes
Copy link
Contributor

bholmes commented Oct 20, 2017

fyi... https://www.nuget.org/packages/Xamarin.SlackTextViewController/1.9.6-preview

@JoeyBodnar
Copy link

does this fix the inserting/deleting cells issue?

@softwaremills
Copy link

I'm not sure if it fixes that, but I have tested it and it does fix the problem where on iOS 11 the spacing at the bottom was all wrong and the text field didn't register taps. Thank you very much @mattleibow and @bholmes !

@mattleibow
Copy link
Collaborator Author

Great to hear that it fixes the iOS 11 issue. Any other issues need to be logged against the native library as this is just the binding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants