Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 28, 2025

Shell.FlyoutBehavior="Disabled" is redundant when using TabBar and causes confusion when users later try to switch to FlyoutItems—the flyout menu silently won't appear.

Changes

  • Removed Shell.FlyoutBehavior="Disabled" from all tutorial AppShell.xaml snippets:
    • notes-app/shell, note, navigation, allnotes
    • notes-mvvm/model, viewmodel-shared, bugs
  • Updated inline code examples in notes-app.md

Before

<Shell
    xmlns:local="clr-namespace:Notes"
    Shell.FlyoutBehavior="Disabled">
    <TabBar>

After

<Shell
    xmlns:local="clr-namespace:Notes">
    <TabBar>
Original prompt

This section details on the original issue you should resolve

<issue_title>Modify the app Shell portion of tutorial doesn't need to have Shell.FlyoutBehavior="Disabled"</issue_title>
<issue_description>## Issue description

We used to have this in our templates as well

image

But we've removed it because it isn't needed if you've specified "Tabbar". This can cause confusion when users try to switch to FlyoutItems because the flyout menu won't show up, so it's best to just leave it off :-)
</issue_description>

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


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@learn-build-service-prod
Copy link

Learn Build status updates of commit 22b7f2f:

✅ 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 f119b60:

✅ Validation status: passed

File Status Preview URL Details
docs/tutorials/notes-app.md ✅Succeeded View (>=net-maui-8.0)
docs/tutorials/snippets/notes-app/allnotes/AppShell.xaml ✅Succeeded View (>=net-maui-8.0)
docs/tutorials/snippets/notes-app/navigation/AppShell.xaml ✅Succeeded
docs/tutorials/snippets/notes-app/note/AppShell.xaml ✅Succeeded View (>=net-maui-8.0)
docs/tutorials/snippets/notes-app/shell/AppShell.xaml ✅Succeeded View (>=net-maui-8.0)
docs/tutorials/snippets/notes-mvvm/bugs/AppShell.xaml ✅Succeeded
docs/tutorials/snippets/notes-mvvm/model/AppShell.xaml ✅Succeeded
docs/tutorials/snippets/notes-mvvm/viewmodel-shared/AppShell.xaml ✅Succeeded

For more details, please refer to the build report.

Copilot AI changed the title [WIP] Remove Shell.FlyoutBehavior from app Shell Remove unnecessary Shell.FlyoutBehavior="Disabled" from tutorial AppShell files Nov 28, 2025
Copilot finished work on behalf of PureWeen November 28, 2025 22:33
Copilot AI requested a review from PureWeen November 28, 2025 22:33
@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 f119b60:

✅ Validation status: passed

File Status Preview URL Details
docs/tutorials/notes-app.md ✅Succeeded View (>=net-maui-8.0)
docs/tutorials/snippets/notes-app/allnotes/AppShell.xaml ✅Succeeded View (>=net-maui-8.0)
docs/tutorials/snippets/notes-app/navigation/AppShell.xaml ✅Succeeded
docs/tutorials/snippets/notes-app/note/AppShell.xaml ✅Succeeded View (>=net-maui-8.0)
docs/tutorials/snippets/notes-app/shell/AppShell.xaml ✅Succeeded View (>=net-maui-8.0)
docs/tutorials/snippets/notes-mvvm/bugs/AppShell.xaml ✅Succeeded
docs/tutorials/snippets/notes-mvvm/model/AppShell.xaml ✅Succeeded
docs/tutorials/snippets/notes-mvvm/viewmodel-shared/AppShell.xaml ✅Succeeded

For more details, please refer to the build report.

@PureWeen PureWeen marked this pull request as ready for review November 28, 2025 23:47
@PureWeen PureWeen requested a review from a team as a code owner November 28, 2025 23:47
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.

Modify the app Shell portion of tutorial doesn't need to have Shell.FlyoutBehavior="Disabled"

2 participants