Skip to content

Added missing examples for TouchableHighlight and TouchableWithoutFeedback#30364

Closed
kshitij86 wants to merge 1 commit into
facebook:masterfrom
MLH-Fellowship:add-examples-touchable-highlight
Closed

Added missing examples for TouchableHighlight and TouchableWithoutFeedback#30364
kshitij86 wants to merge 1 commit into
facebook:masterfrom
MLH-Fellowship:add-examples-touchable-highlight

Conversation

@kshitij86
Copy link
Copy Markdown
Contributor

@kshitij86 kshitij86 commented Nov 11, 2020

Summary

Added example for the following missing props in TouchableHighlight
  • onHideUnderlay
  • onShowUnderlay

Motivation - Missing examples for these props in the RNTester app

Changelog

[General] [Added] - Added example to ToucableHighlight

Test Plan

  • Tested on an Android device
    under_2
    under_1
    Screenshot_1605441612

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 11, 2020
@react-native-bot react-native-bot added the Type: Enhancement A new feature or enhancement of an existing feature. label Nov 11, 2020
@analysis-bot
Copy link
Copy Markdown

analysis-bot commented Nov 11, 2020

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: 9c827f6

@kshitij86 kshitij86 changed the title Add examples touchable highlight Added examples for TouchableHighlight visibility Nov 11, 2020
@kshitij86 kshitij86 changed the title Added examples for TouchableHighlight visibility Added examples for TouchableHighlight underlay visibility Nov 11, 2020
@mkcode
Copy link
Copy Markdown
Contributor

mkcode commented Nov 13, 2020

Looks great to me

@analysis-bot
Copy link
Copy Markdown

analysis-bot commented Nov 15, 2020

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 7,537,724 +866
android hermes armeabi-v7a 7,122,092 +868
android hermes x86 7,977,489 +872
android hermes x86_64 7,889,086 +876
android jsc arm64-v8a 8,992,895 +0
android jsc armeabi-v7a 8,560,374 +0
android jsc x86 8,994,147 +0
android jsc x86_64 9,546,018 +0

Base commit: 408bcde

@kshitij86 kshitij86 changed the title Added examples for TouchableHighlight underlay visibility Added missing examples for TouchableHighlight and TouchableWithoutFeedback Nov 25, 2020
@mkcode
Copy link
Copy Markdown
Contributor

mkcode commented Dec 8, 2020

@kshitij86 - This looks good, but has lots of extra commits. Let's rebase this.

@kshitij86 kshitij86 requested a review from mhorowitz as a code owner December 9, 2020 04:39
@kshitij86 kshitij86 force-pushed the add-examples-touchable-highlight branch from 1291658 to ca202b6 Compare December 9, 2020 06:01
@kshitij86
Copy link
Copy Markdown
Contributor Author

@mkcode Done, squashed all those commits into a single one.

@kshitij86
Copy link
Copy Markdown
Contributor Author

@rickhanlonii Can you please take a look at these changes?

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@rickhanlonii has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@rickhanlonii merged this pull request in b64ffd7.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Jan 8, 2021
@rickhanlonii rickhanlonii deleted the add-examples-touchable-highlight branch January 8, 2021 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Type: Enhancement A new feature or enhancement of an existing feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants