From a89ac928464f49a7956524693b8238cb4b037d5c Mon Sep 17 00:00:00 2001 From: Dimitri POSTOLOV Date: Sat, 1 Apr 2023 01:56:50 +0200 Subject: [PATCH] fix --- .eslintrc.js | 2 +- packages/codemirror-graphql/src/utils/info-addon.ts | 4 ++-- .../explorer/components/__tests__/type-documentation.spec.tsx | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/.eslintrc.js b/.eslintrc.js index 22f1d3fdd68..4fb72c272ed 100644 --- a/.eslintrc.js +++ b/.eslintrc.js @@ -213,7 +213,7 @@ module.exports = { 'sort-vars': 0, 'spaced-comment': ['error', 'always', { markers: ['/'] }], 'wrap-regex': 0, - + 'unicorn/prefer-dom-node-remove': 'error', // ECMAScript 6 (http://eslint.org/docs/rules/#ecmascript-6) 'arrow-body-style': 0, 'no-duplicate-imports': 0, diff --git a/packages/codemirror-graphql/src/utils/info-addon.ts b/packages/codemirror-graphql/src/utils/info-addon.ts index 6fb4fe39bc3..ed145273303 100644 --- a/packages/codemirror-graphql/src/utils/info-addon.ts +++ b/packages/codemirror-graphql/src/utils/info-addon.ts @@ -169,11 +169,11 @@ function showPopup(cm: CodeMirror.Editor, box: DOMRect, info: HTMLDivElement) { popup.style.opacity = '0'; setTimeout(() => { if (popup.parentNode) { - popup.parentNode.removeChild(popup); + popup.remove(); } }, 600); } else if (popup.parentNode) { - popup.parentNode.removeChild(popup); + popup.remove(); } }; diff --git a/packages/graphiql-react/src/explorer/components/__tests__/type-documentation.spec.tsx b/packages/graphiql-react/src/explorer/components/__tests__/type-documentation.spec.tsx index 3b241f3dfad..22832067950 100644 --- a/packages/graphiql-react/src/explorer/components/__tests__/type-documentation.spec.tsx +++ b/packages/graphiql-react/src/explorer/components/__tests__/type-documentation.spec.tsx @@ -128,7 +128,7 @@ describe('TypeDocumentation', () => { const deprecatedTitle = container.querySelectorAll( '.graphiql-doc-explorer-section-title', )[1]; - deprecatedTitle.removeChild(deprecatedTitle.childNodes[0]); + deprecatedTitle.childNodes[0].remove(); expect(deprecatedTitle).toHaveTextContent('Deprecated Enum Values'); enums = container.querySelectorAll('.graphiql-doc-explorer-enum-value');