eduvia/workavia-mail-front
11
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Mutualize event listener in web editor widget

Browse Source 
Instead of listening multiple times for message from editor JS side,
let's add only one listener and route inside the callback.
This commit is contained in:
Théo Poizat
2025-12-11 11:03:23 +01:00
committed by Dat H. Pham
parent 75f5817e1c
commit 2228596c1c
1 changed files with 14 additions and 32 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/features/composer/presentation/widgets/web/web_editor_widget.dart
+14 -32
View File
@@ -83,10 +83,8 @@ class _WebEditorState extends State<WebEditorWidget> with TextSelectionMixin {
static const double _defaultHtmlEditorHeight = 550;
late HtmlEditorController _editorController;
bool _dropListenerRegistered = false;
bool _selectionChangeListenerRegistered = false;
Function(Event)? _dropListener;
Function(Event)? _selectionChangeListener;
bool _editorListenerRegistered = false;
Function(Event)? _editorListener;
OverlayEntry? _signatureTooltipEntry;
final GlobalKey _signatureTooltipKey = GlobalKey();
@@ -101,34 +99,25 @@ class _WebEditorState extends State<WebEditorWidget> with TextSelectionMixin {
super.initState();
_editorController = widget.editorController;
_dropListener = (event) {
if (event is MessageEvent) {
if (jsonDecode(event.data)['name'] == HtmlUtils.registerDropListener.name) {
_editorController.evaluateJavascriptWeb(HtmlUtils.removeLineHeight1px.name);
}
}
};
if (_dropListener != null) {
window.addEventListener("message", _dropListener!);
}
_selectionChangeListener = (event) {
_editorListener = (event) {
try {
if (event is MessageEvent) {
final data = jsonDecode(event.data);
if (data['name'] == HtmlUtils.registerSelectionChangeListener.name) {
if (data['name'] == HtmlUtils.registerDropListener.name) {
_editorController.evaluateJavascriptWeb(HtmlUtils.removeLineHeight1px.name);
} else if (data['name'] == HtmlUtils.registerSelectionChangeListener.name) {
handleSelectionChange(data);
}
}
} catch (e) {
logError(
'_WebEditorState::_selectionChangeListener: Unable to parse selection data = $e',
'_WebEditorState::_editorListener: Unable to parse message data = $e',
);
}
};
if (_selectionChangeListener != null) {
window.addEventListener("message", _selectionChangeListener!);
if (_editorListener != null) {
window.addEventListener("message", _editorListener!);
}
}
@@ -145,13 +134,9 @@ class _WebEditorState extends State<WebEditorWidget> with TextSelectionMixin {
void dispose() {
_editorController.evaluateJavascriptWeb(
HtmlUtils.unregisterDropListener.name);
if (_dropListener != null) {
window.removeEventListener("message", _dropListener!);
_dropListener = null;
}
if (_selectionChangeListener != null) {
window.removeEventListener("message", _selectionChangeListener!);
_selectionChangeListener = null;
if (_editorListener != null) {
window.removeEventListener("message", _editorListener!);
_editorListener = null;
}
_hideSignatureTooltip();
super.dispose();
@@ -216,15 +201,12 @@ class _WebEditorState extends State<WebEditorWidget> with TextSelectionMixin {
onChangeContent: widget.onChangeContent,
onInit: () {
widget.onInitial?.call(widget.content);
if (!_dropListenerRegistered) {
if (!_editorListenerRegistered) {
_editorController.evaluateJavascriptWeb(
HtmlUtils.registerDropListener.name);
_dropListenerRegistered = true;
}
if (!_selectionChangeListenerRegistered) {
_editorController.evaluateJavascriptWeb(
HtmlUtils.registerSelectionChangeListener.name);
_selectionChangeListenerRegistered = true;
_editorListenerRegistered = true;
}
},
onFocus: widget.onFocus,