Bug 265520
| Summary: | REGRESSION (iOS 17): Video bug z-index pointer-events doesn't work well | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Rodrigo Ludgero <rludgero> |
| Component: | Media | Assignee: | Jer Noble <jer.noble> |
| Status: | RESOLVED FIXED | ||
| Severity: | Critical | CC: | jer.noble, simon.fraser, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari 17 | ||
| Hardware: | iPhone / iPad | ||
| OS: | iOS 17 | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=261950 | ||
| Bug Depends on: | |||
| Bug Blocks: | 271764 | ||
Rodrigo Ludgero
I don't know what severity to set, maybe it's between critical and blocker.
We have a report that we have published, and it turns out that with the full screen video using object-fit, when you reach the screen that contains the video you can no longer continue scrolling.
It works OK with version 16.7.
In the end, the one that is in production had to be removed and published only with images.
https://us.as.com/futbol/lionel-messi-ha-revolucionado-estados-unidos-n/
I have published a version with the video in a test URL so you can see what is happening.
https://as.com/pruebas/arc/estaticos/especiales/2023/futbol/top10mls/test-index.html
For us it is something that is blocking us, because we cannot publish the videos that we had scheduled to publish.
Since most of our clientele in the USA uses IOS iPhone and many with version 17.
Thanks in advance.
Rodrigo
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/118936715>
Alexey Proskuryakov
*** This bug has been marked as a duplicate of bug 261950 ***
Rodrigo Ludgero
It's been 2 months and this bug is not resolved as it says in the issue https://bugs.webkit.org/show_bug.cgi?id=261563 status.
Either it has not been tested to see if it really is a duplicate bug of https://bugs.webkit.org/show_bug.cgi?id=261950, or it is not resolved as stated in the issue https://bugs.webkit.org/show_bug.cgi?id=261563.
Please, would it be possible to recheck the reason why it has stopped working.
Because it is affecting the navigation of all users who have versions equal to or greater than version 17.
Since with versions prior to 17, it did work perfectly.
Tested on version 17.3 today and still not working.
Alexey Proskuryakov
Thank you for re-testing. The fix for https://bugs.webkit.org/show_bug.cgi?id=261563 was included in iOS 17.1, so yes, this definitely needs another look.
Rodrigo Ludgero
(In reply to Alexey Proskuryakov from comment #4)
> Thank you for re-testing. The fix for
> https://bugs.webkit.org/show_bug.cgi?id=261563 was included in iOS 17.1, so
> yes, this definitely needs another look.
Hi, Alexey.
I find it very strange that the browser is not interpreting the pointer-events: none; property with this value. Because the video is inside a parent element with the pointer-events: none;.
Additionally, the same video element also contains the pointer-events: none;.
Perhaps by regressing to version iOS 16.7.5 that does work and comparing with version 17, it will be possible to find what is failing.
I have been looking for other issues that are reopened and perhaps this https://bugs.webkit.org/show_bug.cgi?id=183870 is the one that is related to the error we are having, because it is very similar, although not the same.
Let's hope it is resolved as soon as possible, since it affects us directly, making it impossible for us to publish news using videos as part of the content.
Jer Noble
Hello, fixes for this issue have landed in the following changes:
https://commits.webkit.org/268041@main
https://commits.webkit.org/268331@main
https://commits.webkit.org/268566@main
These should be available in iOS 17.2. Please let us know if you can still reproduce!
Rodrigo Ludgero
Hi Jer, I just tested it on an iPhone 13 mini with version 17.3.1 and it still doesn't work, this is the url https://as.com/pruebas/arc/estaticos/especiales/2023/futbol/top10mls/test-index.html I'm testing on.
Thank you and greetings.
Jer Noble
Pull request: https://github.com/WebKit/WebKit/pull/25545
EWS
Committed 276807@main (3ee555da60e6): <https://commits.webkit.org/276807@main>
Reviewed commits have been landed. Closing PR #25545 and removing active labels.