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(pip): use animation keyframe #4669

Merged

Conversation

jasonwoodland
Copy link
Contributor

What this PR does 📖

  • Use animation keyframe for PIP drag and drop animation
  • Add animations utility functions
  • Add drop shadow to PIP

Which issue(s) this PR fixes 🔨

Resolve #4668

Special notes for reviewers 🗒️

Additional comments 🎤

@jasonwoodland jasonwoodland self-assigned this Sep 8, 2022
@github-actions github-actions bot added the Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA label Sep 8, 2022
@molimauro
Copy link
Contributor

LGTM! Did you try this on Safari? If yes we can merge ;)

@WanderingHogan WanderingHogan added Ready for QA Ready for QA team to test, Devs approved. and removed Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA labels Sep 8, 2022
@WanderingHogan
Copy link
Contributor

@phillsatellite can you merge this if it works on safari, and can you test on mobile and make a note here if there is an issue

@phillsatellite phillsatellite merged commit e55fbc6 into dev Sep 8, 2022
@phillsatellite phillsatellite deleted the 4668-use-requestanimationframe-for-pip-animations branch September 8, 2022 22:42
@github-actions github-actions bot removed the Ready for QA Ready for QA team to test, Devs approved. label Sep 8, 2022
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.

use requestAnimationFrame for PIP animations
4 participants