diff --git a/src/components/common/ActionButton.tsx b/src/components/common/ActionButton.tsx index 837762a..16818ab 100644 --- a/src/components/common/ActionButton.tsx +++ b/src/components/common/ActionButton.tsx @@ -41,6 +41,8 @@ const ActionButton: React.FC = ({ } }; + const isIconOnly = icon && iconSlot === ActionButtonIconSlot.ICON_ONLY && !children; + return ( = ({ size={getSize()} className={className} style={{ - minWidth: '40px', + minWidth: isIconOnly ? '40px' : '40px', minHeight: '40px', + width: isIconOnly ? '40px' : 'auto', + height: isIconOnly ? '40px' : 'auto', display: 'flex', alignItems: 'center', - justifyContent: 'center' + justifyContent: 'center', + padding: isIconOnly ? '0' : undefined }} > {icon && { Songs by {selectedArtist} - - - +
+ +
diff --git a/src/features/Queue/Queue.tsx b/src/features/Queue/Queue.tsx index 186c0cc..29ccf06 100644 --- a/src/features/Queue/Queue.tsx +++ b/src/features/Queue/Queue.tsx @@ -234,20 +234,20 @@ const Queue: React.FC = () => { return ( <> -
- {isAdmin && ( - - )} -
-
+
+ {isAdmin && ( + + )} +
+ {/* First Item (Currently Playing) */} {renderFirstItem()} diff --git a/src/features/Settings/Settings.tsx b/src/features/Settings/Settings.tsx index 9bf3a11..8a62aaa 100644 --- a/src/features/Settings/Settings.tsx +++ b/src/features/Settings/Settings.tsx @@ -134,13 +134,16 @@ const Settings: React.FC = () => { Disabled Songs ({filteredDisabledSongs.length}) - setShowDisabledSongsModal(false)} - > - Close - +
+ setShowDisabledSongsModal(false)} + variant={ActionButtonVariant.SECONDARY} + size={ActionButtonSize.SMALL} + icon={Icons.CLOSE} + iconSlot={ActionButtonIconSlot.ICON_ONLY} + fill="clear" + /> +
diff --git a/src/features/Singers/Singers.tsx b/src/features/Singers/Singers.tsx index baebff4..bee0923 100644 --- a/src/features/Singers/Singers.tsx +++ b/src/features/Singers/Singers.tsx @@ -74,20 +74,20 @@ const Singers: React.FC = () => { return ( <> -
- {isAdmin && ( - - )} -
-
+
+ {isAdmin && ( + + )} +
+ items={singers} isLoading={singersCount === 0} @@ -112,13 +112,16 @@ const Singers: React.FC = () => { Add New Singer - +
+ +
diff --git a/src/features/SongLists/SongLists.tsx b/src/features/SongLists/SongLists.tsx index 690bef8..f859d82 100644 --- a/src/features/SongLists/SongLists.tsx +++ b/src/features/SongLists/SongLists.tsx @@ -1,7 +1,9 @@ import React, { useState, useMemo, useCallback } from 'react'; -import { IonItem, IonModal, IonHeader, IonToolbar, IonTitle, IonButton, IonIcon, IonChip, IonContent, IonList, IonAccordionGroup, IonAccordion } from '@ionic/react'; -import { close, list } from 'ionicons/icons'; -import { InfiniteScrollList, SongItem, ListItem, TwoLineDisplay, NumberDisplay, SongInfo } from '../../components/common'; +import { IonItem, IonModal, IonHeader, IonToolbar, IonTitle, IonChip, IonContent, IonList, IonAccordionGroup, IonAccordion } from '@ionic/react'; +import { list } from 'ionicons/icons'; +import { InfiniteScrollList, SongItem, ListItem, TwoLineDisplay, NumberDisplay, SongInfo, ActionButton } from '../../components/common'; +import { ActionButtonVariant, ActionButtonSize, ActionButtonIconSlot } from '../../types'; +import { Icons } from '../../constants'; import { useSongLists } from '../../hooks'; import { useAppSelector } from '../../redux'; import { selectSongList } from '../../redux'; @@ -101,9 +103,16 @@ const SongLists: React.FC = () => { {finalSelectedList?.title} - - - +
+ +
diff --git a/src/features/TopPlayed/Top100.tsx b/src/features/TopPlayed/Top100.tsx index 978291d..8fed00b 100644 --- a/src/features/TopPlayed/Top100.tsx +++ b/src/features/TopPlayed/Top100.tsx @@ -141,13 +141,16 @@ const Top100: React.FC = () => { {selectedTopPlayed?.artist} - +
+ +