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

Debugging on real devices #735

Merged
merged 1 commit into from
Apr 7, 2015
Merged

Debugging on real devices #735

merged 1 commit into from
Apr 7, 2015

Conversation

nivoc
Copy link
Contributor

@nivoc nivoc commented Apr 7, 2015

Added a hint about how to enable debugging on real devices.

Added a hint about how to enable debugging on real devices.
@vjeux
Copy link
Contributor

vjeux commented Apr 7, 2015

Awesome thanks!

vjeux added a commit that referenced this pull request Apr 7, 2015
Debugging on real devices
@vjeux vjeux merged commit b1d3e44 into facebook:master Apr 7, 2015
@ide
Copy link
Contributor

ide commented Apr 7, 2015

After #652 is updated this will be easier to do (won't have to modify React's source).

@nivoc nivoc deleted the patch-2 branch April 7, 2015 23:24
ayushjainrksh referenced this pull request in MLH-Fellowship/react-native Jul 2, 2020
React Native has this issue where when using a `TextInput` with `multiline` set as `true`, the text is aligned to the top on iOS and to the center on Android by default, which causes confusion, evidenced by the amount of likes in [this comment](facebook#13897 (comment)). I believe the `textAlignVertical` default should be changed when `multiline` is `true`, but since I don't know if there are any reasons the issue I linked was closed, I suggest this is at least documented.
mganandraj pushed a commit to mganandraj/react-native that referenced this pull request Mar 10, 2021
* Add nullability checks (facebook#704)

* Update RCTCxxBridge.mm

* add nullability checks

* CI should run on PRs to future stable branches (facebook#718)

* Enclose the app path in quotes, so that the `open` command doesn't split it if it contains spaces (facebook#678)

* Enclose the app path in quotes, so that the `open` command doesn't split it if it contains spaces

* [ado] Remove no longer needed homebrew fix

Co-authored-by: Eloy Durán <[email protected]>

* update podfile.lock

* Enclose the app path in quotes, so that the `open` command doesn't split it if it contains spaces (facebook#678)

* Enclose the app path in quotes, so that the `open` command doesn't split it if it contains spaces

* [ado] Remove no longer needed homebrew fix

Co-authored-by: Eloy Durán <[email protected]>

* update podfile.lock

* windows dep updates to 63

Co-authored-by: Andrew Coates <[email protected]>
Co-authored-by: Sergi Mansilla <[email protected]>
Co-authored-by: Eloy Durán <[email protected]>
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.

3 participants