-
Notifications
You must be signed in to change notification settings - Fork 239
Remove unnecessary Shell.FlyoutBehavior="Disabled" from tutorial AppShell files #3116
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
base: main
Are you sure you want to change the base?
Conversation
|
Learn Build status updates of commit 22b7f2f: ✅ Validation status: passedFor more details, please refer to the build report. |
Co-authored-by: PureWeen <[email protected]>
PoliCheck Scan ReportThe 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 foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit f119b60: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe 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 foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit f119b60: ✅ Validation status: passed
For more details, please refer to the build report. |
Shell.FlyoutBehavior="Disabled"is redundant when usingTabBarand causes confusion when users later try to switch toFlyoutItems—the flyout menu silently won't appear.Changes
Shell.FlyoutBehavior="Disabled"from all tutorial AppShell.xaml snippets:notes-app/shell,note,navigation,allnotesnotes-mvvm/model,viewmodel-shared,bugsnotes-app.mdBefore
After
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.