Skip to content

Persistent Scrollbars (Android only) #39

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

Closed
wants to merge 1 commit into from
Closed

Persistent Scrollbars (Android only) #39

wants to merge 1 commit into from

Conversation

kpsroka
Copy link

@kpsroka kpsroka commented Oct 10, 2018

Expose Android's setScrollbarFadingEnabled via a ScrollView prop.

@kelset kelset added the 💡 Proposal This label identifies a proposal label Oct 12, 2018
@axe-fb
Copy link
Collaborator

axe-fb commented Oct 15, 2018

cc @mdvacca, @ayc1

@axe-fb
Copy link
Collaborator

axe-fb commented Oct 15, 2018

@ayc1 said it looks good. Should we get a PR and get this done ?

@jamonholmgren
Copy link
Collaborator

@kpsroka This looks good to me! IMO, this is a small and non-breaking change, so it probably can just be a pull request on the main repo rather than an RFC. If there's sufficient push-back about it there then we can turn it into an RFC.

@axe-fb
Copy link
Collaborator

axe-fb commented Oct 17, 2018

closing this issue - will wait for the PR on the main repo. If the PR takes time, ping me, and i can fast track the process

@grabbou
Copy link
Member

grabbou commented Nov 19, 2018

FYI @axe-fb, a PR has been submitted: facebook/react-native#22300

facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Nov 27, 2018
Summary:
This PR is a follow-up from the closed [discussions-and-proposals PR](react-native-community/discussions-and-proposals#39), that explains in more detail the rationale for adding this feature.
Pull Request resolved: #22300

Reviewed By: fkgozali

Differential Revision: D13121748

Pulled By: mdvacca

fbshipit-source-id: 899641be79bdb41fa6649df0772c602a5e09b3b9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 Proposal This label identifies a proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants