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

Simplified by using a table view controller and reloading on app resume. #35

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

malhal
Copy link

@malhal malhal commented Feb 26, 2016

By using a table view controller the keyboard is handled automatically which significantly simplifies the class. The settings table was removed and all keyboard notifications removed. Instead it uses the built-in table.

Now it handles the case of leaving the app and changing a setting in preferences, when resuming the app the settings will reload to show the change.

By using a table view controller the keyboard is handled automatically
which significantly simplifies the class. The settings table was
removed and all keyboard notifications removed. Instead it uses the
built-in table.

Now it handles the case of leaving the app and changing a setting in
preferences, when resuming the app the settings will reload to show the
change.
By using a table view controller the keyboard is handled automatically
which significantly simplifies the class. The settings table was
removed and all keyboard notifications removed. Instead it uses the
built-in table.

Now it handles the case of leaving the app and changing a setting in
preferences, when resuming the app the settings will reload to show the
change.
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