Fix order of focusable selectors

[matallo]

WHY are these changes introduced?

Fixes an error when using Modal component with testing libraries.

Error: Uncaught [SyntaxError: ':disabled):not([aria-disabled="true"]' is not a valid selector]

WHAT is this pull request doing?

Order of selectors for button:not(:disabled):not([aria-disabled="true"]):not([tabindex="-1"]) was returning an error, and needed :not(:disabled) to be used last.

How to 🎩

🖥 Local development instructions
🗒 General tophatting guidelines
📄 Changelog guidelines

Copy-paste this code in playground/Playground.tsx:

import React from 'react';
import {Page} from '../src';

export function Playground() {
  return (
    <Page title="Playground">
      {/* Add the code you want to test in here */}
    </Page>
  );
}

🎩 checklist

• Tested on mobile
• Tested on multiple browsers
• Tested for accessibility
• Updated the component's README.md with documentation changes
• Tophatted documentation changes in the style guide

[github-actions]

size-limit report 📦

Path	Size
polaris-react-cjs	240.61 KB (0%)
polaris-react-esm	156.76 KB (-0.01% 🔽)
polaris-react-esnext	218.97 KB (-0.01% 🔽)
polaris-react-css	47.24 KB (0%)

[acmertz]

/snapit

[github-actions]

🫰✨ Thanks @acmertz! Your snapshots have been published to npm.

Test the snapshots by updating your package.json with the newly published versions:

yarn add @shopify/polaris-cli@0.0.0-snapshot-release-20230420205546

yarn add @shopify/polaris@0.0.0-snapshot-release-20230420205546

[acmertz]

LGTM! I used the snapshot from #9039 (comment) to test this in web and confirm that the fix from #7831 still works.