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

Shell BackButtonBehavior improvement #23466

Merged
merged 4 commits into from
Jul 6, 2024
Merged

Conversation

kubaflo
Copy link
Contributor

@kubaflo kubaflo commented Jul 6, 2024

Issues Fixed

Fixes #23424
Fixes #5195

IsVisible IsEnabled
Simulator.Screen.Recording.-.iPhone.15.Pro.Max.-.2024-07-06.at.13.38.33.mp4
Simulator.Screen.Recording.-.iPhone.15.Pro.Max.-.2024-07-06.at.13.51.31.mp4

@kubaflo kubaflo requested a review from a team as a code owner July 6, 2024 11:39
@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Jul 6, 2024
@kubaflo kubaflo requested a review from PureWeen July 6, 2024 11:39
@kubaflo kubaflo added platform/iOS 🍎 area-controls-shell Shell Navigation, Routes, Tabs, Flyout labels Jul 6, 2024
@kubaflo kubaflo requested review from jfversluis and removed request for mattleibow and jsuarezruiz July 6, 2024 11:52
@PureWeen
Copy link
Member

PureWeen commented Jul 6, 2024

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

{
Items.Add(new ContentPage());
Routing.RegisterRoute(nameof(DetailPage),typeof(DetailPage));
GoToAsync(nameof(DetailPage));
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we need to register a route here.

Maybe just this to keep it simple

Items.Add(new ContentPage());
		Navigation.PushAsync(new DetailPage());

@PureWeen
Copy link
Member

PureWeen commented Jul 6, 2024

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen
Copy link
Member

PureWeen commented Jul 6, 2024

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen PureWeen merged commit 30562b6 into dotnet:main Jul 6, 2024
95 of 97 checks passed
@samhouts samhouts added fixed-in-8.0.70 fixed-in-net9.0-nightly This may be available in a nightly release! labels Aug 2, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Sep 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-shell Shell Navigation, Routes, Tabs, Flyout community ✨ Community Contribution fixed-in-8.0.70 fixed-in-net9.0-nightly This may be available in a nightly release! platform/iOS 🍎
Projects
None yet
3 participants