This repository has been archived by the owner on Sep 18, 2021. It is now read-only.
Adding header support for TUITableView sections #11
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 5 commits
July 7, 2011 16:47
…elegate) instead of just changing the highlighted state of the selected/deselected cell via: - (void)selectRowAtIndexPath:(TUIFastIndexPath *)indexPath animated:(BOOL)animated scrollPosition:(TUITableViewScrollPosition)scrollPosition; - (void)deselectRowAtIndexPath:(TUIFastIndexPath *)indexPath animated:(BOOL)animated; This also causes selection changes via arrow keys to notify the delegate of the selection change.
- Adding sections to the example project
- Adding bounds checking in table section header index enumeration
|
awesome! I'm going to bang on this for a bit, then merge. |
|
merged! (with some formatting tweaks & I force the header view to autoresize width-wise) |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
UITableView will display a header view for sections when its data source returns a view for the new delegate method tableView:headerViewForSection:. The header view is a plain old TUIView and is not reusable (stored in section info). The height of the header view is not adjusted by the table view and its height determines the height of the section header.
I've also changed the way did select delegate methods are triggered (they're taken out of the cell, since they really belong to the table) so that changing the selection either by clicking or via the arrow keys (or some other programmatic method) now notifies the delegate. Previously, changing the selection by arrow keys did not actually notify the delegate the selection had changed.
I've also added .animateSelectionChanges to TUITableView which can be set FALSE to cause the table not to animation selection changes. There's probably a more elegant way to do this...
The example project has been updated to reflect these changes.