Skip to content

Commit

Permalink
fix (Tearsheet): update portalTarget PropType from node to a DOM Elem…
Browse files Browse the repository at this point in the history
…ent (#3509)
  • Loading branch information
baraaabdelsalam committed Oct 10, 2023
1 parent e9f938f commit 6f9280a
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions packages/ibm-products/src/components/Tearsheet/Tearsheet.js
Original file line number Diff line number Diff line change
Expand Up @@ -204,9 +204,9 @@ Tearsheet.propTypes = {
open: PropTypes.bool,

/**
* The DOM node the tearsheet should be rendered within. Defaults to document.body.
* The DOM element that the tearsheet should be rendered within. Defaults to document.body.
*/
portalTarget: PropTypes.node,
portalTarget: PropTypes.instanceOf(Element),

/**
* Specify a CSS selector that matches the DOM element that should be focused when the Modal opens
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -166,9 +166,9 @@ TearsheetNarrow.propTypes = {
open: PropTypes.bool,

/**
* The DOM node the tearsheet should be rendered within. Defaults to document.body.
* The DOM element that the tearsheet should be rendered within. Defaults to document.body.
*/
portalTarget: PropTypes.node,
portalTarget: PropTypes.instanceOf(Element),

/**
* The main title of the tearsheet, displayed in the header area.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -476,9 +476,9 @@ TearsheetShell.propTypes = {
open: PropTypes.bool,

/**
* The DOM node the tearsheet should be rendered within. Defaults to document.body.
* The DOM element that the tearsheet should be rendered within. Defaults to document.body.
*/
portalTarget: PropTypes.node,
portalTarget: PropTypes.instanceOf(Element),

/**
* Specifies the width of the tearsheet, 'narrow' or 'wide'.
Expand Down

0 comments on commit 6f9280a

Please sign in to comment.