virtual icon indicating copy to clipboard operation
virtual copied to clipboard

Request Fullscreen Issue in Desktop Safari

Open thedivac opened this issue 2 years ago • 4 comments

Describe the bug

I am rendering a long list of content using useWindowVirtualizer. Each list can be shown in full screen mode by clicking a button using domElement.requestFullscreen(). When I scroll down and request fullscreen on an item that is not rendered initially, the full screen pops up and closes immediately. I assume this happens because the virtualizer thinks we just scrolled to the top as soon as the full screen is shown, then unmounts the element that we are displaying in full screen (because it's not visible when the list is scrolled to the top) which then closes the full screen again. See codesandbox for an example. It works in Chrome, but fails in Safari.

Your minimal, reproducible example

https://c9l4ym-5173.csb.app

Steps to reproduce

Open the code sandbox on desktop safari in a new tab, scroll down and click request full screen.

Expected behavior

The element should be shown in full screen.

How often does this bug happen?

Every time

Screenshots or Videos

No response

Platform

Desktop Safari

tanstack-virtual version

3.0.1

TypeScript version

No response

Additional context

No response

Terms & Code of Conduct

  • [X] I agree to follow this project's Code of Conduct
  • [X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

thedivac avatar Dec 20 '23 19:12 thedivac

@thedivac could share a link to codesandbox to checkout the implementation

piecyk avatar Dec 22 '23 12:12 piecyk

Here you go: https://codesandbox.io/p/devbox/fullscreen-error-in-desktop-safari-c9l4ym

thedivac avatar Dec 22 '23 14:12 thedivac

Here you go: https://codesandbox.io/p/devbox/fullscreen-error-in-desktop-safari-c9l4ym

@thedivac looks like a safari bug, we can try to patch it like this https://codesandbox.io/p/devbox/fullscreen-error-in-desktop-safari-forked-zgcnq5?file=%2Fsrc%2Findex.css

piecyk avatar Dec 22 '23 14:12 piecyk

Here you go: https://codesandbox.io/p/devbox/fullscreen-error-in-desktop-safari-c9l4ym

@thedivac looks like a safari bug, we can try to patch it like this https://codesandbox.io/p/devbox/fullscreen-error-in-desktop-safari-forked-zgcnq5?file=%2Fsrc%2Findex.css

Awesome, this did the trick!

thedivac avatar Dec 22 '23 14:12 thedivac