refactor(VisuallyHidden): update base element from div to span (#4691)

* refactor(VisuallyHidden): use span by default over div

* refactor(VisuallyHidden): update base element from div to span

* chore: add changesets

---------

Co-authored-by: Josh Black <joshblack@users.noreply.github.com>

Author: joshblack

.changeset/thick-jars-count.md

@@ -0,0 +1,5 @@
+---
+'@primer/react': minor
+---
+
+Update internal VisuallyHidden helper to use a `span` by default over a `div` to support more nesting scenarios by default

packages/react/src/Spinner/Spinner.tsx

@@ -57,11 +57,7 @@ function Spinner({size: sizeKey = 'medium', srText = 'Loading', 'aria-label': ar
           vectorEffect="non-scaling-stroke"
         />
       </svg>
-      {hasHiddenLabel ? (
-        <VisuallyHidden as="span" id={labelId}>
-          {srText}
-        </VisuallyHidden>
-      ) : null}
+      {hasHiddenLabel ? <VisuallyHidden id={labelId}>{srText}</VisuallyHidden> : null}
     </Box>
   )
 }

packages/react/src/internal/components/VisuallyHidden.tsx

@@ -12,7 +12,7 @@ import sx from '../../sx'
  *
  * @see https://www.scottohara.me/blog/2023/03/21/visually-hidden-hack.html
  */
-export const VisuallyHidden = styled.div<SxProp>`
+export const VisuallyHidden = styled.span<SxProp>`
   &:not(:focus):not(:active):not(:focus-within) {
     clip-path: inset(50%);
     height: 1px;