toeverything/AFFiNE
1
1
Fork 0
mirror of https://github.com/toeverything/AFFiNE.git synced 2024-11-25 11:53:46 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix: add prefer-dom-node-remove rule (#5112)

Browse Source
This commit is contained in:
LongYinan 2023-11-29 04:43:50 +00:00
parent 45690c2756
commit 4dddbbdaee
No known key found for this signature in database
GPG Key ID: 30B1140CE1C07C99
6 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
.eslintrc.js
View File

@ -209,6 +209,7 @@ const config = {
'unicorn/no-useless-fallback-in-spread': 'error',
'unicorn/prefer-dom-node-dataset': 'error',
'unicorn/prefer-dom-node-append': 'error',
'unicorn/prefer-dom-node-remove': 'error',
'unicorn/prefer-array-some': 'error',
'unicorn/no-useless-promise-resolve-reject': 'error',
'sonarjs/no-all-duplicated-branches': 'error',

2
packages/frontend/component/src/components/block-hub/index.tsx
View File

@ -10,7 +10,7 @@ export const RootBlockHub = () => {
const div = ref.current;
if (blockHub) {
if (div.hasChildNodes()) {
div.removeChild(div.firstChild as ChildNode);
(div.firstChild as ChildNode).remove();
}
div.append(blockHub);
}

2
packages/frontend/component/src/components/block-suite-editor/index.tsx
View File

@ -136,7 +136,7 @@ const BlockSuiteEditorImpl = ({
}
container.append(editor);
return () => {
container.removeChild(editor);
editor.remove();
};
}, [editor]);

6
packages/frontend/core/src/components/page-detail-editor.tsx
View File

@ -177,7 +177,7 @@ const EditorWrapper = memo(function EditorWrapper({
document.body.append(div);
return () => {
cleanup();
document.body.removeChild(div);
div.remove();
};
});
});
@ -238,11 +238,11 @@ const PluginContentAdapter = memo<PluginContentAdapterProps>(
root.append(div);
if (abortController.signal.aborted) {
cleanup();
root.removeChild(div);
div.remove();
} else {
const cl = () => {
cleanup();
root.removeChild(div);
div.remove();
};
const dispose = addCleanup(pluginName, cl);
abortController.signal.addEventListener('abort', () => {

2
packages/frontend/core/src/components/pure/plugin-header/index.tsx
View File

@ -30,7 +30,7 @@ export const PluginHeader = () => {
pluginsRef.current = pluginsRef.current.filter(
name => name !== pluginName
);
root.removeChild(div);
div.remove();
cleanup();
});
});

2
packages/plugins/image-preview/src/component/index.tsx
View File

@ -233,7 +233,7 @@ const ImagePreviewModalImpl = (
a.download = block.id ?? 'image';
document.body.append(a);
a.click();
document.body.removeChild(a);
a.remove();
}
},
[props.pageId, props.workspace]