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

[Windows]Fixed Shell Navigating event issue when switching tabs #27197

Merged
merged 2 commits into from
Feb 4, 2025

Conversation

Vignesh-SF3580
Copy link
Contributor

Issue Detail

The Navigating event was not triggered when switching tabs.

Root Cause

There was no code in place to trigger the Navigating event upon tab switches.

Description of Change

Added logic to trigger Navigating when switching to a different tab.

Tested the behavior in the following platforms

  • Android
  • Windows
  • iOS
  • Mac

Issues Fixed

Fixes #25599

Screenshots

Before Issue Fix After Issue Fix
BeforeFix.mp4
AfterFix1.mp4

@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Jan 17, 2025
Copy link
Contributor

Hey there @Vignesh-SF3580! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@karthikraja-arumugam karthikraja-arumugam added the partner/syncfusion Issues / PR's with Syncfusion collaboration label Jan 17, 2025
@jsuarezruiz
Copy link
Contributor

/azp run

This comment was marked as outdated.

@jsuarezruiz jsuarezruiz added platform/windows 🪟 area-controls-shell Shell Navigation, Routes, Tabs, Flyout labels Jan 17, 2025
@Vignesh-SF3580 Vignesh-SF3580 marked this pull request as ready for review January 17, 2025 12:06
@Vignesh-SF3580 Vignesh-SF3580 requested a review from a team as a code owner January 17, 2025 12:06
@jsuarezruiz

This comment was marked as outdated.

@jsuarezruiz
Copy link
Contributor

jsuarezruiz commented Jan 20, 2025

/azp run

This comment was marked as off-topic.

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@jsuarezruiz
Copy link
Contributor

/rebase

@jsuarezruiz
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@Foda
Copy link
Member

Foda commented Jan 30, 2025

For future reference: you could have written a unit test instead that checked for the event 🙂

@jfversluis jfversluis dismissed jsuarezruiz’s stale review January 31, 2025 20:28

Feedback addressed

@jfversluis jfversluis enabled auto-merge (squash) January 31, 2025 20:28
@jfversluis jfversluis merged commit 64227b9 into dotnet:main Feb 4, 2025
316 checks passed
@jfversluis jfversluis added this to the .NET 9 SR4 milestone Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-shell Shell Navigation, Routes, Tabs, Flyout community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration platform/windows 🪟
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OnNavigating wrong target when tapping the same tab
5 participants