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

Glitches on Safari iOS #138

Closed
7 of 9 tasks
dascritch opened this issue Apr 7, 2021 · 10 comments
Closed
7 of 9 tasks

Glitches on Safari iOS #138

dascritch opened this issue Apr 7, 2021 · 10 comments
Labels
buggy browser Incorrect standard implemntation in browser help wanted Extra attention is needed

Comments

@dascritch
Copy link
Owner

dascritch commented Apr 7, 2021

@dascritch dascritch added buggy browser Incorrect standard implemntation in browser help wanted Extra attention is needed labels Apr 7, 2021
dascritch added a commit that referenced this issue Apr 7, 2021
@dascritch
Copy link
Owner Author

dascritch commented Apr 7, 2021

dascritch added a commit that referenced this issue Apr 7, 2021
Glitches on Safari iOS #138
dascritch added a commit that referenced this issue Apr 7, 2021
dascritch added a commit that referenced this issue Apr 7, 2021
Glitches on Safari iOS #138
@dascritch
Copy link
Owner Author

dascritch commented Apr 8, 2021

Most of the actual glitches seems from https://developer.apple.com/forums/thread/90784
https://webkit.org/blog/7734/auto-play-policy-changes-for-macos/ is not really informative as our tests are already in “muted” state, and click events aren't honored. Promises are systematically going in error

This is very bad. And that Apple decides of a white-list is not good at all.

dascritch added a commit that referenced this issue Apr 8, 2021
@dascritch
Copy link
Owner Author

Safari should be the last browser to not be able de calculated CSS properties of <svg> tags on the fly.

The last 4 points :

  • doable
    • restore alternative fine browsing mode (JS related)
    • focus outline of WebComponent (CSS related)
  • really complex to solve
    • incorrect loading state at start (autoplay preventing related)
    • clicking in playlist do change the controller but not starting the audio (autoplay preventing related)

dascritch added a commit that referenced this issue Apr 8, 2021
@dascritch
Copy link
Owner Author

dascritch commented Apr 9, 2021

it's not on <cpu-audio>, it's not on :host , it's not :focus , :focus-within , :hover .....
I'm searching for the source of the default stylesheet of safari :/

@dascritch
Copy link
Owner Author

@dascritch
Copy link
Owner Author

OK, Safari is not understanding yet :focus-visible, butseems to have its specially prefixed property :-webkit-direct-focus and in fact, Safari doesn't understand corectly how to handle "focus" event
https://blogs.igalia.com/mrego/2021/01/28/focus-visible-in-webkit-january-2021/

@dascritch
Copy link
Owner Author

(I write it in French because it really piss me off)

Si vous jouez avec les pseudo-sélecteurs de :focus comme :focus-within ou :focus-visible dans le shadowRoot et que vous vous demandez pourquoi c'est tant le bordel sur Safari. Ben c'est en rapport avec vos problèmes sur la gestion de l'événement focus , buggé depuis toujours sur Safari
https://blogs.igalia.com/mrego/2021/01/28/focus-visible-in-webkit-january-2021/
Le mot-clé qui pourrait vous aider dans vos futures recherches : :-webkit-direct-focus

@dascritch
Copy link
Owner Author

Sticked in "loading" mode wasn't autoplay prevented, but some events missing in Safari iOS.

@dascritch
Copy link
Owner Author

dascritch commented Apr 9, 2021

Some bugs in tests seems related to seeking time operations.
And yes : https://stackoverflow.com/questions/18266437/html5-video-currenttime-not-setting-properly-on-iphone

I had to reduce the ready state limit to 2 (HTMLVideoElement.prototype.HAVE_CURRENT_DATA) as the loadeddata event on iOS would often never come in higher than 2.
Completely crazy

dascritch added a commit that referenced this issue Apr 9, 2021
dascritch added a commit that referenced this issue Apr 9, 2021
@dascritch
Copy link
Owner Author

:host(:focus) {  outline : none; } 

Not working too :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
buggy browser Incorrect standard implemntation in browser help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant