Blazored / TextEditor

Rich text editor for Blazor applications - Uses Quill JS
MIT License
277 stars 61 forks source link

[Bug] [Deprecation] Listener added for a synchronous 'DOMNodeInserted' DOM Mutation Event. #72

Closed Markus243 closed 6 months ago

Markus243 commented 7 months ago

Describe the bug A clear and concise description of what the bug is. Using the cdn's 1.3.7 as per the documentation I encounter: [Deprecation] Listener added for a synchronous 'DOMNodeInserted' DOM Mutation Event. This event type is deprecated () and work is underway to remove it from this browser. Usage of this event listener will cause performance issues today, and represents a risk of future incompatibility. Consider using MutationObserver instead.

To Reproduce Steps to reproduce the behavior:

  1. Follow the documentation for setup
  2. Browser warns: [Deprecation] Listener added for a synchronous 'DOMNodeInserted' DOM Mutation Event. This event type is deprecated () and work is underway to remove it from this browser. Usage of this event listener will cause performance issues today, and represents a risk of future incompatibility. Consider using MutationObserver instead.

Hosting Model (is this issue happening with a certain hosting model?):

Additional context I tried updating to later versions of QuillJs >2.00 but this seems to break compatibility with blazored texteditor. Any help/solutions or workarounds would be greatly appreciated.

ADefWebserver commented 7 months ago

I am not sure if this component should be upgraded. Perhaps a new component should be created instead. However, what issues did you see when you updated to Quill 2.00?

markushaslinger commented 6 months ago

However, what issues did you see when you updated to Quill 2.00?

A simple replacement of the js & css files to current quill 2.0.1 gave me the following error:

Cannot read properties of null (reading '__quill')
    at Object.loadQuillHTMLContent (http://localhost:5204/_content/Blazored.TextEditor/Blazored-BlazorQuill.js:40:33)
ADefWebserver commented 6 months ago

@markushaslinger A new Blazor component would need to be created from scratch to work with Quill 2.0

ADefWebserver commented 6 months ago

@markushaslinger A new Blazor component would need to be created from scratch to work with Quill 2.0