Signed-off-by: Matt Bruce <mbrucedogs@gmail.com>

src/components/common/InfiniteScrollList.tsx

@@ -1,46 +1,35 @@
 import React, { useEffect, useRef } from 'react';
-import { SongItem, EmptyState } from './index';
-import type { Song } from '../../types';
+import { EmptyState } from './index';
 
-interface InfiniteScrollListProps {
-  items: Song[];
+interface InfiniteScrollListProps<T> {
+  items: T[];
   isLoading: boolean;
   hasMore: boolean;
   onLoadMore: () => void;
-  onAddToQueue: (song: Song) => void;
-  onToggleFavorite: (song: Song) => void;
-  onRemoveFromQueue?: (song: Song) => void;
-  context: 'search' | 'queue' | 'history' | 'topPlayed' | 'favorites';
+  renderItem: (item: T, index: number) => React.ReactNode;
   title: string;
   subtitle?: string;
   emptyTitle: string;
   emptyMessage: string;
   loadingTitle?: string;
   loadingMessage?: string;
-  isAdmin?: boolean;
-  renderExtraContent?: (item: Song, index: number) => React.ReactNode;
   debugInfo?: string;
 }
 
-const InfiniteScrollList: React.FC<InfiniteScrollListProps> = ({
+const InfiniteScrollList = <T extends { key?: string }>({
   items,
   isLoading,
   hasMore,
   onLoadMore,
-  onAddToQueue,
-  onToggleFavorite,
-  onRemoveFromQueue,
-  context,
+  renderItem,
   title,
   subtitle,
   emptyTitle,
   emptyMessage,
   loadingTitle = "Loading...",
   loadingMessage = "Please wait while data is being loaded",
-  isAdmin = false,
-  renderExtraContent,
   debugInfo,
-}) => {
+}: InfiniteScrollListProps<T>) => {
   const observerRef = useRef<HTMLDivElement>(null);
 
   // Intersection Observer for infinite scrolling
@@ -111,21 +100,8 @@ const InfiniteScrollList: React.FC<InfiniteScrollListProps> = ({
         ) : (
           <div className="divide-y divide-gray-200">
             {items.map((item, index) => (
-              <div key={item.key} className="flex items-center">
-                {/* Song Info */}
-                <div className="flex-1">
-                  <SongItem
-                    song={item}
-                    context={context}
-                    onAddToQueue={() => onAddToQueue(item)}
-                    onToggleFavorite={() => onToggleFavorite(item)}
-                    onRemoveFromQueue={onRemoveFromQueue ? () => onRemoveFromQueue(item) : undefined}
-                    isAdmin={isAdmin}
-                  />
-                </div>
-
-                {/* Extra Content */}
-                {renderExtraContent && renderExtraContent(item, index)}
+              <div key={item.key}>
+                {renderItem(item, index)}
               </div>
             ))}
             

src/features/Favorites/Favorites.tsx

@@ -1,9 +1,10 @@
 import React from 'react';
 import { IonHeader, IonToolbar, IonTitle, IonChip } from '@ionic/react';
-import { InfiniteScrollList } from '../../components/common';
+import { InfiniteScrollList, SongItem } from '../../components/common';
 import { useFavorites } from '../../hooks';
 import { useAppSelector } from '../../redux';
 import { selectFavorites } from '../../redux';
+import type { Song } from '../../types';
 
 const Favorites: React.FC = () => {
   const {
@@ -34,16 +35,21 @@ const Favorites: React.FC = () => {
         </IonToolbar>
       </IonHeader>
 
-      <InfiniteScrollList
+      <InfiniteScrollList<Song>
         items={favoritesItems}
         isLoading={favoritesCount === 0}
         hasMore={hasMore}
         onLoadMore={loadMore}
-        onAddToQueue={handleAddToQueue}
-        onToggleFavorite={handleToggleFavorite}
-        context="favorites"
-        title=""
-        subtitle=""
+        renderItem={(song) => (
+          <SongItem
+            song={song}
+            context="favorites"
+            onAddToQueue={() => handleAddToQueue(song)}
+            onToggleFavorite={() => handleToggleFavorite(song)}
+          />
+        )}
+        title="Favorites"
+        subtitle={`${favoritesCount} items loaded`}
         emptyTitle="No favorites yet"
         emptyMessage="Add songs to your favorites to see them here"
         loadingTitle="Loading favorites..."

src/features/History/History.tsx

@@ -1,7 +1,7 @@
 import React from 'react';
 import { IonHeader, IonToolbar, IonTitle, IonChip, IonIcon } from '@ionic/react';
 import { time } from 'ionicons/icons';
-import { InfiniteScrollList } from '../../components/common';
+import { InfiniteScrollList, SongItem } from '../../components/common';
 import { useHistory } from '../../hooks';
 import { useAppSelector } from '../../redux';
 import { selectHistory } from '../../redux';
@@ -51,22 +51,31 @@ const History: React.FC = () => {
       </IonHeader>
 
       <div style={{ height: '100%', overflowY: 'auto' }}>
-        <InfiniteScrollList
+        <InfiniteScrollList<Song>
           items={historyItems}
           isLoading={historyCount === 0}
           hasMore={hasMore}
           onLoadMore={loadMore}
-          onAddToQueue={handleAddToQueue}
-          onToggleFavorite={handleToggleFavorite}
-          context="history"
-          title=""
-          subtitle=""
+          renderItem={(song) => (
+            <div className="flex items-center">
+              <div className="flex-1">
+                <SongItem
+                  song={song}
+                  context="history"
+                  onAddToQueue={() => handleAddToQueue(song)}
+                  onToggleFavorite={() => handleToggleFavorite(song)}
+                />
+              </div>
+              {renderExtraContent(song)}
+            </div>
+          )}
+          title="Recently Played"
+          subtitle={`${historyCount} items loaded`}
           emptyTitle="No history yet"
           emptyMessage="Songs will appear here after they've been played"
           loadingTitle="Loading history..."
           loadingMessage="Please wait while history data is being loaded"
           debugInfo={`History items loaded: ${historyCount}`}
-          renderExtraContent={renderExtraContent}
         />
       </div>
     </>

src/features/NewSongs/NewSongs.tsx

@@ -1,9 +1,10 @@
 import React from 'react';
 import { IonHeader, IonToolbar, IonTitle, IonChip } from '@ionic/react';
-import { InfiniteScrollList } from '../../components/common';
+import { InfiniteScrollList, SongItem } from '../../components/common';
 import { useNewSongs } from '../../hooks';
 import { useAppSelector } from '../../redux';
 import { selectNewSongs } from '../../redux';
+import type { Song } from '../../types';
 
 const NewSongs: React.FC = () => {
   const {
@@ -34,16 +35,21 @@ const NewSongs: React.FC = () => {
         </IonToolbar>
       </IonHeader>
 
-      <InfiniteScrollList
+      <InfiniteScrollList<Song>
         items={newSongsItems}
         isLoading={newSongsCount === 0}
         hasMore={hasMore}
         onLoadMore={loadMore}
-        onAddToQueue={handleAddToQueue}
-        onToggleFavorite={handleToggleFavorite}
-        context="search"
-        title=""
-        subtitle=""
+        renderItem={(song) => (
+          <SongItem
+            song={song}
+            context="search"
+            onAddToQueue={() => handleAddToQueue(song)}
+            onToggleFavorite={() => handleToggleFavorite(song)}
+          />
+        )}
+        title="New Songs"
+        subtitle={`${newSongsCount} items loaded`}
         emptyTitle="No new songs"
         emptyMessage="Check back later for new additions"
         loadingTitle="Loading new songs..."

src/features/Search/Search.tsx

@@ -1,9 +1,10 @@
 import React from 'react';
 import { IonSearchbar } from '@ionic/react';
-import { InfiniteScrollList } from '../../components/common';
+import { InfiniteScrollList, SongItem } from '../../components/common';
 import { useSearch } from '../../hooks';
 import { useAppSelector } from '../../redux';
 import { selectIsAdmin, selectSongs } from '../../redux';
+import type { Song } from '../../types';
 
 const Search: React.FC = () => {
   const {
@@ -55,20 +56,25 @@ const Search: React.FC = () => {
       </div>
 
       {/* Search Results */}
-      <InfiniteScrollList
+      <InfiniteScrollList<Song>
         items={searchResults.songs}
         isLoading={songsCount === 0}
         hasMore={searchResults.hasMore}
         onLoadMore={loadMore}
-        onAddToQueue={handleAddToQueue}
-        onToggleFavorite={handleToggleFavorite}
-        context="search"
+        renderItem={(song) => (
+          <SongItem
+            song={song}
+            context="search"
+            onAddToQueue={() => handleAddToQueue(song)}
+            onToggleFavorite={() => handleToggleFavorite(song)}
+            isAdmin={isAdmin}
+          />
+        )}
         title=""
         emptyTitle={searchTerm ? "No songs found" : "No songs available"}
         emptyMessage={searchTerm ? "Try adjusting your search terms" : "Songs will appear here once loaded"}
         loadingTitle="Loading songs..."
         loadingMessage="Please wait while songs are being loaded from the database"
-        isAdmin={isAdmin}
         debugInfo=""
       />
 

src/features/TopPlayed/Top100.tsx

@@ -1,8 +1,9 @@
 import React from 'react';
-import { IonHeader, IonToolbar, IonTitle, IonChip, IonButton } from '@ionic/react';
+import { IonHeader, IonToolbar, IonTitle, IonChip } from '@ionic/react';
 import { useTopPlayed } from '../../hooks';
 import { useAppSelector } from '../../redux';
 import { selectTopPlayed } from '../../redux';
+import { InfiniteScrollList } from '../../components/common';
 import type { TopPlayed } from '../../types';
 
 const Top100: React.FC = () => {
@@ -109,78 +110,35 @@ const Top100: React.FC = () => {
         </IonToolbar>
       </IonHeader>
 
-      <div className="max-w-4xl mx-auto p-6">
-        {/* Debug info */}
-        <div className="mt-2 text-sm text-gray-500 mb-4">
-          Top played items loaded: {displayCount} (Mock Data)
-        </div>
-
-        {/* Content */}
-        <div className="bg-white rounded-lg shadow">
-          {displayCount === 0 ? (
-            <div className="p-8 text-center">
-              <div className="text-gray-400 mb-4">
-                <svg className="h-12 w-12 mx-auto animate-spin" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                  <path strokeLinecap="round" strokeLinejoin="round" strokeWidth={2} d="M4 4v5h.582m15.356 2A8.001 8.001 0 004.582 9m0 0H9m11 11v-5h-.581m0 0a8.003 8.003 0 01-15.357-2m15.357 2H15" />
-                </svg>
+      <InfiniteScrollList<TopPlayed>
+        items={displayItems}
+        isLoading={false}
+        hasMore={displayHasMore}
+        onLoadMore={loadMore}
+        renderItem={(item, index) => (
+          <div style={{ display: 'flex', alignItems: 'flex-start', padding: '12px 16px', borderBottom: '1px solid #e5e7eb' }}>
+            <div style={{ width: '80px', textAlign: 'right', paddingRight: '16px', flexShrink: 0 }}>
+              <span style={{ fontSize: '18px', fontWeight: 'bold', color: '#374151' }}>
+                {index + 1})
+              </span>
+            </div>
+            <div style={{ width: '16px', flexShrink: 0 }}></div>
+            <div style={{ flex: 1 }}>
+              <div style={{ fontSize: '18px', fontWeight: 'bold', color: '#111827', marginBottom: '4px' }}>
+                {item.title} ({item.count})
               </div>
-              <h3 className="text-lg font-medium text-gray-900 mb-2">Loading top played songs...</h3>
-              <p className="text-sm text-gray-500">Please wait while top played data is being loaded</p>
-            </div>
-          ) : displayItems.length === 0 ? (
-            <div className="p-8 text-center">
-              <div className="text-gray-400 mb-4">
-                <svg className="h-12 w-12 mx-auto" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                  <path strokeLinecap="round" strokeLinejoin="round" strokeWidth={2} d="M9 19V6l12-3v13M9 19c0 1.105-1.343 2-3 2s-3-.895-3-2 1.343-2 3-2 3 .895 3 2zm12-3c0 1.105-1.343 2-3 2s-3-.895-3-2 1.343-2 3-2 3 .895 3 2zM9 10l12-3" />
-                </svg>
+              <div style={{ fontSize: '16px', fontStyle: 'italic', color: '#4b5563' }}>
+                {item.artist}
               </div>
-              <h3 className="text-lg font-medium text-gray-900 mb-2">No top played songs</h3>
-              <p className="text-sm text-gray-500">Play some songs to see the top played list</p>
             </div>
-          ) : (
-            <div className="w-full">
-              {displayItems.map((item, index) => (
-                <div key={item.key} style={{ display: 'flex', alignItems: 'flex-start', padding: '12px 16px', borderBottom: '1px solid #e5e7eb' }}>
-                  <div style={{ width: '80px', textAlign: 'right', paddingRight: '16px', flexShrink: 0 }}>
-                    <span style={{ fontSize: '18px', fontWeight: 'bold', color: '#374151' }}>
-                      {index + 1})
-                    </span>
-                  </div>
-                  <div style={{ width: '16px', flexShrink: 0 }}></div>
-                  <div style={{ flex: 1 }}>
-                    <div style={{ fontSize: '18px', fontWeight: 'bold', color: '#111827', marginBottom: '4px' }}>
-                      {item.title} ({item.count})
-                    </div>
-                    <div style={{ fontSize: '16px', fontStyle: 'italic', color: '#4b5563' }}>
-                      {item.artist}
-                    </div>
-                  </div>
-                </div>
-              ))}
-              
-              {/* Load more button */}
-              {displayHasMore && (
-                <div className="p-4 text-center">
-                  <IonButton
-                    fill="outline"
-                    onClick={loadMore}
-                  >
-                    Load More
-                  </IonButton>
-                </div>
-              )}
-            </div>
-          )}
-        </div>
-
-        {/* Stats */}
-        {displayItems.length > 0 && (
-          <div className="mt-4 text-sm text-gray-500 text-center">
-            Showing {displayItems.length} item{displayItems.length !== 1 ? 's' : ''}
-            {displayHasMore && ` • Click "Load More" to see more`}
           </div>
         )}
-      </div>
+        title="Top 100 Played"
+        subtitle={`${displayCount} items loaded (Mock Data)`}
+        emptyTitle="No top played songs"
+        emptyMessage="Play some songs to see the top played list"
+        debugInfo={`Top played items loaded: ${displayCount} (Mock Data)`}
+      />
     </>
   );
 };