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

Ignore bogus onScroll values #5499

Merged
merged 4 commits into from
Sep 27, 2024
Merged

Ignore bogus onScroll values #5499

merged 4 commits into from
Sep 27, 2024

Conversation

mozzius
Copy link
Member

@mozzius mozzius commented Sep 26, 2024

The scroll listener is reporting strange values when the profile first loads (equal to negative header height).

This is a little bit hacky, but it appears safe to ignore those values, since they're typically way out of the range you can possibly achieve by just overscrolling.

Video showing the bogus values:

Screen.Recording.2024-09-26.at.16.17.01.mov

@arcalinea arcalinea temporarily deployed to samuel/strange-onscroll-values - social-app PR #5499 September 26, 2024 15:38 — with Render Destroyed
Copy link

github-actions bot commented Sep 26, 2024

Old size New size Diff
9.77 MB 9.77 MB 0 B (0.00%)

@arcalinea arcalinea temporarily deployed to samuel/strange-onscroll-values - social-app PR #5499 September 27, 2024 07:45 — with Render Destroyed
@mozzius mozzius merged commit 4553e6b into main Sep 27, 2024
6 checks passed
@mozzius mozzius deleted the samuel/strange-onscroll-values branch September 27, 2024 09:09
estrattonbailey added a commit that referenced this pull request Sep 30, 2024
* origin/main:
  Use Inter variable font (#5540)
  Fix name in `feature_request.yml` (#5542)
  Fix sticky offset, gear color (#5537)
  Move email test to root tests dir (#5527)
  revamp issue templates, add new arch template (#5532)
  use PressableScale for animation (#5541)
  Rework native autocomplete (#5521)
  Pinned posts (#5055)
  Remove Segment (#5518)
  Fix alignment of cancel button on search (#5520)
  [Share Extension] Support on Android for sharing videos to app (#5466)
  Ignore bogus onScroll values (#5499)
  add podcasts to spotify embeds (#5514)
  Tweak font size of "Write your reply" (#5513)
  Lazy load ViewShot (#5517)
  [Share Extension] Support images/movies from other apps like iMessage (#5515)
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.

3 participants