engineering/tpmjs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2.2k

fix(tool-search): fix download sorting and enable text selection

Browse source 
- Add nullish coalescing for downloads (handle null/undefined values)
- Add select-text class to Link and Card for text selection
This commit is contained in:
Ajax Davis 2025-12-11 09:33:45 +10:00
parent a99b572ac2
commit 833a023688
1 changed files with 5 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

8
apps/web/src/app/tool/tool-search/page.tsx
View file

@ -88,7 +88,9 @@ export default function ToolSearchPage(): React.ReactElement {
if (!aIsBroken && bIsBroken) return -1; if (!aIsBroken && bIsBroken) return -1;
// Within same broken status, sort by selected criterion // Within same broken status, sort by selected criterion
if (sortBy === 'downloads') { if (sortBy === 'downloads') {
return b.package.npmDownloadsLastMonth - a.package.npmDownloadsLastMonth; const aDownloads = a.package.npmDownloadsLastMonth ?? 0;
const bDownloads = b.package.npmDownloadsLastMonth ?? 0;
return bDownloads - aDownloads;
} }
// Sort by recent (createdAt descending) // Sort by recent (createdAt descending)
return new Date(b.createdAt).getTime() - new Date(a.createdAt).getTime(); return new Date(b.createdAt).getTime() - new Date(a.createdAt).getTime();
@ -240,9 +242,9 @@ export default function ToolSearchPage(): React.ReactElement {
<Link <Link
key={tool.id} key={tool.id}
href={`/tool/${tool.package.npmPackageName}/${tool.exportName}`} href={`/tool/${tool.package.npmPackageName}/${tool.exportName}`}
className="block" className="block select-text"
> >
<Card className="flex flex-col h-full hover:border-foreground-tertiary transition-colors cursor-pointer"> <Card className="flex flex-col h-full hover:border-foreground-tertiary transition-colors cursor-pointer select-text">
<CardHeader className="flex-none"> <CardHeader className="flex-none">
{/* Top row: Title + metadata */} {/* Top row: Title + metadata */}
<div className="flex items-start justify-between gap-2"> <div className="flex items-start justify-between gap-2">