Skip to content

Move sticky header out of the way so foreground touches work - #112

Open
nicotroia wants to merge 3 commits into
i6mi6:masterfrom
nicotroia:fix/sticky-header-position-when-hidden
Open

Move sticky header out of the way so foreground touches work#112
nicotroia wants to merge 3 commits into
i6mi6:masterfrom
nicotroia:fix/sticky-header-position-when-hidden

Conversation

@nicotroia

Copy link
Copy Markdown

Touchable items in the foreground are currently blocked when they are positioned over the sticky header.

This change animates the sticky header to move it out of the way before the user scrolls up, allowing presses to happen in that area.

@flikQ flikQ left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

this PR causes issue:

Invariant Violation: Transform with key of "translateY" must be a number:
{"translateY": -70}

the -70 being the height set by stickyHeaderHeight prop

@flikQ

flikQ commented Sep 6, 2018

Copy link
Copy Markdown

ok found the issue, you forgot an Animated.View on the stickyheader container.

Make parent View Animated
@nicotroia

Copy link
Copy Markdown
Author

@flikQ Good catch, fixed

@Aloving

Aloving commented Nov 15, 2018

Copy link
Copy Markdown

What now, will it be merged?

@maherzaidoune maherzaidoune left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

works fine , press detected on Foreground

@dsgoers

dsgoers commented Mar 6, 2019

Copy link
Copy Markdown

Any update on this? This fix would really help my project.

@Josephdias92

Copy link
Copy Markdown

Please merge this. I'm stuck coz of this issue

darcien added a commit to darcien/react-native-parallax-scroll-view that referenced this pull request Apr 16, 2019
@charro0407

Copy link
Copy Markdown

You are a Hero!

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.

10 participants