From eb3f5e367b647d93dedf4ae448c11d88bf6b1167 Mon Sep 17 00:00:00 2001 From: Gregory Douglas Date: Tue, 14 Jan 2025 14:20:57 -0500 Subject: [PATCH] Add comments referencing issue See: https://github.com/palantir/blueprint/issues/7165 --- .../core/src/components/context-menu/contextMenuSingleton.tsx | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/packages/core/src/components/context-menu/contextMenuSingleton.tsx b/packages/core/src/components/context-menu/contextMenuSingleton.tsx index c1fd6c2005..620473acfd 100644 --- a/packages/core/src/components/context-menu/contextMenuSingleton.tsx +++ b/packages/core/src/components/context-menu/contextMenuSingleton.tsx @@ -50,8 +50,10 @@ export function showContextMenu( ) { const { container = document.body, + // TODO(React 18): Replace deprecated ReactDOM methods. See: https://github.com/palantir/blueprint/issues/7165 // eslint-disable-next-line deprecation/deprecation domRenderer = ReactDOM.render, + // TODO(React 18): Replace deprecated ReactDOM methods. See: https://github.com/palantir/blueprint/issues/7165 // eslint-disable-next-line deprecation/deprecation domUnmounter = ReactDOM.unmountComponentAtNode, } = options; @@ -67,6 +69,7 @@ export function showContextMenu( domUnmounter(contextMenuElement); } + // TODO(React 18): Replace deprecated ReactDOM methods. See: https://github.com/palantir/blueprint/issues/7165 // eslint-disable-next-line deprecation/deprecation domRenderer( @@ -84,6 +87,7 @@ export function showContextMenu( * @see https://blueprintjs.com/docs/#core/components/context-menu-popover.imperative-api */ export function hideContextMenu(options: DOMMountOptions = {}) { + // TODO(React 18): Replace deprecated ReactDOM methods. See: https://github.com/palantir/blueprint/issues/7165 // eslint-disable-next-line deprecation/deprecation const { domUnmounter = ReactDOM.unmountComponentAtNode } = options;