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

[$250] Update Wallet and Generic Folder empty state to use lottie animations #48013

Open
shawnborton opened this issue Aug 26, 2024 · 20 comments
Assignees
Labels
External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item. Reviewing Has a PR in review Weekly KSv2

Comments

@shawnborton
Copy link
Contributor

shawnborton commented Aug 26, 2024

We have some empty states in our product that are currently static, and we want to update them to have looping Lottie animations.

The first place is the Wallet empty state:
CleanShot 2024-08-26 at 11 55 25@2x

And the other place is anywhere we show the folder empty state, like Search or Workspaces > Tags, Workspaces > Report fields, etc:
CleanShot 2024-08-26 at 11 58 43@2x

Here are the lottie files needed:

SettingsWallet-082324-1

GenericEmptyState-082324-1

cc @roryabraham @Expensify/design @luacmartins for visibility

@trjExpensify - I wonder if this is considered #wave-control polish or something?

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0164ffd5b6f9684066
  • Upwork Job ID: 1828144309254029734
  • Last Price Increase: 2024-08-26
  • Automatic offers:
    • Krishna2323 | Contributor | 103690909
Issue OwnerCurrent Issue Owner: @hungvu193
@shawnborton shawnborton added the NewFeature Something to build that is a new item. label Aug 26, 2024
@shawnborton shawnborton self-assigned this Aug 26, 2024
Copy link

melvin-bot bot commented Aug 26, 2024

Triggered auto assignment to @anmurali (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added the Weekly KSv2 label Aug 26, 2024
Copy link

melvin-bot bot commented Aug 26, 2024

⚠️ It looks like this issue is labelled as a New Feature but not tied to any GitHub Project. Keep in mind that all new features should be tied to GitHub Projects in order to properly track external CAP software time ⚠️

Copy link

melvin-bot bot commented Aug 26, 2024

Current assignee @shawnborton is eligible for the NewFeature assigner, not assigning anyone new.

This comment was marked as off-topic.

@Krishna2323
Copy link
Contributor

Krishna2323 commented Aug 26, 2024

Edited by proposal-police: This proposal was edited at 2024-08-26 17:21:01 UTC.

Proposal

Please re-state the problem that we are trying to solve in this issue.

Update Wallet and Generic Folder empty state to use lottie animations

What is the root cause of that problem?

Improvement

What changes do you think we should make in order to solve the problem?

We need to add the animations in this file and update the props below:

headerMedia: Illustrations.EmptyState,

illustration={Illustrations.BigVault}

headerMedia={Illustrations.EmptyState}

headerMedia={Illustrations.EmptyState}

EmptyStateExpenses

icon={Illustrations.EmptyStateExpenses}

headerMedia={Illustrations.EmptyStateExpenses}

headerMedia={Illustrations.EmptyStateExpenses}

We might also need to update several other props like headerMediaType

headerMediaType={CONST.EMPTY_STATE_MEDIA.ILLUSTRATION}

  • In IOURequestStepCategory we can use EmptyStateComponent instead of WorkspaceEmptyStateSection, or we can update WorkspaceEmptyStateSection to use EmptyStateComponent, we also need to update in NetSuiteImportCustomFieldPage.
    <WorkspaceEmptyStateSection
    shouldStyleAsCard={false}
    icon={Illustrations.EmptyStateExpenses}
    title={translate('workspace.categories.emptyCategories.title')}
    subtitle={translate('workspace.categories.emptyCategories.subtitle')}
    containerStyle={[styles.flex1, styles.justifyContentCenter]}
    />

What alternative solutions did you explore? (Optional)


Result

Monosnap.screencast.2024-08-26.23-35-46.mp4

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Aug 26, 2024
@Krishna2323
Copy link
Contributor

Proposal Updated

  • Added result video

@roryabraham
Copy link
Contributor

@Krishna2323 I think you missed a few spots where this change is needed, such as here. Please enumerate all the locations in the app where Illustrations.EmptyState is used.

@Krishna2323
Copy link
Contributor

@roryabraham thanks, proposal updated, we also need to update in WorkspaceCategoriesPage & WorkspaceTagsPage.

@shawnborton
Copy link
Contributor Author

There's also a spot when creating reportFields that we have an empty list view as well.

@Krishna2323
Copy link
Contributor

@shawnborton, you mean this illustration also, right?

Monosnap (208) New Expensify 2024-08-27 00-22-01

@roryabraham
Copy link
Contributor

@shawnborton that illustration ^ is called Illustrations.EmptyStateExpenses. It's very similar to Illustrations.EmptyState - I think the only difference is that the receipts poking out are purple instead of blue. Can you confirm if we also should replace this with the animated alternative?

@shawnborton
Copy link
Contributor Author

Oh interesting, that particular page should be using the same empty state as Search & Tags, not sure why it doesn't.

The one I was referencing was this:
CleanShot 2024-08-26 at 14 55 25@2x

@roryabraham roryabraham self-assigned this Aug 26, 2024
@roryabraham roryabraham added the External Added to denote the issue can be worked on by a contributor label Aug 26, 2024
Copy link

melvin-bot bot commented Aug 26, 2024

Job added to Upwork: https://www.upwork.com/jobs/~0164ffd5b6f9684066

@melvin-bot melvin-bot bot changed the title Update Wallet and Generic Folder empty state to use lottie animations [$250] Update Wallet and Generic Folder empty state to use lottie animations Aug 26, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 26, 2024
Copy link

melvin-bot bot commented Aug 26, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @hungvu193 (External)

@shawnborton
Copy link
Contributor Author

Can you confirm if we also should replace this with the animated alternative?

Yes, but I think the whole view should be replaced with the new empty state style that we see everywhere else. I think it was missed when we updated the empty state views for Search, Tags, etc.

@Krishna2323
Copy link
Contributor

Proposal Updated

  • Added the other places which uses EmptyState & EmptyStateExpenses illustration.

@roryabraham
Copy link
Contributor

Optimized .lottie versions of the new animations (let's use these instead of the .json ones):
OptimizedAnimations.zip

@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 26, 2024
Copy link

melvin-bot bot commented Aug 26, 2024

📣 @Krishna2323 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

@Krishna2323
Copy link
Contributor

Thanks for assigning. I will start working on it within an hour.

@trjExpensify
Copy link
Contributor

@trjExpensify - I wonder if this is considered #wave-control polish or something?

I'll pop it in #wave-collect. 👍

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item. Reviewing Has a PR in review Weekly KSv2
Projects
Status: Polish
Development

No branches or pull requests

6 participants