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

[TIMOB-4272] & [TIMOB-4363] -- TableView doc fixes #926

Merged
merged 5 commits into from Dec 13, 2011

Conversation

arthurevans
Copy link
Contributor

No description provided.

Arthur Evans added 3 commits December 12, 2011 10:51
TIMOB-4272 &
TIMOB-4363

Changes:

- Added missing argument to updateRow
- Fixed summaries for insertRowBefore and insertRowAfter
- Took some liberties by inventing a new pseudotype for animation properties.
- Added a pseudotype for indexEntries (for the iOS-only index property) as well.
- Removed unsupported events.
- Added the missing click event.
- Marked index, style, selectionStyle
- Marked selectRow, deselectRow as unsupported on Android
- Edited for style & content & expanded some descriptions.

Whew!
Updated to address review comments.
Updated to address review comments.
Added some missing types to the event properties.
@billdawson
Copy link
Contributor

Code review and functional test (validate and a docgen spot check) accepted.

type: Number
- name: scrollable
summary: true (default) if tableview can be scrolled
summary: If true (default) if tableview can be scrolled
Copy link
Contributor

Choose a reason for hiding this comment

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

If true if tableview can be scrolled?

@WhichKatieDid
Copy link
Contributor

Aha. Didn't see the fix. Now it looks good. ACCEPTED.

@arthurevans
Copy link
Contributor Author

D'oh. Fixed typo. Um, for some reason github shows my fix for this coming in 17 minutes before Blain pointed it out. Rest assured this is due to temporary flux in the time-space matrix.

@billdawson
Copy link
Contributor

Code re-review and re-test accepted

billdawson added a commit that referenced this pull request Dec 13, 2011
[TIMOB-4272] & [TIMOB-4363] -- TableView doc fixes
@billdawson billdawson merged commit c4343c2 into tidev:1_8_X Dec 13, 2011
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.

None yet

3 participants