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

[TIDOC-414] [TIDOC-120] [TIDOC-245] [TIDOC-188] APIDoc: UI.ScrollView rewrite #1515

Merged
merged 6 commits into from
Mar 7, 2012

Conversation

- name: scale
summary: New scaling factor as a float.
type: Number

- name: x
Copy link
Contributor

Choose a reason for hiding this comment

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

This property is not on the scale event in iOS.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in next update

@sptramer
Copy link
Contributor

Docs reviewed. Please address comments.

type: Number

- name: setContentOffset
Copy link
Contributor

Choose a reason for hiding this comment

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

Android does not have setContentOffset. However, there is a scrollToBottom() method.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in next update.

@ayeung
Copy link
Contributor

ayeung commented Feb 27, 2012

Code reviewed, left a couple of comments.

@hal-gh
Copy link
Contributor Author

hal-gh commented Mar 2, 2012

Updated. Please review.

- name: y
summary: Y coordinate of the event from the `source` view's coordinate system.
- name: x
summary: X coordinate of the event from the `source` view's coordinate system.
Copy link
Contributor

Choose a reason for hiding this comment

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

x & y should not be removed. This still exists in android.

Copy link
Contributor

Choose a reason for hiding this comment

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

ignore this comment, didn't see it below.

@ayeung
Copy link
Contributor

ayeung commented Mar 5, 2012

Code reviewed and functionally tested. Request Accepted

summary: disable view bouncing.
summary: Determines whether all scroll bounce of the scrollable region is enabled.
description: |
Set to `true` to disable vertical bounce.
Copy link
Contributor

Choose a reason for hiding this comment

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

Also disables horizontal bounce.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Addressed in next update.

@sptramer
Copy link
Contributor

sptramer commented Mar 5, 2012

Docs reviewed; please address comments. Looks like I was wrong about some of the initial information for bounce parameters.

@hal-gh
Copy link
Contributor Author

hal-gh commented Mar 6, 2012

Updated. Please review.


#### Scrolling and Dragging

This view supports two types of gesturedrag is when your finger is still on the scroll view and you are physically dragging it versus the scrolling which is due to momentum (you scrolled the view with your finger, lifted your finger and the view cotinues to move)
Copy link
Contributor

Choose a reason for hiding this comment

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

Space/punctuation needed in gesturedrag

Copy link
Contributor

Choose a reason for hiding this comment

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

May want a comma in here; dragging it, versus the

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, Stephen, I didn't mean to leave this in there. I've explained this in the events.
Fixed in next update.

@sptramer
Copy link
Contributor

sptramer commented Mar 6, 2012

Docs reviewed, additional comments

@hal-gh
Copy link
Contributor Author

hal-gh commented Mar 6, 2012

Updated. Please review.

@sptramer
Copy link
Contributor

sptramer commented Mar 6, 2012

Docs reviewed, APPROVED.

@ayeung
Copy link
Contributor

ayeung commented Mar 7, 2012

Code reviewed and functionally tested. Request Accepted

ayeung pushed a commit that referenced this pull request Mar 7, 2012
[TIDOC-414] [TIDOC-120] [TIDOC-245] [TIDOC-188] APIDoc: UI.ScrollView rewrite
@ayeung ayeung merged commit a9ddade into tidev:master Mar 7, 2012
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