LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

ListOverlay: don't loop through all items if we just want 2

Browse source 
`map` does not short-circuit
This commit is contained in:
infinite-persistence 2022-03-12 09:27:41 +08:00 committed by Thomas Zarebczan
parent 5b2c901496
commit 2e3f9f94ce
1 changed files with 7 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
ui/component/collectionPreviewOverlay/view.jsx
View file

@ -25,20 +25,17 @@ function CollectionPreviewOverlay(props: Props) {
}, [collectionId, collectionItemUrls, fetchCollectionItems]);
if (collectionItemUrls && collectionItemUrls.length > 0) {
const displayed = collectionItemUrls.slice(0, 2);
return (
<div className="collection-preview__overlay-thumbs">
<div className="collection-preview__overlay-side" />
<div className="collection-preview__overlay-grid">
{collectionItemUrls &&
collectionItemUrls.map((item, index) => {
if (index < 2) {
return (
<div className="collection-preview__overlay-grid-items" key={item}>
<FileThumbnail uri={item} />
</div>
);
}
})}
{displayed.map((uri) => (
<div className="collection-preview__overlay-grid-items" key={uri}>
<FileThumbnail uri={uri} />
</div>
))}
</div>
</div>
);