Restore use of IFileOpenPicker and IFileSavePicker
Categories
(Core :: Widget: Win32, enhancement, P2)
Tracking
()
People
(Reporter: saschanaz, Assigned: saschanaz)
References
Details
Attachments
(2 files, 1 obsolete file)
image.png
It allows to connect to UWP apps including Camera and Photos, and get files right from them just as the file picker in Firefox for Android can.
|
Assignee | |
Comment 1•2 years ago
|
||
|
||
Updated•2 years ago
|
|
|
Assignee | |
Comment 2•2 years ago
|
||
That opens a question: Does IFileSavePicker have any advantage too? (My patch uses it anyway, but)
|
|
||
Comment 3•2 years ago
•
|
||
- It changes the default dialog and button title.
- It changes the default overwrite behavior.
- It may inform accessibility tools of more accurate information.
|
|
Assignee | |
Comment 4•2 years ago
•
|
||
Thank you!
Just to log some chat from Slack: This should probably be rewritten in C++/WinRT instead of deprecated WRL, unless we have an important reason to support this in Windows 8 too. (I doubt it, it can just use the old API.) (Edit: Since cppwinrt is just a header-only library, I guess it shouldn't have compat issue with windows 8?)
https://docs.microsoft.com/en-us/windows/apps/desktop/modernize/desktop-to-uwp-supported-api?tabs=csharp#classes-that-use-iinitializewithwindow
https://docs.microsoft.com/en-us/windows/uwp/cpp-and-winrt-apis/move-to-winrt-from-wrl
|
|
Assignee | |
Comment 5•2 years ago
|
||
Some caveats:
- IFileOpenPicker does not support named filter groups. No grouped "Image Files", just a separate set of filter. IFileSavePicker does support it, weirdly.
- IFileOpenPicker defaults to All Files without support for filter index. This is particularly bad for
<input accept>. - IFileSavePicker does not support "All Files (.)", althought IFileOpenPicker does, also weirdly.
|
||
Updated•2 years ago
|
|
||
Updated•2 years ago
|
|
|
Assignee | |
Comment 6•3 months ago
|
||
Time to revisit as a hobby work item...
|
|
Assignee | |
Comment 7•2 months ago
|
||
Description
•