Add comments referencing issue

See: #7165
palantir · Jan 14, 2025 · eb3f5e3 · eb3f5e3
1 parent 57f0a6e
commit eb3f5e3
Showing 1 changed file with 4 additions and 0 deletions.
diff --git 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(
         <OverlaysProvider>
@@ -84,6 +87,7 @@ export function showContextMenu(
  * @see https://blueprintjs.com/docs/#core/components/context-menu-popover.imperative-api
  */
 export function hideContextMenu(options: DOMMountOptions<ContextMenuPopoverProps> = {}) {
+    // 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;