Skip to content

Commit

Permalink
Add aria-haspopup to CircularOptionPicker component (#25564)
Browse files Browse the repository at this point in the history
  • Loading branch information
nosolosw committed Sep 24, 2020
1 parent da3ce97 commit 093bdf0
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,7 @@ exports[`ColorPaletteControl matches the snapshot 1`] = `
>
<button
aria-expanded={false}
aria-haspopup="true"
aria-label="Custom color picker"
className="components-button is-link"
onClick={[Function]}
Expand Down
1 change: 1 addition & 0 deletions packages/components/src/circular-option-picker/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@ function DropdownLinkAction( {
renderToggle={ ( { isOpen, onToggle } ) => (
<Button
aria-expanded={ isOpen }
aria-haspopup="true"
onClick={ onToggle }
isLink
{ ...buttonProps }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,7 @@ exports[`ColorPalette Dropdown .renderContent should render dropdown content 1`]
exports[`ColorPalette Dropdown .renderToggle should render dropdown content 1`] = `
<button
aria-expanded={true}
aria-haspopup="true"
aria-label="Custom color picker"
className="components-button is-link"
onClick={[MockFunction]}
Expand All @@ -110,12 +111,14 @@ exports[`ColorPalette Dropdown should render it correctly 1`] = `
>
<ForwardRef(Button)
aria-expanded={false}
aria-haspopup="true"
aria-label="Custom color picker"
isLink={true}
onClick={[Function]}
>
<button
aria-expanded={false}
aria-haspopup="true"
aria-label="Custom color picker"
className="components-button is-link"
onClick={[Function]}
Expand Down Expand Up @@ -538,12 +541,14 @@ exports[`ColorPalette should render a dynamic toolbar of colors 1`] = `
>
<ForwardRef(Button)
aria-expanded={false}
aria-haspopup="true"
aria-label="Custom color picker"
isLink={true}
onClick={[Function]}
>
<button
aria-expanded={false}
aria-haspopup="true"
aria-label="Custom color picker"
className="components-button is-link"
onClick={[Function]}
Expand Down

0 comments on commit 093bdf0

Please sign in to comment.