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

Better default empty states with animation #12261

Closed
shawnborton opened this issue Oct 28, 2022 · 134 comments
Closed

Better default empty states with animation #12261

shawnborton opened this issue Oct 28, 2022 · 134 comments
Assignees
Labels

Comments

@shawnborton
Copy link
Contributor

Note: This issue is part of the greater Implement Our New Branding in NewDot project.

What is it:
Bring life to our app by animating various empty states illustrations. We can split this project into two phases: first, let’s identify areas where we can use illustrations for empty states. Augenblick can help us create new illustrations. Then we can take those illustrations and bring them to life with animation. Again, Augenblick can help us by animating these illustrations. We can then use Lottie to implement the animations into the product.

image

Does it require a mini design doc:
Yes

Design finished:
No

cc @GabiHExpensify

@shawnborton shawnborton self-assigned this Oct 28, 2022
@melvin-bot
Copy link

melvin-bot bot commented Oct 28, 2022

Auto-assign attempt failed, all eligible assignees are OOO.

@shawnborton
Copy link
Contributor Author

Adding a note that as part of this conversation here, we've decided to remove this one from the WAQ Project. That doesn't mean we shouldn't do this, but let's prioritize all other WAQ issues first before moving onto this one.

@melvin-bot melvin-bot bot added the Overdue label Nov 10, 2022
@roryabraham
Copy link
Contributor

I personally haven't invested any time into this yet, but I'm definitely excited to work on it when the time is right. @shawnborton keep me posted on next steps and let me know how I can help!

We'll also want keep an eye on this absolutely monstrous PR to add new architecture support in Lottie

@melvin-bot melvin-bot bot removed the Overdue label Nov 11, 2022
@JmillsExpensify
Copy link

Ha @shawnborton what a coincidence. Check out the reviewer on that linked PR!

@shawnborton
Copy link
Contributor Author

Hah, what are the odds!

In other news, I'll post some ideas to the room today and we'll get kicked off with Augenblick next week.

@shawnborton
Copy link
Contributor Author

Augenblick sent over some test animation Lottie files for us - they are mostly looking for freedback on if this format will work, and how the file size is. Thoughts @roryabraham ?

Files: Expensify_LottieTests_112222.zip

Given that the JSON file size is less than 100KB, I think this is going to work great.

runningTest

ufoTest

dollarTest

@roryabraham
Copy link
Contributor

roryabraham commented Nov 23, 2022

I'll see if I can get a POC together ... @shawnborton is there any existing screen that I can work on putting one of these in already? That way we could actually merge it and the POC won't end up getting really out-of-date

@shawnborton
Copy link
Contributor Author

I don't think we have an existing screen to use for this yet, we'd be making new layouts as part of this project.

@JmillsExpensify
Copy link

Shouldn't this issue be on hold for WAQ?

@shawnborton
Copy link
Contributor Author

We discussed this previously and agreed that it makes sense to get started with Augenblick on this because we can do that in parallel to WAQ without needing Engineering resources. That being said, I can see an argument for Rory to hold off on doing any kind of engineering testing here given that he should be prioritizing WAQ. I still think it makes sense for us to be doing this work with Augenblick in the meantime though.

@roryabraham
Copy link
Contributor

Haven't invested any time in this yet, but I'm excited to do so when the time comes!

@JmillsExpensify
Copy link

Ok cool, yeah totally agree we can push forward with Augenblick, I just don't think we should push forward with any engineering time until we're in a better spot with WAQ. I'm throwing a HOLD on this issue to make sure that's clear, though please feel free to use the issue for on-going Augenblick updates.

@JmillsExpensify JmillsExpensify changed the title Better default empty states with animation [HOLD WAQ] Better default empty states with animation Nov 29, 2022
@melvin-bot melvin-bot bot added the Overdue label Dec 8, 2022
@roryabraham
Copy link
Contributor

Still on HOLD (for my part, anyways...)

@melvin-bot melvin-bot bot removed the Overdue label Dec 8, 2022
@melvin-bot melvin-bot bot added the Overdue label Dec 19, 2022
@shawnborton
Copy link
Contributor Author

