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

Uncaught ReferenceError: MutationEvent is not defined #724

Closed
3 tasks done
dheid opened this issue Nov 4, 2024 · 2 comments
Closed
3 tasks done

Uncaught ReferenceError: MutationEvent is not defined #724

dheid opened this issue Nov 4, 2024 · 2 comments
Assignees
Labels
blazor Concerns the Blazor integration piral-blazor. bug Something isn't working
Milestone

Comments

@dheid
Copy link

dheid commented Nov 4, 2024

Bug Report

Prerequisites

  • Can you reproduce the problem in a MWE?
  • Are you running the latest version?
  • Did you perform a search in the issues?

Environment Details and Version

  • Chrome version: 130.0.6723.92 (Official Build) (64-bit)
  • Operating System: Windows 10

Description

I'm experiencing an error when using the Piral Blazor converter with my application. The JavaScript console of my Chrome browser (version 130.0.6723.92) shows a repeated exception:

Uncaught ReferenceError: MutationEvent is not defined
    at n.Vht (events.js:61:28)
    at s.invokeTask (zone.js:402:33)
    at i.runTask (zone.js:159:47)
    at c.invokeTask [as invoke] (zone.js:483:34)
    at y (zone.js:1138:18)
    at w (zone.js:1169:29)
    at n.x (zone.js:1202:16)

This error occurs over 763 times and even more frequently when I move the mouse around.

Steps to Reproduce

  1. Run your application using the Piral Blazor converter.
  2. Open the JavaScript console in Chrome (version 130.0.6723.92).
  3. Observe the repeated exception messages with Uncaught ReferenceError: MutationEvent is not defined.

Expected behavior

I expect to see no errors or exceptions when running my application using the Piral Blazor converter.

Actual behavior

There is a huge amount of errors slowing my application down.

Possible Origin/Solution

Remove the check as the MutationEvent is no longer supported by modern browsers.

@dheid dheid added the bug Something isn't working label Nov 4, 2024
@dheid dheid mentioned this issue Nov 4, 2024
8 tasks
@dheid
Copy link
Author

dheid commented Nov 4, 2024

@FlorianRappl Could you look into this, please?

@FlorianRappl FlorianRappl added this to the 1.8.0 milestone Nov 4, 2024
@FlorianRappl FlorianRappl added blazor Concerns the Blazor integration piral-blazor. in-implementation The item is currently being implemented. labels Nov 4, 2024
@FlorianRappl
Copy link
Contributor

Hi @dheid thanks for the report. Indeed the whole story around mutation events changed drastically this year (the mutation observer is out for a while - but now Chrome is aggressively pushing away the classic methods / events).

Thanks for the PR #725 - I think this is a suitable solution as our only aim is to properly forward the events (for this we use the list as found in the Blazor JS source code).

dheid pushed a commit to dheid/piral that referenced this issue Nov 4, 2024
FlorianRappl added a commit that referenced this issue Nov 4, 2024
@FlorianRappl FlorianRappl removed the in-implementation The item is currently being implemented. label Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blazor Concerns the Blazor integration piral-blazor. bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants