Skip to content
This repository was archived by the owner on Feb 16, 2026. It is now read-only.
This repository was archived by the owner on Feb 16, 2026. It is now read-only.

The target prop should be optional in the TypeScript type definition#972

Description

@coryhouse

Hi! 馃憢

Firstly, thanks for your work on this project! 馃檪

Today I used patch-package to patch react-overlays@5.1.1 for the project I'm working on.

Per the docs, the target prop should be optional. But TypeScript is requiring it.

Here is the diff that solved my problem:

diff --git a/node_modules/react-overlays/esm/Overlay.d.ts b/node_modules/react-overlays/esm/Overlay.d.ts
index 2a06425..ee42e82 100644
--- a/node_modules/react-overlays/esm/Overlay.d.ts
+++ b/node_modules/react-overlays/esm/Overlay.d.ts
@@ -10,7 +10,7 @@ export interface OverlayProps extends TransitionCallbacks {
     containerPadding?: number;
     popperConfig?: Omit<UsePopperOptions, 'placement'>;
     container?: DOMContainer;
-    target: DOMContainer;
+    target?: DOMContainer;
     show?: boolean;
     transition?: React.ComponentType<{
         in?: boolean;

This issue body was partially generated by patch-package.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions