[Box] Add position and visibility properties

[kyledurand]

This unblocks resource item work and will unblock lots of other components as well
Closes #7733

import React from 'react';

import {Box, Page} from '../src';

export function Playground() {
  return (
    <Page title="Playground">
      <Box
        position="relative"
        width="100%"
        minHeight="600px"
        background="action-primary-depressed"
      >
        <Box
          position="absolute"
          left="0"
          bottom="0"
          width="50px"
          minHeight="50px"
          background="action-critical-depressed"
          zIndex="1"
        />
        <Box
          position="absolute"
          left="0"
          bottom="25px"
          width="50px"
          minHeight="50px"
          background="action-critical"
          zIndex="2"
        />

        <Box
          position="absolute"
          top="var(--p-space-4)"
          right="var(--p-space-4)"
          width="100px"
          minHeight="100px"
          background="action-critical"
          zIndex="2"
        >
          <Box
            position="absolute"
            bottom="var(--p-space-2)"
            left="0"
            width="50px"
            minHeight="50px"
            background="action-secondary"
            zIndex="1"
          >
            <Box visuallyHidden>asdf</Box>
          </Box>
        </Box>
      </Box>
    </Page>
  );
}

[github-actions]

size-limit report 📦

Path	Size
polaris-react-cjs	210.84 KB (+0.05% 🔺)
polaris-react-esm	136.2 KB (+0.1% 🔺)
polaris-react-esnext	191.8 KB (+0.07% 🔺)
polaris-react-css	41.87 KB (+0.11% 🔺)

[kyledurand]

Any reason why we were remapping these? Just seemed to add complexity to me

[aveline]

Yeah I agree. I actually removed the padding one when I added responsive padding to Box. I think it would be fine to remove these as well.

[laurkim]

Nice, tophatted locally and lgtm! 💯
Minor thing, I was getting TS errors when using the playground example for position properties that weren't the spacing token (i.e., top="var(--p-space-4)") but works fine once I replaced them with the spacing token values.

[laurkim]

Nit: can we add prop descriptions to all of these so that they show up in the style guide?

[kyledurand]

Yes definitely

[aveline]

We should rename this to insetBlockStart to be consistent with the other properties where we have started to use the logical properties naming. Same with insetBlockEnd, insetInlineStart and insetInlineEnd.

[laurkim]

Issue to update: #7771