Skip to content

Commit

Permalink
fix(tearsheet): Firefox focuses Tearsheet content div with scroll (ca…
Browse files Browse the repository at this point in the history
…rbon-design-system#5973)

* fix(tearsheet): provide tabindex -1 for content div

* chore(Wrap): include tabIndex in props interface
  • Loading branch information
makafsal authored Sep 11, 2024
1 parent baf0ec2 commit 19e319b
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -550,6 +550,7 @@ export const TearsheetShell = React.forwardRef(
alwaysRender={
!!(influencer && influencerPosition === 'right')
}
tabIndex={-1}
>
{children}
</Wrap>
Expand Down
5 changes: 5 additions & 0 deletions packages/ibm-products/src/global/js/utils/Wrap.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,11 @@ interface WrapProps extends PropsWithChildren {
neverRender?: boolean;

className?: string;

/**
* Tab index for the wrapper div
*/
tabIndex?: number;
}

/**
Expand Down

0 comments on commit 19e319b

Please sign in to comment.