Skip to content

Commit

Permalink
[docs] Use focus-visible instead of focus for Menu demos
Browse files Browse the repository at this point in the history
  • Loading branch information
michaldudak committed Apr 11, 2023
1 parent 44748d4 commit 326dc65
Show file tree
Hide file tree
Showing 8 changed files with 18 additions and 14 deletions.
5 changes: 3 additions & 2 deletions docs/data/base/components/menu/MenuSimple.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ import MenuUnstyled from '@mui/base/MenuUnstyled';
import MenuItemUnstyled, {
menuItemUnstyledClasses,
} from '@mui/base/MenuItemUnstyled';
import ButtonUnstyled, { buttonUnstyledClasses } from '@mui/base/ButtonUnstyled';
import PopperUnstyled from '@mui/base/PopperUnstyled';
import { styled } from '@mui/system';
import { ListActionTypes } from '@mui/base/useList';
Expand Down Expand Up @@ -167,7 +168,7 @@ const StyledMenuItem = styled(MenuItemUnstyled)(
`,
);

const TriggerButton = styled('button')(
const TriggerButton = styled(ButtonUnstyled)(
({ theme }) => `
font-family: IBM Plex Sans, sans-serif;
font-size: 0.875rem;
Expand All @@ -189,7 +190,7 @@ const TriggerButton = styled('button')(
border-color: ${theme.palette.mode === 'dark' ? grey[600] : grey[300]};
}
&:focus {
&.${buttonUnstyledClasses.focusVisible} {
border-color: ${blue[400]};
outline: 3px solid ${theme.palette.mode === 'dark' ? blue[500] : blue[200]};
}
Expand Down
5 changes: 3 additions & 2 deletions docs/data/base/components/menu/MenuSimple.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ import MenuUnstyled, { MenuUnstyledActions } from '@mui/base/MenuUnstyled';
import MenuItemUnstyled, {
menuItemUnstyledClasses,
} from '@mui/base/MenuItemUnstyled';
import ButtonUnstyled, { buttonUnstyledClasses } from '@mui/base/ButtonUnstyled';
import PopperUnstyled from '@mui/base/PopperUnstyled';
import { styled } from '@mui/system';
import { ListActionTypes } from '@mui/base/useList';
Expand Down Expand Up @@ -168,7 +169,7 @@ const StyledMenuItem = styled(MenuItemUnstyled)(
`,
);

const TriggerButton = styled('button')(
const TriggerButton = styled(ButtonUnstyled)(
({ theme }) => `
font-family: IBM Plex Sans, sans-serif;
font-size: 0.875rem;
Expand All @@ -190,7 +191,7 @@ const TriggerButton = styled('button')(
border-color: ${theme.palette.mode === 'dark' ? grey[600] : grey[300]};
}
&:focus {
&.${buttonUnstyledClasses.focusVisible} {
border-color: ${blue[400]};
outline: 3px solid ${theme.palette.mode === 'dark' ? blue[500] : blue[200]};
}
Expand Down
5 changes: 3 additions & 2 deletions docs/data/base/components/menu/UnstyledMenuIntroduction.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ import MenuUnstyled from '@mui/base/MenuUnstyled';
import MenuItemUnstyled, {
menuItemUnstyledClasses,
} from '@mui/base/MenuItemUnstyled';
import ButtonUnstyled, { buttonUnstyledClasses } from '@mui/base/ButtonUnstyled';
import PopperUnstyled from '@mui/base/PopperUnstyled';
import { styled } from '@mui/system';
import { ListActionTypes } from '@mui/base/useList';
Expand Down Expand Up @@ -165,7 +166,7 @@ const StyledMenuItem = styled(MenuItemUnstyled)(
`,
);

const TriggerButton = styled('button')(
const TriggerButton = styled(ButtonUnstyled)(
({ theme }) => `
font-family: IBM Plex Sans, sans-serif;
font-size: 0.875rem;
Expand All @@ -187,7 +188,7 @@ const TriggerButton = styled('button')(
border-color: ${theme.palette.mode === 'dark' ? grey[600] : grey[300]};
}
&:focus {
&.${buttonUnstyledClasses.focusVisible} {
border-color: ${blue[400]};
outline: 3px solid ${theme.palette.mode === 'dark' ? blue[500] : blue[200]};
}
Expand Down
5 changes: 3 additions & 2 deletions docs/data/base/components/menu/UnstyledMenuIntroduction.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ import MenuUnstyled, { MenuUnstyledActions } from '@mui/base/MenuUnstyled';
import MenuItemUnstyled, {
menuItemUnstyledClasses,
} from '@mui/base/MenuItemUnstyled';
import ButtonUnstyled, { buttonUnstyledClasses } from '@mui/base/ButtonUnstyled';
import PopperUnstyled from '@mui/base/PopperUnstyled';
import { styled } from '@mui/system';
import { ListActionTypes } from '@mui/base/useList';
Expand Down Expand Up @@ -166,7 +167,7 @@ const StyledMenuItem = styled(MenuItemUnstyled)(
`,
);

const TriggerButton = styled('button')(
const TriggerButton = styled(ButtonUnstyled)(
({ theme }) => `
font-family: IBM Plex Sans, sans-serif;
font-size: 0.875rem;
Expand All @@ -188,7 +189,7 @@ const TriggerButton = styled('button')(
border-color: ${theme.palette.mode === 'dark' ? grey[600] : grey[300]};
}
&:focus {
&.${buttonUnstyledClasses.focusVisible} {
border-color: ${blue[400]};
outline: 3px solid ${theme.palette.mode === 'dark' ? blue[500] : blue[200]};
}
Expand Down
2 changes: 1 addition & 1 deletion docs/data/base/components/menu/UseMenu.js
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@ const styles = `
border-color: ${grey[300]};
}
&:focus {
&:focus-visible {
border-color: ${blue[400]};
outline: 3px solid ${blue[200]};
}
Expand Down
2 changes: 1 addition & 1 deletion docs/data/base/components/menu/UseMenu.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -236,7 +236,7 @@ const styles = `
border-color: ${grey[300]};
}
&:focus {
&:focus-visible {
border-color: ${blue[400]};
outline: 3px solid ${blue[200]};
}
Expand Down
4 changes: 2 additions & 2 deletions docs/data/base/components/menu/WrappedMenuItems.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import MenuUnstyled from '@mui/base/MenuUnstyled';
import MenuItemUnstyled, {
menuItemUnstyledClasses,
} from '@mui/base/MenuItemUnstyled';
import { buttonUnstyledClasses } from '@mui/base/ButtonUnstyled';
import ButtonUnstyled, { buttonUnstyledClasses } from '@mui/base/ButtonUnstyled';
import PopperUnstyled from '@mui/base/PopperUnstyled';
import { styled } from '@mui/system';
import { ListActionTypes } from '@mui/base/useList';
Expand Down Expand Up @@ -201,7 +201,7 @@ const StyledMenuItem = styled(MenuItemUnstyled)(
`,
);

const TriggerButton = styled('button')(
const TriggerButton = styled(ButtonUnstyled)(
({ theme }) => `
font-family: IBM Plex Sans, sans-serif;
font-size: 0.875rem;
Expand Down
4 changes: 2 additions & 2 deletions docs/data/base/components/menu/WrappedMenuItems.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import MenuUnstyled, { MenuUnstyledActions } from '@mui/base/MenuUnstyled';
import MenuItemUnstyled, {
menuItemUnstyledClasses,
} from '@mui/base/MenuItemUnstyled';
import { buttonUnstyledClasses } from '@mui/base/ButtonUnstyled';
import ButtonUnstyled, { buttonUnstyledClasses } from '@mui/base/ButtonUnstyled';
import PopperUnstyled from '@mui/base/PopperUnstyled';
import { styled } from '@mui/system';
import { ListActionTypes } from '@mui/base/useList';
Expand Down Expand Up @@ -196,7 +196,7 @@ const StyledMenuItem = styled(MenuItemUnstyled)(
`,
);

const TriggerButton = styled('button')(
const TriggerButton = styled(ButtonUnstyled)(
({ theme }) => `
font-family: IBM Plex Sans, sans-serif;
font-size: 0.875rem;
Expand Down

0 comments on commit 326dc65

Please sign in to comment.