Skip to content

Commit

Permalink
fix(ADA-95): Purpose of logo link on player is not clear in context
Browse files Browse the repository at this point in the history
  • Loading branch information
Tzipi-kaltura committed Dec 17, 2023
1 parent 628049f commit 2289000
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions src/components/logo/logo.js
Original file line number Diff line number Diff line change
Expand Up @@ -53,11 +53,8 @@ class Logo extends Component {
return undefined;
}
return (
<div
className={[style.controlButtonContainer, !props.config.url ? style.emptyUrl : ''].join(' ')}
aria-label={props.logoText}
title={props.config.text}>
<a className={style.controlButton} href={props.config.url} target="_blank" rel="noopener noreferrer">
<div className={[style.controlButtonContainer, !props.config.url ? style.emptyUrl : ''].join(' ')} title={props.config.text}>
<a className={style.controlButton} href={props.config.url} aria-label={props.logoText} target="_blank" rel="noopener noreferrer">
<img className={style.icon} src={props.config.img} />
</a>
</div>
Expand Down

0 comments on commit 2289000

Please sign in to comment.