Skip to content
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

@coryhouse

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
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions