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

Add NavigationManager.OnLocationChanging event (#15428) #15432

Closed
wants to merge 1 commit into from
Closed

Add NavigationManager.OnLocationChanging event (#15428) #15432

wants to merge 1 commit into from

Conversation

mrpmorris
Copy link

  • Added NavigationManager.OnLocationChanging
  • Enable navigation prevention via OnLocationChangingEventArgs.PreventNavigation()

Addresses #15428

@mrpmorris mrpmorris changed the title Add NavigationManager.OnLocationChanging to enable preventing navigation (#15428) Add NavigationManager.OnLocationChanging event (#15428) Oct 26, 2019
@mrpmorris mrpmorris closed this Oct 27, 2019
@ShaunCurtis
Copy link

Has this actually been implemented? I can't find PreventNavigation() anywhere.

@mrpmorris
Copy link
Author

No.

@ShaunCurtis
Copy link

NP. I've been working on some solutions to the navigation issues of SPAs. Will post them when I have them in a presentable form

@cs-sketch
Copy link

Is this been released?
couldn't find in any of the versions of Microsoft.AspNetCore.Components package.

@mrpmorris
Copy link
Author

There's a better PR, but it's marked as a candidate for v6.

#24417

@nelson-b
Copy link

Is it implemented? If yes then let us know the details
C# version
.Net core Version
Syncfusion blazor version

Also let us know any other details

@ghost
Copy link

ghost commented Nov 26, 2021

Hi @nelson-b. It looks like you just commented on a closed PR. The team will most probably miss it. If you'd like to bring something important up to their attention, consider filing a new issue and add enough details to build context.

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.

4 participants