Skip to content

Update main-thread.md #2969

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

Merged
merged 2 commits into from
Aug 5, 2025
Merged

Update main-thread.md #2969

merged 2 commits into from
Aug 5, 2025

Conversation

JohanSmarius
Copy link
Contributor

Fix #2384
No highlight applied as in the request, but changed the sentence to make the point more clear. Highlight are not that often used in the docs.

Fix dotnet#2384
No highlight applied as in the request, but changed the sentence to make the point more clear.
@Copilot Copilot AI review requested due to automatic review settings August 4, 2025 20:51
@JohanSmarius JohanSmarius requested a review from a team as a code owner August 4, 2025 20:51
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 PR addresses issue #2384 by improving the clarity of documentation text in the main-thread.md file. The change makes it clearer that the thread check being discussed is specific to a particular scenario.

  • Adds "For this scenario," to the beginning of a sentence to provide better context

Copy link

Learn Build status updates of commit dc7b21e:

✅ Validation status: passed

File Status Preview URL Details
docs/platform-integration/appmodel/main-thread.md ✅Succeeded View (>=net-maui-8.0)

For more details, please refer to the build report.

Copy link

Learn Build status updates of commit 3e7a44f:

✅ Validation status: passed

File Status Preview URL Details
docs/platform-integration/appmodel/main-thread.md ✅Succeeded View (>=net-maui-8.0)

For more details, please refer to the build report.

@jfversluis jfversluis merged commit c0b25a8 into dotnet:main Aug 5, 2025
5 checks passed
jfversluis added a commit that referenced this pull request Aug 5, 2025
* Update webview.md (#2970)

* Update webview.md

Fix #2931
Reformatted the JS code to make it a bit more readable as requested. Added rounded brackets as well to better show the scope of the if statement.

* Update docs/user-interface/controls/webview.md

Co-authored-by: Copilot <[email protected]>

---------

Co-authored-by: Gerald Versluis <[email protected]>
Co-authored-by: Copilot <[email protected]>

* Update main-thread.md (#2969)

Fix #2384
No highlight applied as in the request, but changed the sentence to make the point more clear.

Co-authored-by: Gerald Versluis <[email protected]>

---------

Co-authored-by: Johan Smarius <[email protected]>
Co-authored-by: Gerald Versluis <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Gerald Versluis <[email protected]>
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.

Highlight certain information
2 participants