Creepso/stremio-web
1
0
Fork 0
mirror of https://github.com/Stremio/stremio-web.git synced 2026-03-11 21:27:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

refactor(Stream): use memo deps instead of passing props to Popup

Browse source
This commit is contained in:
Tim 2024-07-26 18:06:54 +02:00
parent 396f44ed8a
commit d8fbd1855b
1 changed files with 3 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/routes/MetaDetails/StreamsList/Stream/Stream.js
View file

@ -147,9 +147,9 @@ const Stream = ({ className, videoId, videoReleased, addonName, name, descriptio
<Icon className={styles['placeholder-icon']} name={'ic_broken_link'} /> <Icon className={styles['placeholder-icon']} name={'ic_broken_link'} />
), []); ), []);
const renderLabel = React.useMemo(() => function renderLabel({ className, thumbnail, progress, addonName, name, description, children, ...props }) { const renderLabel = React.useMemo(() => function renderLabel({ className, children, ...props }) {
return ( return (
<Button className={classnames(className, styles['stream-container'])} title={addonName} href={href} download={download} target={target} onClick={onClick} {...props}> <Button className={classnames(className, styles['stream-container'])} title={addonName} href={href} target={target} download={download} onClick={onClick} {...props}>
<div className={styles['info-container']}> <div className={styles['info-container']}>
{ {
typeof thumbnail === 'string' && thumbnail.length > 0 ? typeof thumbnail === 'string' && thumbnail.length > 0 ?
@ -181,7 +181,7 @@ const Stream = ({ className, videoId, videoReleased, addonName, name, descriptio
{children} {children}
</Button> </Button>
); );
}, [onClick]); }, [thumbnail, progress, addonName, name, description, href, target, download, onClick]);
const renderMenu = React.useMemo(() => function renderMenu() { const renderMenu = React.useMemo(() => function renderMenu() {
return ( return (
@ -208,13 +208,6 @@ const Stream = ({ className, videoId, videoReleased, addonName, name, descriptio
return ( return (
<Popup <Popup
className={className} className={className}
thumbnail={thumbnail}
progress={progress}
addonName={addonName}
name={name}
description={description}
href={href}
{...props}
onMouseUp={popupLabelOnMouseUp} onMouseUp={popupLabelOnMouseUp}
onLongPress={popupLabelOnLongPress} onLongPress={popupLabelOnLongPress}
onContextMenu={popupLabelOnContextMenu} onContextMenu={popupLabelOnContextMenu}