Add mapping of "Box" to "div"

[khiga8]

Fixes: #170

This PR adds a mapping of Box to div to expand a11y linting coverage.

eslint-plugin-github has an a11y rule, a11y-role-supports-aria-props, which will flag invalid ARIA usage such as:

<Box as="div" aria-label="Something">

The plugin can lint the above markup by reading the value of the as prop, and interpreting it as a div.

However, because we don't have a mapping defined for Box for when as isn't set, the following isn't currently flagged even though it should be flagged as invalid:

<Box aria-label="Something">

This PR should will address this issue. I've tested this in dotcom.

I opted to only add the mapping for eslint-plugin-github because we have not enabled polymorphic linting for jsx-a11y rules yet, and if we define the mapping for jsx-a11y, it may raise false positives by always treating Box as a div.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 6824024

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[TylerJDev]

Nice! Thanks for adding this ✨

[khiga8]

Oh crap! I need to add a changeset 🤦‍♀️