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

feat: new banner animation performance optimization #1235

Conversation

pygman
Copy link
Contributor

@pygman pygman commented Mar 20, 2023

display different banner between old and new, according to the FPS

@pygman pygman force-pushed the new-banner-animation-performance-optimization branch 3 times, most recently from 468b019 to 9551e06 Compare April 11, 2023 16:21
@pygman
Copy link
Contributor Author

pygman commented Apr 11, 2023

display different banner between old and new, according to the FPS

The first video shows that the old banner is displayed due to low FPS of test render

2023-04-12.00.17.40.mov

This video shows that the new banner displayed after performance pass in test render

2023-04-12.00.18.25.mov

@pygman pygman marked this pull request as ready for review April 11, 2023 16:23
@Keith-CY
Copy link
Collaborator

Keith-CY commented Apr 11, 2023

Please briefly describe these two videos because it's hard to understand in which scene they are working, especially for users outside the team.

@Keith-CY
Copy link
Collaborator

Keith-CY commented Apr 11, 2023

And please rebase and PR to https://github.com/Magickbase/ckb-explorer-frontend/ so a preview will be deployed on vercel, as https://ckb-explorer-git-develop-magickbase.vercel.app/. With that, the functionality could be verified on customers' computers.

@pygman pygman force-pushed the new-banner-animation-performance-optimization branch from 9551e06 to 0dadde9 Compare April 12, 2023 02:22
@pygman pygman closed this Apr 12, 2023
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.

2 participants