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

Implements SLKeyboard protocol for custom keyboards #105

Merged
merged 2 commits into from
Feb 11, 2014

Conversation

j-mutter
Copy link
Contributor

@j-mutter j-mutter commented Dec 3, 2013

With iOS 7 SDK, UIKit now exposes UIInputView to allow custom inputs/keyboards to be created.

This PR provides an SLKeyboard protocol for such views, refactors the original SLKeyboard class to implement the protocol methods, and updates SLTextField and SLTextView to use different keyboards as needed.

@wearhere
Copy link
Contributor

This is neat @j-mutter! Thanks and sorry for the delay in reviewing--I'm slowly getting back into Subliminal after a hefty release of Inkling right before the holidays. I just fixed some things in the build process--would you mind rebasing on top of master so that the tests will run against this before I merge?

To tap individual keys on the keyboard, use SLKeyboardKey.
*/
@interface SLKeyboard : SLStaticElement
@protocol SLKeyboard <NSObject>
Copy link
Contributor

Choose a reason for hiding this comment

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

This is an elegant way to support different keyboards. Before your request I hadn't heard of UIInputView--would you mind describing it/the rationale here in documentation for the SLKeyboard protocol?

@j-mutter
Copy link
Contributor Author

@wearhere Done and done :) Sorry for the delay

@wearhere
Copy link
Contributor

Looks like everything passed except for some documentation errors--missing parameters on the new methods added to SLTextField and SLTextView. Can you fix @j-mutter?

@j-mutter
Copy link
Contributor Author

Travis is 💚 now :)

@wearhere
Copy link
Contributor

👍 Thanks again!

wearhere added a commit that referenced this pull request Feb 11, 2014
Implements SLKeyboard protocol for custom keyboards
@wearhere wearhere merged commit 1b8a756 into inkling:master Feb 11, 2014
@j-mutter j-mutter deleted the feature/keyboards branch March 5, 2014 21:55
@j-mutter j-mutter restored the feature/keyboards branch March 5, 2014 21:55
@j-mutter j-mutter deleted the feature/keyboards branch April 21, 2014 14:02
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.

2 participants