AppFlowy-IO / AppFlowy-IO/appflowy-editor

[Bug] ValueNotifier disposed exception when reopening page with shrinkWrap=true EditorScrollController

Ouverte
#1,151 0 commentaires 1 réaction 0 personnes assignées Voir sur GitHub
Langage dominant
Dart
Étoiles
684
Forks
329
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

### Bug Description

When using AppFlowyEditor with EditorScrollController(shrinkWrap: true) and reusing the same EditorState instance, the app throws a “ValueNotifier was used after being disposed” exception after navigating away and reopening the page.

This happens because EditorScrollController adds a listener to editorState.document.root when shrinkWrap is enabled, but never removes it in dispose().
When the editor is rebuilt or reopened, the stale listener still tries to update the disposed visibleRangeNotifier or offsetNotifier, causing the crash.

### How to Reproduce

Create an editor page that uses a persistent EditorState:

final editorState = EditorState.blank(withInitialText: true);

Inside your page, use:

AppFlowyEditor(
editorState: editorState,
shrinkWrap: true,
)

Navigate away from the page (so the widget and controller are disposed).

Navigate back to the same page using the same editorState.

Try typing or editing text — the exception appears:

A ValueNotifier was used after being disposed.

### Expected Behavior

The editor should work normally when reopened, even when reusing the same EditorState.
No exceptions should occur from disposed notifiers or stale listeners.

### Operating System

IOS, Android

### AppFlowy Editor Version(s)

5.2.0

### Screenshots

_No response_

### Additional Context

_No response_

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.