Skip to content

Conversation

BitRyuu
Copy link
Contributor

@BitRyuu BitRyuu commented Sep 8, 2025

Summary

This PR fixes the MarkdownContent example by removing a dangling path = line.

Details

  • The current docs contain a path = line in the constructor that will not compile.
  • ContentPage does not define a path property, so the line is invalid.
  • The example already uses doc_path properly with OpenUrlCommand.

Fix

  • Removed the invalid line.
  • Verified that the example now compiles cleanly.

docs: fix MarkdownContent sample by removing invalid `path =` line

The sample currently contains a dangling `path =` statement that causes
compilation errors. This property does not exist on ContentPage, and the
example already uses `doc_path` correctly in OpenUrlCommand.
Copy link
Contributor

@BitRyuu : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

Copy link
Contributor

@BitRyuu : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

Copy link
Contributor

Learn Build status updates of commit 63fe2f0:

✅ Validation status: passed

File Status Preview URL Details
hub/powertoys/command-palette/using-markdown-content.md ✅Succeeded

For more details, please refer to the build report.

@alvinashcraft
Copy link
Contributor

Thanks for the contribution. You're right that this looks like the line was mistakenly added.

@alvinashcraft alvinashcraft merged commit 6e513d5 into MicrosoftDocs:docs Sep 8, 2025
2 checks passed
@BitRyuu BitRyuu deleted the patch-1 branch September 8, 2025 15:32
desigrit pushed a commit to desigrit/windows-dev-docs that referenced this pull request Sep 8, 2025
…1326052378sync_temp

For protected branch, push strategy should use PR and merge to target branch method to work around git push error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants