-
Notifications
You must be signed in to change notification settings - Fork 3.3k
[file_selector] Add MIME type support on macOS #3862
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
[file_selector] Add MIME type support on macOS #3862
Conversation
Adds a macOS 11+ codepath that uses `UTType`s, allowing for supporting MIME type (and moving off of the deprecated `allowedFileTypes`). Fixes flutter/flutter#117843
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes to this file are just my running swift-format
on it, which I hadn't done previously; we don't yet have enforcement of formatting for Swift in CI.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! One optional suggestion.
packages/file_selector/file_selector_macos/macos/Classes/FileSelectorPlugin.swift
Outdated
Show resolved
Hide resolved
Adds a macOS 11+ codepath that uses `UTType`s, allowing for supporting MIME type (and moving off of the deprecated `allowedFileTypes`). Fixes flutter/flutter#117843
flutter/packages@f163786...407b7da 2023-05-03 [email protected] Update Cirrus to Xcode 14.3 (flutter/packages#3890) 2023-05-03 [email protected] [file_selector] Deprecates `macUTIs` (flutter/packages#3888) 2023-05-03 [email protected] [pigeon]enable treat warning as errors for swift code in unit test (flutter/packages#3889) 2023-05-02 [email protected] Update xcode to 14e222b (flutter/packages#3868) 2023-05-02 [email protected] [pigeon]fix "as Any" workaround due to nested optional (flutter/packages#3658) 2023-05-02 [email protected] [webview_flutter_android] Adds support to accept third party cookies (flutter/packages#3834) 2023-05-02 [email protected] [webview_flutter_wkwebview] Fixes an exception caused by the `onUrlChange` callback returning a null url (flutter/packages#3848) 2023-05-02 [email protected] [google_maps_flutter] [Docs] Note regarding usage within a bounded & an unbound widget (flutter/packages#3691) 2023-05-02 [email protected] [local_auth_android] Fix Android lint warnings (flutter/packages#3764) 2023-05-02 [email protected] [go_router_builder] Support go_router v7 (flutter/packages#3858) 2023-05-02 [email protected] [webview_flutter_wkwebview] Fixes internal enum type and adds unknown enum values (flutter/packages#3812) 2023-05-02 [email protected] [file_selector] Add MIME type support on macOS (flutter/packages#3862) 2023-05-02 [email protected] Roll Flutter from 828a040 to db6074a (12 revisions) (flutter/packages#3881) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages-flutter-autoroll Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Adds a macOS 11+ codepath that uses `UTType`s, allowing for supporting MIME type (and moving off of the deprecated `allowedFileTypes`). Fixes flutter/flutter#117843
Adds a macOS 11+ codepath that uses
UTType
s, allowing for supporting MIME type (and moving off of the deprecatedallowedFileTypes
).Fixes flutter/flutter#117843
Pre-launch Checklist
dart format
.)[shared_preferences]
pubspec.yaml
with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.CHANGELOG.md
to add a description of the change, following repository CHANGELOG style.///
).