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

Feature/fade on top scale num columns #1

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

Conversation

UnattendedFlight
Copy link

Added flatlist numColumns functionality to VegaScrollList + config parameters

Allows for the topmost items to fade out when scrolling
Allows for setting the minimum scale for the scale out animation
By passing numColumns to the <VegaScrollList/> component
the FlatList layout will reflect the chosen cols
and the VegaScrollItems will animate for all top cols
@UnattendedFlight
Copy link
Author

Reason for pull request:

I noticed that when more columns was added using numColumns, the actual columns were never taken into account for the animations. Only the first item would animate, then scrolling further down. Things got messy.

As such, I propose these minimal changes to allow for animating flatlists with more than one column.

Also additionally, there was a lack of configuration capabilities, so alongside numColumns feature, I've also added 2 more parameters.

fadeOutTop (boolean)  - decides whether the topmost items should fade out when scrolling further
scaleOutMin (number) - the scale number to animate to when the scale animation is applied.

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