Skip to content

Low FPS on Safari 13.1 #725

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

Closed
ppamorim opened this issue Apr 21, 2020 · 4 comments · Fixed by #750
Closed

Low FPS on Safari 13.1 #725

ppamorim opened this issue Apr 21, 2020 · 4 comments · Fixed by #750

Comments

@ppamorim
Copy link

I noticed a low FPS when opening the docs on Safari 13.1. This problem does not happen when the page is not focused and does not happen on Chrome. I do not use any extension.

I tested the page by compiling the source code from this repo and configuring it to create the docs for the crate postgres.

I have recorded two videos showing the problem, being the first one a screen recording (less noticiable) and the second one I recorded my screen from a iPhone. I recommend you to watch the one recorded from the iPhone since the low FPS is more visible.

The low FPS does not happen if the page is not focused, I show this in the second video. To simulate this just open the page and move to another app.

https://www.youtube.com/watch?v=zhvuoTpoz2c&feature=youtu.be
https://www.youtube.com/watch?v=Y7Sy7rsCfsM&feature=youtu.be

@jyn514
Copy link
Member

jyn514 commented Apr 21, 2020

Possibly related to #595

@Nemo157
Copy link
Member

Nemo157 commented May 6, 2020

After some testing it seems like these safari scrolling issues are more related to the .focus() calls in the javascript than the position: fixed css. Commenting out all focusing seems to fix the lagginess.

@jyn514
Copy link
Member

jyn514 commented May 7, 2020

@ppamorim we just deployed a possible fix, can you check https://docs.rs/postgres/0.17.3/postgres/ and see if the problem is still there?

@ppamorim
Copy link
Author

@jyn514 Thank you very much! This beauty is running smooth at 144hz now!

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