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

Animation frame wrong on iPad split view #293

Open
plu opened this issue Jan 30, 2020 · 6 comments · May be fixed by #294
Open

Animation frame wrong on iPad split view #293

plu opened this issue Jan 30, 2020 · 6 comments · May be fixed by #294

Comments

@plu
Copy link
Contributor

plu commented Jan 30, 2020

When you run your app on iPad in multitasking split screen, the animation looks wrong.

Artboard

plu added a commit to technology-ebay-de/NYTPhotoViewer that referenced this issue Jan 30, 2020
@plu plu linked a pull request Jan 30, 2020 that will close this issue
@ZevEisenberg
Copy link
Contributor

@plu I'm having trouble reproducing the underlying issue, or understanding what's wrong from the screenshots you posted. Would you mind posting full repro steps, and a GIF or video link, if possible?

@mr-fixit
Copy link
Contributor

pinging @plu

@plu
Copy link
Contributor Author

plu commented May 4, 2020

Sorry for the late reply. It's also possible that this is an issue only related to the usage of NYTPhotoViewer in our project. If you cannot reproduce it, feel free to close the issue.

@mr-fixit
Copy link
Contributor

mr-fixit commented May 4, 2020

If it's still happening for you, can you take a screen recording movie?

@plu
Copy link
Contributor Author

plu commented May 5, 2020

ezgif-5-5e785b98c2b0

@ZevEisenberg
Copy link
Contributor

@plu thanks, that's quite helpful! I wasn't able to reproduce the issue in the Objective-C example project, which looks like it's using the same transition style. (The Swift project uses a cross-fade instead of a zoom animation.) It would definitely be interesting if you have time to make a reduced test case.

No_Repro

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 a pull request may close this issue.

3 participants