Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 15, 2025

Include statements in Picker, DatePicker, and TimePicker documentation were rendering as literal text instead of including content due to incorrect syntax and paths.

Changes

  • Fixed include syntax: Changed from ::: include to [!INCLUDE [text](path)]
  • Fixed include paths: Changed from ../../includes/ to ../includes/

Files Modified

  • docs/user-interface/controls/datepicker.md (2 includes)
  • docs/user-interface/controls/timepicker.md (2 includes)
  • docs/user-interface/controls/picker.md (1 include)

Example

Before:

::: include ../../includes/pickers-open-close-dotnet10.md

After:

[!INCLUDE [Pickers open and close](../includes/pickers-open-close-dotnet10.md)]

This affects .NET 10 sections documenting programmatic picker control and MVVM Toolkit usage.

Original prompt

This section details on the original issue you should resolve

<issue_title>Picker, DatePicker, and TimePicker have incorrect includes</issue_title>
<issue_description><!--
Before you open an issue

If the issue is:

  • A simple typo or similar correction, consider submitting a PR to fix it instead of logging an issue. See the contributor guide for instructions.
  • A general support question, consider asking on a support forum site.
  • A site design concern, create an issue at MicrosoftDocs/feedback.
  • A problem completing a tutorial, compare your code with the completed sample.
  • A duplicate of an open or closed issue, leave a comment on that issue.
    -->

Issue description

The pages for DatePicker, TimePicker, and Picker have been updated to reference an include for programmatically opening and closing them in .NET 10. However, the format appears to be incorrect and the referenced file does not exist. See:

Result:

Image

Note: TimePicker and DatePicker have a similar issue with a reference to the MVVM Community Toolkit a few lines above as well.</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.


Internal previews

📄 File 🔗 Preview link
docs/user-interface/controls/datepicker.md docs/user-interface/controls/datepicker
docs/user-interface/controls/picker.md docs/user-interface/controls/picker
docs/user-interface/controls/timepicker.md docs/user-interface/controls/timepicker

@learn-build-service-prod
Copy link

Learn Build status updates of commit 8e232d2:

✅ Validation status: passed

For more details, please refer to the build report.

@learn-build-service-prod
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 and severity-2 issues. 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.

@learn-build-service-prod
Copy link

Learn Build status updates of commit f5800b3:

✅ Validation status: passed

File Status Preview URL Details
docs/user-interface/controls/datepicker.md ✅Succeeded View (>=net-maui-8.0)
docs/user-interface/controls/picker.md ✅Succeeded View (>=net-maui-8.0)
docs/user-interface/controls/timepicker.md ✅Succeeded View (>=net-maui-8.0)

For more details, please refer to the build report.

Copilot AI changed the title [WIP] Fix incorrect includes for Picker, DatePicker, and TimePicker Fix include syntax and paths in Picker, DatePicker, and TimePicker docs Dec 15, 2025
Copilot AI requested a review from jfversluis December 15, 2025 11:43
@jfversluis jfversluis marked this pull request as ready for review December 15, 2025 11:44
@jfversluis jfversluis requested a review from a team as a code owner December 15, 2025 11:44
@jfversluis jfversluis merged commit 8212001 into main Dec 15, 2025
6 checks passed
@jfversluis jfversluis deleted the copilot/fix-picker-includes-issue branch December 15, 2025 16:27
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.

Picker, DatePicker, and TimePicker have incorrect includes

3 participants