Skip to content

Commit

Permalink
Add accessible names to icon stories (#5621)
Browse files Browse the repository at this point in the history
* Add accName to `CircleOcticon`

* Add alt to `AvatarPair` examples
  • Loading branch information
TylerJDev authored Feb 3, 2025
1 parent c9b0a72 commit 7b644ce
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 9 deletions.
8 changes: 4 additions & 4 deletions packages/react/src/AvatarPair/AvatarPair.features.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,14 +10,14 @@ export default {

export const ParentCircle = () => (
<AvatarPair>
<Avatar src="https://avatars.githubusercontent.com/u/7143434?v=4" />
<Avatar square src="https://avatars.githubusercontent.com/primer" />
<Avatar src="https://avatars.githubusercontent.com/u/7143434?v=4" alt="Primer avatar, parent" />
<Avatar square src="https://avatars.githubusercontent.com/primer" alt="Primer avatar, child" />
</AvatarPair>
)

export const ParentSquare = () => (
<AvatarPair>
<Avatar square src="https://avatars.githubusercontent.com/primer" />
<Avatar src="https://avatars.githubusercontent.com/u/7143434?v=4" />
<Avatar square src="https://avatars.githubusercontent.com/primer" alt="Primer avatar, parent" />
<Avatar src="https://avatars.githubusercontent.com/u/7143434?v=4" alt="Primer avatar, child" />
</AvatarPair>
)
4 changes: 2 additions & 2 deletions packages/react/src/AvatarPair/AvatarPair.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ export default {

export const Default = () => (
<AvatarPair>
<Avatar src="https://avatars.githubusercontent.com/u/7143434?v=4" />
<Avatar src="https://avatars.githubusercontent.com/primer" />
<Avatar src="https://avatars.githubusercontent.com/u/7143434?v=4" alt="Primer avatar, parent" />
<Avatar src="https://avatars.githubusercontent.com/primer" alt="Primer avatar, child" />
</AvatarPair>
)
8 changes: 5 additions & 3 deletions packages/react/src/CircleOcticon/CircleOcticon.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,11 @@ export const Default = () => (
)

type PlaygroundTypes = Omit<CircleOcticonProps, 'icon'> & {icon: keyof typeof Icons}
export const Playground: StoryFn<PlaygroundTypes> = ({icon: iconName, 'aria-label': ariaLabel, ...args}) => (
<CircleOcticon icon={Icons[iconName]} aria-label={ariaLabel ? ariaLabel : undefined} {...args} />
)
export const Playground: StoryFn<PlaygroundTypes> = ({
icon: iconName,
'aria-label': ariaLabel = 'Changes approved',
...args
}) => <CircleOcticon icon={Icons[iconName]} aria-label={ariaLabel ? ariaLabel : undefined} {...args} />

Playground.args = {
size: 32,
Expand Down

0 comments on commit 7b644ce

Please sign in to comment.