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

[HOLD for payment 2021-09-27] Add a scroll bounce or elasticity animation when reaching the top or bottom of a page #4816

Closed
isagoico opened this issue Aug 25, 2021 · 27 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Engineering External Added to denote the issue can be worked on by a contributor Improvement Item broken or needs improvement. Reviewing Has a PR in review Weekly KSv2

Comments

@isagoico
Copy link

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Action Performed:

  1. Open New Expensify on Android or iOS
  2. Navigate to a conversation
  3. Scroll to the top (or scroll a bit upwards and then to the bottom)

Expected Result:

There should be a bounce or elasticity animation when reaching the top or bottom

Actual Result:

No animation is shown when reaching the top or bottom of a page or conversation

Platform:

Where is this issue occurring?

  • iOS
  • Android

Version Number: 1.0.86-9

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos:

Expensify/Expensify Issue URL:

View all open jobs on GitHub


From @shawnborton https://expensify.slack.com/archives/C01GTK53T8Q/p1629717577129000

FEATURE REQUEST: This might be nit-picky but I noticed that scroll views on iOS don’t have any kind of bounce or elasticity when you reach the end. For example, the list of chats, or any view in Settings, or even when you are viewing a chat with someone. Looks like this is supported in RN via scrollview but not sure how that would work with our implementation?

@isagoico isagoico added AutoAssignerTriage Auto assign issues for triage to an available triage team member Daily KSv2 labels Aug 25, 2021
@MelvinBot
Copy link

Triggered auto assignment to @strepanier03 (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

@MelvinBot MelvinBot added Overdue and removed AutoAssignerTriage Auto assign issues for triage to an available triage team member labels Aug 25, 2021
@strepanier03
Copy link
Contributor

OoO, reassigning.

@MelvinBot MelvinBot removed the Overdue label Aug 27, 2021
@strepanier03 strepanier03 removed their assignment Aug 27, 2021
@strepanier03 strepanier03 added the AutoAssignerTriage Auto assign issues for triage to an available triage team member label Aug 27, 2021
@MelvinBot
Copy link

Triggered auto assignment to @arielgreen (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

@MelvinBot MelvinBot removed the AutoAssignerTriage Auto assign issues for triage to an available triage team member label Aug 27, 2021
@arielgreen arielgreen removed their assignment Aug 27, 2021
@MelvinBot
Copy link

Triggered auto assignment to @cead22 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@arielgreen arielgreen added the Improvement Item broken or needs improvement. label Aug 27, 2021
@cead22 cead22 removed their assignment Aug 27, 2021
@cead22 cead22 added Engineering External Added to denote the issue can be worked on by a contributor and removed Engineering labels Aug 27, 2021
@MelvinBot
Copy link

Triggered auto assignment to @MitchExpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@parasharrajat
Copy link
Member

parasharrajat commented Aug 28, 2021

Proposal

  1. We have to use bounces:true prop on the ScrollViews to make them bounce naturally on IOS.
    e.g.
    <ScrollView style={styles.flex1} contentContainerStyle={styles.p5}>
  2. I will update every page so that it has bounces behavior.
  3. There are few places where we will jointly decide if we want the bounce for example chat list, Users list in LHN and etc. I will update those based on the team's feedback.
  4. We can also add the overScrollMode prop for Android devices so that it shows a natural glow effect when reaching the end of the list.

@MitchExpensify
Copy link
Contributor

Upwork Job

@MelvinBot MelvinBot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 30, 2021
@MelvinBot
Copy link

Triggered auto assignment to @chiragsalian (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@MelvinBot MelvinBot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 31, 2021
@MitchExpensify
Copy link
Contributor

Hired!

@chiragsalian
Copy link
Contributor

Any update on this issue @parasharrajat?

@MelvinBot MelvinBot removed the Overdue label Sep 2, 2021
@parasharrajat
Copy link
Member

I will drop the PR soon.

@MelvinBot
Copy link

@chiragsalian, @parasharrajat Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@parasharrajat
Copy link
Member

I am on vacation. back on 8.

@MelvinBot MelvinBot removed the Overdue label Sep 7, 2021
@arielgreen arielgreen added Weekly KSv2 and removed Daily KSv2 labels Sep 7, 2021
@arielgreen
Copy link
Contributor

@MitchExpensify reassigned to you, this should remain assigned for the duration of the issue.

@MitchExpensify
Copy link
Contributor

Ah whoops, thanks for reassigning! Noted

@parasharrajat
Copy link
Member

@shawnborton What color do we want for the OverScroll Effect on Android?
Here is the video of what I am asking.

effect.mp4

@shawnborton
Copy link
Contributor

Maybe the same color we use for our icons or borders? Something that is gray and a bit more subtle, but noticeable. Thoughts on that?

@parasharrajat
Copy link
Member

Hmm, I would say that works but let me check it.

@chiragsalian chiragsalian added the Reviewing Has a PR in review label Sep 14, 2021
@botify
Copy link

botify commented Sep 14, 2021

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@botify
Copy link

botify commented Sep 15, 2021

🚀 Deployed to staging by @chiragsalian in version: 1.0.98-2 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@mountiny mountiny added the Awaiting Payment Auto-added when associated PR is deployed to production label Sep 22, 2021
@mountiny mountiny changed the title Add a scroll bounce or elasticity animation when reaching the top or bottom of a page [HOLD for payment 2021-09-27] Add a scroll bounce or elasticity animation when reaching the top or bottom of a page Sep 22, 2021
@mountiny
Copy link
Contributor

The PR has been deployed to production on Monday 20th September. Updating the title appropriately 🙌

@parasharrajat
Copy link
Member

Ping for
image

@mountiny
Copy link
Contributor

@parasharrajat You should be paid shortly hopefully!

@dylanexpensify
Copy link
Contributor

Payment sent! Job closed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Engineering External Added to denote the issue can be worked on by a contributor Improvement Item broken or needs improvement. Reviewing Has a PR in review Weekly KSv2
Projects
None yet
Development

No branches or pull requests