Skip to content

Commit

Permalink
prevent tabbing to hidden submit
Browse files Browse the repository at this point in the history
  • Loading branch information
dleroux committed Nov 20, 2019
1 parent 35d61d2 commit 53206c7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions UNRELEASED.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
- Added `external` prop to `ResourceList` ([#2408](https://github.com/Shopify/polaris-react/pull/2408))
- Added `onMouseEnter` and `onTouchStart` props to `Button` ([#2409](https://github.com/Shopify/polaris-react/pull/2409))
- Added `ariaHaspopup` prop to `Popover` ([#2248](https://github.com/Shopify/polaris-react/pull/2248))
- Fixed an accessibility issue where the `Form` implicit submit was still accessible via keyboard ([#2447](https://github.com/Shopify/polaris-react/pull/2447))

### Bug fixes

Expand Down
2 changes: 1 addition & 1 deletion src/components/Form/Form.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ export function Form({

const submitMarkup = implicitSubmit ? (
<VisuallyHidden>
<button type="submit" aria-hidden="true">
<button type="submit" aria-hidden="true" tabIndex={-1}>
{i18n.translate('Polaris.Common.submit')}
</button>
</VisuallyHidden>
Expand Down

0 comments on commit 53206c7

Please sign in to comment.