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

Adding scrollWithoutAnimationTo method for ScrollViews #509

Closed
wants to merge 1 commit into from
Closed

Adding scrollWithoutAnimationTo method for ScrollViews #509

wants to merge 1 commit into from

Conversation

ccheever
Copy link
Contributor

Implementing the consensus approach from the comments on
this PR:
#486

We use a boolean flag in the Obj-C code to determine whether
to animate or not, and then provide two public JS functions
that call the Obj-C with or without the flag.

Implementing the consensus approach from the comments on
this PR:
#486

We use a boolean flag in the Obj-C code to determine whether
to animate or not, and then provide two public JS functions
that call the Obj-C with or without the flag.
@sahrens
Copy link
Contributor

sahrens commented Apr 1, 2015

Merged internally and synced.

@sahrens sahrens closed this Apr 1, 2015
vjeux pushed a commit to vjeux/react-native that referenced this pull request Apr 13, 2015
Summary:
Implementing the consensus approach from the comments on
this PR:
facebook#486

We use a boolean flag in the Obj-C code to determine whether
to animate or not, and then provide two public JS functions
that call the Obj-C with or without the flag.
Closes facebook#509
Github Author: Charlie Cheever <ccheever@gmail.com>

Test Plan: Imported from GitHub, without a `Test Plan:` line.
vjeux pushed a commit to vjeux/react-native that referenced this pull request Apr 14, 2015
Summary:
Implementing the consensus approach from the comments on
this PR:
facebook#486

We use a boolean flag in the Obj-C code to determine whether
to animate or not, and then provide two public JS functions
that call the Obj-C with or without the flag.
Closes facebook#509
Github Author: Charlie Cheever <ccheever@gmail.com>

Test Plan: Imported from GitHub, without a `Test Plan:` line.
vjeux pushed a commit to vjeux/react-native that referenced this pull request Apr 15, 2015
Summary:
Implementing the consensus approach from the comments on
this PR:
facebook#486

We use a boolean flag in the Obj-C code to determine whether
to animate or not, and then provide two public JS functions
that call the Obj-C with or without the flag.
Closes facebook#509
Github Author: Charlie Cheever <ccheever@gmail.com>

Test Plan: Imported from GitHub, without a `Test Plan:` line.
jfrolich pushed a commit to jfrolich/react-native that referenced this pull request Apr 22, 2020
I deleted all the style docs as I am working on it & don't want to keep it in sync ftm.

Closes facebook#503
mganandraj pushed a commit to mganandraj/react-native that referenced this pull request Jul 21, 2020
* Fix onFocus/onBlur event bubbling (facebook#506)

* Update scripts to publish react-native-macos-init

* Clean up merge markers

* Restored ios:macos RNTester parity except for InputAccessoryView.

* Revert "Restored ios:macos RNTester parity except for InputAccessoryView."

This reverts commit 5a67ae0.

* Remove unnecessary android builds and tar file upload.

* Fix onFocus/onBlur View events to properly bubble.

* The Text component can also be selectable={true} and needs the same focus/blur event triggering as View.

Co-authored-by: React-Native Bot <53619745+rnbot@users.noreply.github.com>

* Added new RCTFocusChangeEvent to static xcodeproj's

* Change vmImage to ubuntu-latest for Android builds.

Co-authored-by: React-Native Bot <53619745+rnbot@users.noreply.github.com>
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