show macOS warning one, don't show artist link if invalid

This commit is contained in:
Kendall Garner 2024-04-17 22:44:35 -07:00
parent 5d6503c1f4
commit ebd2f07447
No known key found for this signature in database
GPG key ID: 18D2767419676C87
2 changed files with 29 additions and 18 deletions

View file

@ -6,18 +6,20 @@ import { store } from '../settings';
export const enableMediaKeys = (window: BrowserWindow | null) => { export const enableMediaKeys = (window: BrowserWindow | null) => {
if (isMacOS()) { if (isMacOS()) {
const shouldPrompt = store.get('should_prompt_accessibility', true) as boolean; const shouldPrompt = store.get('should_prompt_accessibility', true) as boolean;
const shownWarning = store.get('shown_accessibility_warning', false) as boolean;
const trusted = systemPreferences.isTrustedAccessibilityClient(shouldPrompt); const trusted = systemPreferences.isTrustedAccessibilityClient(shouldPrompt);
if (shouldPrompt) { if (shouldPrompt) {
store.set('should_prompt_accessibility', false); store.set('should_prompt_accessibility', false);
} }
if (!trusted) { if (!trusted && !shownWarning) {
window?.webContents.send('toast-from-main', { window?.webContents.send('toast-from-main', {
message: message:
'Feishin is not a trusted accessibility client. Media keys will not work until this setting is changed', 'Feishin is not a trusted accessibility client. Media keys will not work until this setting is changed',
type: 'warning', type: 'warning',
}); });
store.set('shown_accessibility_warning', true);
} }
} }

View file

@ -48,8 +48,9 @@ const handleRow = <T extends AnyLibraryItem>(t: TFunction, item: T, rule: ItemDe
const formatArtists = (isAlbumArtist: boolean) => (item: Album | Song) => const formatArtists = (isAlbumArtist: boolean) => (item: Album | Song) =>
(isAlbumArtist ? item.albumArtists : item.artists)?.map((artist, index) => ( (isAlbumArtist ? item.albumArtists : item.artists)?.map((artist, index) => (
<span key={artist.id}> <span key={artist.id || artist.name}>
{index > 0 && <Separator />} {index > 0 && <Separator />}
{artist.id ? (
<Text <Text
$link $link
component={Link} component={Link}
@ -66,6 +67,14 @@ const formatArtists = (isAlbumArtist: boolean) => (item: Album | Song) =>
> >
{artist.name || '—'} {artist.name || '—'}
</Text> </Text>
) : (
<Text
overflow="visible"
size="md"
>
{artist.name || '-'}
</Text>
)}
</span> </span>
)); ));