Not overdue. We have all of our animations and Lottie files completed, we just need to wait for WAQ to be finished and then we can hopefully start working through this one.

Some treats for you:
Expensify-Magic-Code-121222-kjs-1

Expensify-Magic-Link-120922-kjs-1

Expensify-Fast-Money-2-121322-kjs-1

@melvin-bot melvin-bot bot removed the Overdue label Dec 19, 2022
@JmillsExpensify
Copy link

We're getting real close on the bugs. The bigger blockers for WAQ are going to be the headline initiatives, so once we get those out of the way we'll be back at it!

@melvin-bot melvin-bot bot added the Overdue label Dec 27, 2022
@shawnborton
Copy link
Contributor Author

Not overdue, still holding.

@melvin-bot melvin-bot bot removed the Overdue label Dec 29, 2022
@melvin-bot melvin-bot bot added the Overdue label Jan 6, 2023
@allroundexperts
Copy link
Contributor

I reviewed #21852. Bumping for the payment!

@melvin-bot melvin-bot bot added the Overdue label Aug 24, 2023
@melvin-bot
Copy link

melvin-bot bot commented Aug 25, 2023

@shawnborton, @roryabraham Whoops! This issue is 2 days overdue. Let's get this updated quick!

@melvin-bot
Copy link

melvin-bot bot commented Aug 29, 2023

@shawnborton, @roryabraham 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

@shawnborton
Copy link
Contributor Author

@roryabraham what is the latest here? Can we please try to wrap this up as soon as possible? It feels like we're so close but this one keeps dragging on. Does it make sense to give this one to some contributors or even an agency?

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Aug 29, 2023
@melvin-bot
Copy link

melvin-bot bot commented Sep 4, 2023

@shawnborton, @roryabraham Huh... This is 4 days overdue. Who can take care of this?

@melvin-bot
Copy link

melvin-bot bot commented Sep 5, 2023

@shawnborton, @roryabraham Still overdue 6 days?! Let's take care of this!

@shawnborton
Copy link
Contributor Author

@roryabraham can you please provide an update here?

@melvin-bot melvin-bot bot removed the Overdue label Sep 5, 2023
@roryabraham
Copy link
Contributor

roryabraham commented Sep 5, 2023

@roryabraham
Copy link
Contributor

Created a separate issue to track enabling lottie file format: #26857

@melvin-bot melvin-bot bot added the Overdue label Sep 11, 2023
@roryabraham
Copy link
Contributor

We're working in parallel on the security and about pages, as well as DotLottie file formats. @shawnborton

Also created #27211

@melvin-bot melvin-bot bot removed the Overdue label Sep 12, 2023
@roryabraham
Copy link
Contributor

Created #27407 – the final page for this issue

@melvin-bot melvin-bot bot added the Overdue label Sep 18, 2023
@roryabraham
Copy link
Contributor

Just fixed the animation for the About page

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Sep 19, 2023
@roryabraham
Copy link
Contributor

So the last piece here is the about page, and for some reason the animation just doesn't work normally on iOS Safari. We can hack a fix by using a canvas renderer instead of svg, but that messes up the heights. We should be able to hack a fix for the height, but I wonder if we might just ask Augenblick if they can see if there's something wrong with the animation in AfterEffects

@melvin-bot melvin-bot bot removed the Overdue label Sep 22, 2023
@roryabraham
Copy link
Contributor

Maybe we hack a fix then create a follow-up

@melvin-bot melvin-bot bot added the Overdue label Sep 25, 2023
@melvin-bot
Copy link

melvin-bot bot commented Sep 25, 2023

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

@roryabraham
Copy link
Contributor

Because the last holdout here is the About page, and the About page is being handled in #26779, I think we can close this out! We're also implementing more improvements to our usage of lottie, but they're also being handled separately

@melvin-bot melvin-bot bot removed the Overdue label Sep 25, 2023
@shawnborton
Copy link
Contributor Author

shawnborton commented Sep 26, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

8 participants