Skip to content
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

docs: Update Hot Reload documentation based on latest changes #19250

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

agneszitte
Copy link
Contributor

PR Type

What kind of change does this PR introduce?

  • Documentation content changes

Description

Update Hot Reload documentation based on latest changes

PR Checklist

Please check if your PR fulfills the following requirements:

Mobile targets now support both XAML and C# Hot Reload. Debugger-specific variations apply depending on the IDE.

- In Visual Studio:
- The "Hot Reload on File Save" feature must be disabled to avoid crashing the app. You can find this feature by clicking on the down arrow next to the red flame button in the Visual Studio toolbar.
Copy link
Member

Choose a reason for hiding this comment

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

Never use this ... but not aware it could crash the app. Is this something you experienced?

Copy link
Contributor Author

@agneszitte agneszitte Jan 16, 2025

Choose a reason for hiding this comment

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

@dr1rrb Yes it was added a long time ago in the doc as in the past you would experience some crashes.
@jeromelaban lately I am not sure if the Hot Reload on File Save option is still something that should be disabled in some specific cases for Visual Studio?

doc/articles/features/working-with-xaml-hot-reload.md Outdated Show resolved Hide resolved
Co-authored-by: David <github.site@torick.net>
@unodevops
Copy link
Contributor

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-19250/index.html

<td>⌛<sup><a href="#hr-footnotes">[3]</a></sup>🛜</td><td>✅<sup><a href="#hr-footnotes">[4]</a></sup>🛜</td>
<td>🔳</td><td>🔳</td>
<td>✅<sup><a href="#hr-footnotes">[4]</a></sup></td>
<td>🔳</td>
Copy link
Member

Choose a reason for hiding this comment

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

We should have another icon for this case: it's not that this target is not supported, it's only that HR does not work in that configuration.

BTW please keep the alignment of 2 table columns per code code row so one code row is still one "main case" while the two table columns are only with or without debugger

@unodevops
Copy link
Contributor

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-19250/index.html

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.

4 participants