fix: smart edit bugs (#1911)

This commit is contained in:
Lucas.Xu 2023-03-02 19:34:08 +08:00 committed by GitHub
parent a1a5675875
commit fd41459a30
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 53 additions and 38 deletions

View File

@ -177,7 +177,9 @@ class _AppFlowyEditorPageState extends State<_AppFlowyEditorPage> {
]
],
toolbarItems: [
if (openAIKey != null && openAIKey!.isNotEmpty) ...[
smartEditItem,
]
],
themeData: theme.copyWith(extensions: [
...theme.extensions.values,

View File

@ -42,11 +42,13 @@ class _SmartEditWidgetState extends State<_SmartEditWidget> {
.toList(),
buildChild: (controller) {
return FlowyIconButton(
hoverColor: Colors.transparent,
tooltipText: 'Smart Edit',
preferBelow: false,
icon: const Icon(
Icons.edit,
size: 14,
Icons.lightbulb_outline,
size: 13,
color: Colors.white,
),
onPressed: () {
controller.show();
@ -55,6 +57,13 @@ class _SmartEditWidgetState extends State<_SmartEditWidget> {
},
onSelected: (action, controller) {
controller.close();
_insertSmartEditNode(action);
},
);
}
Future<void> _insertSmartEditNode(
SmartEditActionWrapper actionWrapper) async {
final selection =
widget.editorState.service.selectionService.currentSelection.value;
if (selection == null) {
@ -74,12 +83,12 @@ class _SmartEditWidgetState extends State<_SmartEditWidget> {
Node(
type: kSmartEditType,
attributes: {
kSmartEditInstructionType: action.inner.toInstruction,
kSmartEditInstructionType: actionWrapper.inner.toInstruction,
kSmartEditInputType: input,
},
),
);
widget.editorState.apply(
return widget.editorState.apply(
transaction,
options: const ApplyOptions(
recordUndo: false,
@ -87,7 +96,5 @@ class _SmartEditWidgetState extends State<_SmartEditWidget> {
),
withUpdateCursor: false,
);
},
);
}
}

View File

@ -59,12 +59,18 @@ extension CommandExtension on EditorState {
List<String> res = [];
if (!selection.isCollapsed) {
for (var i = 0; i < textNodes.length; i++) {
final plainText = textNodes[i].toPlainText();
if (i == 0) {
res.add(textNodes[i].toPlainText().substring(selection.startIndex));
res.add(
plainText.substring(
selection.startIndex,
plainText.length,
),
);
} else if (i == textNodes.length - 1) {
res.add(textNodes[i].toPlainText().substring(0, selection.endIndex));
res.add(plainText.substring(0, selection.endIndex));
} else {
res.add(textNodes[i].toPlainText());
res.add(plainText);
}
}
}