Fix focus trap issue in sankey chart (#31310)

microsoft · May 14, 2024 · bf854a4 · bf854a4
1 parent aa5c395
commit bf854a4
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/change/@fluentui-react-charting-4e7f95fa-3196-4b13-84bd-e646c674683c.json b/change/@fluentui-react-charting-4e7f95fa-3196-4b13-84bd-e646c674683c.json
@@ -0,0 +1,7 @@
+{
+ "type": "patch",
+ "comment": "Fix focus trap issue in sankey chart",
+ "packageName": "@fluentui/react-charting",
+ "email": "[email protected]",
+ "dependentChangeType": "patch"
+}
diff --git a/packages/react-charting/src/components/SankeyChart/SankeyChart.base.tsx b/packages/react-charting/src/components/SankeyChart/SankeyChart.base.tsx
@@ -1,4 +1,4 @@
-import { FocusZone, FocusZoneDirection, FocusZoneTabbableElements } from '@fluentui/react-focus';
+import { FocusZone, FocusZoneDirection } from '@fluentui/react-focus';
 import { Callout, DirectionalHint } from '@fluentui/react/lib/Callout';
 import { IProcessedStyleSet, ITheme } from '@fluentui/react/lib/Styling';
 import {
@@ -792,11 +792,7 @@ export class SankeyChartBase extends React.Component<ISankeyChartProps, ISankeyC
  ref={(rootElem: HTMLDivElement) => (this.chartContainer = rootElem)}
  onMouseLeave={this._onCloseCallout}
  >
- <FocusZone
- direction={FocusZoneDirection.bidirectional}
- isCircularNavigation={true}
- handleTabKey={FocusZoneTabbableElements.all}
- >
+ <FocusZone direction={FocusZoneDirection.bidirectional}>
  <svg width={width} height={height} id={this._chartId}>
  <g className={classNames.links} strokeOpacity={1}>
  {linkData}