pl-fe: Remove comment for obsolete workaround
Signed-off-by: marcin mikołajczak <git@mkljczk.pl>
This commit is contained in:
parent
0e267b762d
commit
05fff4c381
1 changed files with 8 additions and 16 deletions
|
@ -14,21 +14,13 @@ interface IForkAwesomeIcon extends React.HTMLAttributes<HTMLLIElement> {
|
||||||
fixedWidth?: boolean;
|
fixedWidth?: boolean;
|
||||||
}
|
}
|
||||||
|
|
||||||
const ForkAwesomeIcon: React.FC<IForkAwesomeIcon> = ({ id, className, fixedWidth, ...rest }) => {
|
const ForkAwesomeIcon: React.FC<IForkAwesomeIcon> = ({ id, className, fixedWidth, ...rest }) => (
|
||||||
// Use the Fork Awesome retweet icon, but change its alt
|
<i
|
||||||
// tag. There is a common adblocker rule which hides elements with
|
role='img'
|
||||||
// alt='retweet' unless the domain is twitter.com. This should
|
// alt={alt}
|
||||||
// change what screenreaders call it as well.
|
className={clsx('fa', `fa-${id}`, className, { 'fa-fw': fixedWidth })}
|
||||||
// const alt = (id === 'retweet') ? 'repost' : id;
|
{...rest}
|
||||||
|
/>
|
||||||
return (
|
);
|
||||||
<i
|
|
||||||
role='img'
|
|
||||||
// alt={alt}
|
|
||||||
className={clsx('fa', `fa-${id}`, className, { 'fa-fw': fixedWidth })}
|
|
||||||
{...rest}
|
|
||||||
/>
|
|
||||||
);
|
|
||||||
};
|
|
||||||
|
|
||||||
export { ForkAwesomeIcon as default };
|
export { ForkAwesomeIcon as default };
|
||||||
|
|
Loading…
Reference in a new issue