[core] Handle tab-index as a number

mui · Jan 3, 2021 · f973ebf · f973ebf
1 parent 138161d
commit f973ebf
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/docs/pages/index.js b/docs/pages/index.js
@@ -126,7 +126,7 @@ export default function LandingPage(props) {
     <AppFrame>
       <div className={classes.root}>
         <Head />
-        <main id="main-content" tabIndex="-1">
+        <main id="main-content" tabIndex={-1}>
           <div className={classes.hero}>
             <Container maxWidth="md" className={classes.content}>
               <img src="/static/logo_raw.svg" alt="" className={classes.logo} />

diff --git a/packages/material-ui/src/Breadcrumbs/Breadcrumbs.test.js b/packages/material-ui/src/Breadcrumbs/Breadcrumbs.test.js
@@ -68,7 +68,7 @@ describe('<Breadcrumbs />', () => {
   it('should expand when `BreadcrumbCollapsed` is clicked', () => {
     const { getAllByRole, getByRole, getByText } = render(
       <Breadcrumbs>
-        <span tabIndex="-1">first</span>
+        <span tabIndex={-1}>first</span>
         <span>second</span>
         <span>third</span>
         <span>fourth</span>

diff --git a/packages/material-ui/src/Popover/Popover.test.js b/packages/material-ui/src/Popover/Popover.test.js
@@ -813,7 +813,7 @@ describe('<Popover />', () => {
     });
 
     it('should not apply the auto prop if not supported', () => {
-      const TransitionComponent = React.forwardRef((_, ref) => <div ref={ref} tabIndex="-1" />);
+      const TransitionComponent = React.forwardRef((_, ref) => <div ref={ref} tabIndex={-1} />);
       const wrapper = mount(
         <Popover {...defaultProps} open TransitionComponent={TransitionComponent}>
           <div />