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

Remove filter pills from example page #288

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

suminkimm
Copy link

Summary

Closes #178

Changelog

Removed filter pills from example page by changing hideFilterPills property to true in RNTesterExampleList.js

[Android] [Change] - Remove filter pills
[iOS] [Change] - Remove filter pills

Test Plan

RNTester app builds and runs as expected and filter pills are removed from the screen.

@princiya
Copy link
Collaborator

👏 @suminkimm @JustinSo1

@princiya
Copy link
Collaborator

@suminkimm @JustinSo1 please open this PR against the main react-native repo and tag @rickhanlonii there so that he can get this merged into the main repo. :) 🎉

@mkcode mkcode force-pushed the remove-filter-pills branch 2 times, most recently from e4dbd29 to 9a13aa1 Compare October 23, 2020 19:47
@suminkimm suminkimm force-pushed the remove-filter-pills branch 2 times, most recently from d8961af to 16ef240 Compare October 27, 2020 23:34
@suminkimm suminkimm force-pushed the remove-filter-pills branch from f958ef0 to 1451ebe Compare October 27, 2020 23:49
Co-authored-by: Su Min Kim <[email protected]>
Co-authored-by: Justin So <[email protected]>
@suminkimm suminkimm force-pushed the remove-filter-pills branch from 1451ebe to b56d0e5 Compare October 27, 2020 23:53
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.

Remove filter pills from the example screen.
2 participants