Skip to content

Can't select any SWF file on Safari on iPad OS 17 (iOS bug?) #176

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
orazioedoardo opened this issue Nov 14, 2023 · 8 comments · Fixed by ruffle-rs/ruffle#14017
Closed

Comments

@orazioedoardo
Copy link

orazioedoardo commented Nov 14, 2023

Clicking "Select File" opens up the system file select sheet, as expected. However all SWF files are greyed out and thus not selectable. Through some empirical test, it looks like to me that Safari is not allowing input elements type="file" with attribute accept=".swf,.spl". I've also tried application/x-shockwave-flash and it’s not accepted either. However if I select "Request mobile site" in the Safari UI, then I can select any file, not just SWF and SPL, which is exactly what happens on iOS, where mobile version is the default. Can you confirm?

@danielhjacobs
Copy link
Collaborator

What's your User Agent? Unfortunately, even though it's bad practice, the presence or lack of the "accept" is determined by user-agent sniffing. See ruffle-rs/ruffle#5025

@danielhjacobs
Copy link
Collaborator

"With iPadOS, Apple has started to position the iPad as a productivity device much like a traditional laptop or desktop. As part of that change, the Safari web browser on iPadOS defaults to sending a User-Agent string that appears to be from a macOS device instead of an iOS or an iPadOS device.": https://help.duo.com/s/article/5187?language=en_US

@danielhjacobs
Copy link
Collaborator

I think we need this check: https://stackoverflow.com/a/62979491

@orazioedoardo
Copy link
Author

What's your User Agent?

Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.1.1 Safari/605.1.15

@danielhjacobs
Copy link
Collaborator

What about navigator.platform and navigator.standalone?

@orazioedoardo
Copy link
Author

MacIntel and false.

@danielhjacobs
Copy link
Collaborator

Ok, this should be fixed once I fix the TS errors on ruffle-rs/ruffle#14017 and it gets merged.

@orazioedoardo
Copy link
Author

this should be fixed

Confirmed.

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.

2 participants