Skip to content

Conversation

davidortinau
Copy link
Contributor

@davidortinau davidortinau commented Aug 18, 2025

Description:

  • Summary: Add .NET 10 guidance for multi-select media picking using PickPhotosAsync/PickVideosAsync (returning List) and document MediaPickerOptions (SelectionLimit, MaximumWidth/Height, CompressionQuality, RotateImage, PreserveMetaData, Title).
  • Files changed:
    • docs/platform-integration/device-media/picker.md
  • Monikers: Preserves <= .NET 9 content; adds >= .NET 10 section/examples for multi-select.
  • Platform notes:
    • Android: The picker UI may not strictly enforce SelectionLimit.
    • Windows: SelectionLimit not supported; behavior may vary.
    • Cancelation returns an empty list.
  • Verification: Cross-checked against MAUI net10.0 branch types and options.

Internal previews

📄 File 🔗 Preview link
docs/platform-integration/device-media/picker.md docs/platform-integration/device-media/picker

@davidortinau davidortinau requested a review from a team as a code owner August 18, 2025 18:25
@davidortinau davidortinau requested a review from Copilot August 18, 2025 18:26
Copy link

Learn Build status updates of commit 00b5258:

✅ Validation status: passed

File Status Preview URL Details
.github/copilot-instructions.md ✅Succeeded
docs/platform-integration/device-media/picker.md ✅Succeeded View (>=net-maui-8.0)
docs/user-interface/includes/pop-ups-dotnet10.md ✅Succeeded View (>=net-maui-8.0)

For more details, please refer to the build report.

Copy link

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request updates documentation for .NET MAUI 10's MediaPicker with multiselect support and improves async guidance for pop-ups. The changes use moniker ranges to provide version-specific documentation while maintaining backward compatibility.

  • Adds comprehensive .NET 10 documentation for MediaPicker multiselect functionality with new APIs and examples
  • Updates pop-ups documentation with async guidance for better UI responsiveness
  • Includes detailed platform-specific notes and limitations for multiselect operations

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
docs/user-interface/includes/pop-ups-dotnet10.md Adds guidance on using async methods from UI-thread contexts
docs/platform-integration/device-media/picker.md Documents new .NET 10 multiselect MediaPicker APIs with examples and platform notes
.github/copilot-instructions.md Adds comprehensive working instructions for .NET 10 documentation staging

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

…kVideosAsync, examples, and platform notes; moniker split maintained
@jfversluis jfversluis force-pushed the net10-pr02-mediapicker-multiselect branch from 00b5258 to 638936e Compare August 27, 2025 08:42
Copy link

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

Copy link

Learn Build status updates of commit 638936e:

✅ Validation status: passed

File Status Preview URL Details
docs/platform-integration/device-media/picker.md ✅Succeeded View (>=net-maui-8.0)

For more details, please refer to the build report.

@jfversluis jfversluis merged commit ae983fb into main Aug 27, 2025
5 checks passed
@jfversluis jfversluis deleted the net10-pr02-mediapicker-multiselect branch August 27, 2025 08:44
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 this pull request may close these issues.

2 participants