diff --git a/.changeset/ten-jobs-attack.md b/.changeset/ten-jobs-attack.md new file mode 100644 index 00000000000..4ca8a9834db --- /dev/null +++ b/.changeset/ten-jobs-attack.md @@ -0,0 +1,5 @@ +--- +"@primer/react": patch +--- + +Breadcrumbs: Fix esc button not being able to focus on the menubuttonRef diff --git a/packages/react/src/Breadcrumbs/Breadcrumbs.tsx b/packages/react/src/Breadcrumbs/Breadcrumbs.tsx index 72cf11d6301..d42022092e2 100644 --- a/packages/react/src/Breadcrumbs/Breadcrumbs.tsx +++ b/packages/react/src/Breadcrumbs/Breadcrumbs.tsx @@ -9,7 +9,6 @@ import {BoxWithFallback} from '../internal/components/BoxWithFallback' import Details from '../Details' import {ActionList} from '../ActionList' import {IconButton} from '../Button/IconButton' -import {Tooltip} from '../TooltipV2' import {KebabHorizontalIcon} from '@primer/octicons-react' import {useResizeObserver} from '../hooks/useResizeObserver' import type {ResizeObserverEntry} from '../hooks/useResizeObserver' @@ -52,7 +51,7 @@ const BreadcrumbsMenuItem = React.forwardRef { const [isOpen, setIsOpen] = useState(false) const detailsRef = useRef(null) - const menuButtonRef = useRef(null) + const menuButtonRef = useRef(null) const menuContainerRef = useRef(null) const detailsRefCallback = useCallback( (element: HTMLDetailsElement | null) => { @@ -104,20 +103,19 @@ const BreadcrumbsMenuItem = React.forwardRef - - - +
{items.map((item, index) => {