Skip to content

Commit

Permalink
fix: 943-audit-avt1-homepage-banner-role (#944)
Browse files Browse the repository at this point in the history
* fix: issue 808 - removed button in ImageGallereyImage

* Revert "fix: issue 808 - removed button in ImageGallereyImage"

This reverts commit 8d797b2.

* fix: 943-audit-avt1-homepage-banner-role

Co-authored-by: andrea.cardonaibm.com <andrea.cardonaibm.com>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Co-authored-by: Vince Picone <vpicone@gmail.com>
  • Loading branch information
3 people authored Jul 27, 2020
1 parent effea78 commit 949f3c0
Showing 1 changed file with 6 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -44,18 +44,19 @@ const Container = ({ children, homepage, theme }) => {

return (
<>
<span hidden id="overlay-label">
Collapse navigation items
</span>
<div
role="button"
aria-labelledby="overlay-label"
tabIndex="0"
className={cx(overlay, { [visible]: overlayVisible })}
onClick={closeNavs}
onKeyPress={closeNavs}
role="presentation"
tabIndex="-1"
/>
<main
id="main-content"
role="presentation" // needed for jsx-a11y/no-noninteractive-element-interactions
onClick={closeNavs}
onKeyPress={closeNavs}
aria-hidden={overlayVisible}
className={containerClassNames}>
{children}
Expand Down

1 comment on commit 949f3c0

@vercel
Copy link

@vercel vercel bot commented on 949f3c0 Jul 27, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.