danvics/soulsync
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Fix Spotify badge icon & compact card badge layout for overflow

Browse source
This commit is contained in:
Broque Thomas 2026-03-10 12:54:18 -07:00
parent 87d567151e
commit 927fe6338e
3 changed files with 144 additions and 66 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
database/music_database.py
View file

@ -5919,6 +5919,8 @@ class MusicDatabase:
a.itunes_artist_id,
a.deezer_id,
a.audiodb_id,
a.lastfm_url,
a.genius_url,
COUNT(DISTINCT al.id) as album_count,
COUNT(DISTINCT t.id) as track_count
FROM artists a
@ -5929,7 +5931,7 @@ class MusicDatabase:
WHERE {where_clause}
AND a.id = (SELECT MIN(a2.id) FROM artists a2
WHERE a2.name = a.name AND a2.server_source = a.server_source)
GROUP BY a.id, a.name, a.thumb_url, a.genres, a.musicbrainz_id, a.deezer_id, a.audiodb_id
GROUP BY a.id, a.name, a.thumb_url, a.genres, a.musicbrainz_id, a.spotify_artist_id, a.itunes_artist_id, a.deezer_id, a.audiodb_id, a.lastfm_url, a.genius_url
ORDER BY a.name COLLATE NOCASE
LIMIT ? OFFSET ?
"""
@ -5977,6 +5979,8 @@ class MusicDatabase:
'itunes_artist_id': row['itunes_artist_id'],
'deezer_id': row['deezer_id'],
'audiodb_id': row['audiodb_id'],
'lastfm_url': row['lastfm_url'],
'genius_url': row['genius_url'],
'album_count': row['album_count'] or 0,
'track_count': row['track_count'] or 0,
'is_watched': bool(is_watched)
@ -6032,7 +6036,9 @@ class MusicDatabase:
# Get artist information
cursor.execute("""
SELECT
id, name, thumb_url, genres, server_source, musicbrainz_id, deezer_id, audiodb_id
id, name, thumb_url, genres, server_source,
musicbrainz_id, deezer_id, audiodb_id,
spotify_artist_id, itunes_artist_id, lastfm_url, genius_url
FROM artists
WHERE id = ?
""", (artist_id,))
@ -6181,6 +6187,10 @@ class MusicDatabase:
'musicbrainz_id': artist_row['musicbrainz_id'],
'deezer_id': artist_row['deezer_id'],
'audiodb_id': artist_row['audiodb_id'],
'spotify_artist_id': artist_row['spotify_artist_id'],
'itunes_artist_id': artist_row['itunes_artist_id'],
'lastfm_url': artist_row['lastfm_url'],
'genius_url': artist_row['genius_url'],
'album_count': album_count,
'track_count': track_count
},

88
webui/static/script.js
View file

@ -398,9 +398,13 @@ function handleDashboardWishlistCount(data) {
// END WEBSOCKET CONNECTION MANAGER
// ===============================
// --- MusicBrainz Integration Constants ---
// --- Service Integration Logo Constants ---
const MUSICBRAINZ_LOGO_URL = 'https://upload.wikimedia.org/wikipedia/commons/thumb/9/9e/MusicBrainz_Logo_%282016%29.svg/500px-MusicBrainz_Logo_%282016%29.svg.png';
const DEEZER_LOGO_URL = 'https://cdn.brandfetch.io/idEUKgCNtu/theme/dark/symbol.svg?c=1bxid64Mup7aczewSAYMX&t=1758260798610';
const SPOTIFY_LOGO_URL = 'https://storage.googleapis.com/pr-newsroom-wp/1/2023/05/Spotify_Primary_Logo_RGB_Green.png';
const ITUNES_LOGO_URL = 'https://upload.wikimedia.org/wikipedia/commons/thumb/d/df/ITunes_logo.svg/960px-ITunes_logo.svg.png';
const LASTFM_LOGO_URL = 'https://www.last.fm/static/images/lastfm_avatar_twitter.52a5d69a85ac.png';
const GENIUS_LOGO_URL = 'https://images.genius.com/8ed669cadd956443e29c70361ec4f372.1000x1000x1.png';
function getAudioDBLogoURL() { const el = document.querySelector('img.audiodb-logo'); return el ? el.src : null; }
// --- Wishlist Modal Persistence State Management ---
@ -31780,8 +31784,11 @@ function createLibraryArtistCard(artist) {
card.style.position = 'relative';
card.style.transition = 'transform 0.2s, box-shadow 0.2s';
// Add source badges (MusicBrainz, Deezer, AudioDB) stacked on top-right
// Add source badges stacked on top-right
const badgeSources = [];
if (artist.spotify_artist_id) {
badgeSources.push({ cls: 'spotify-card-icon', logo: SPOTIFY_LOGO_URL, fallback: 'SP', title: 'View on Spotify', url: `https://open.spotify.com/artist/${artist.spotify_artist_id}` });
}
if (artist.musicbrainz_id) {
badgeSources.push({ cls: 'mb-card-icon', logo: MUSICBRAINZ_LOGO_URL, fallback: 'MB', title: 'View on MusicBrainz', url: `https://musicbrainz.org/artist/${artist.musicbrainz_id}` });
}
@ -31792,28 +31799,39 @@ function createLibraryArtistCard(artist) {
const adbSlug = artist.name ? artist.name.replace(/\s+/g, '-').replace(/[^a-zA-Z0-9-]/g, '') : '';
badgeSources.push({ cls: 'audiodb-card-icon', logo: getAudioDBLogoURL(), fallback: 'ADB', title: 'View on TheAudioDB', url: `https://www.theaudiodb.com/artist/${artist.audiodb_id}-${adbSlug}` });
}
let badgeOffset = 8;
badgeSources.forEach(source => {
const icon = document.createElement('div');
icon.className = `${source.cls} source-card-icon`;
icon.style.top = badgeOffset + 'px';
icon.title = source.title;
if (source.logo) {
const img = document.createElement('img');
img.src = source.logo;
img.style.cssText = 'width: 20px; height: auto; display: block;';
img.onerror = () => { icon.textContent = source.fallback; };
icon.appendChild(img);
} else {
icon.textContent = source.fallback;
}
icon.onclick = (e) => {
e.stopPropagation();
window.open(source.url, '_blank');
};
card.appendChild(icon);
badgeOffset += 32;
});
if (artist.itunes_artist_id) {
badgeSources.push({ cls: 'itunes-card-icon', logo: ITUNES_LOGO_URL, fallback: 'IT', title: 'View on Apple Music', url: `https://music.apple.com/artist/${artist.itunes_artist_id}` });
}
if (artist.lastfm_url) {
badgeSources.push({ cls: 'lastfm-card-icon', logo: LASTFM_LOGO_URL, fallback: 'LFM', title: 'View on Last.fm', url: artist.lastfm_url });
}
if (artist.genius_url) {
badgeSources.push({ cls: 'genius-card-icon', logo: GENIUS_LOGO_URL, fallback: 'GEN', title: 'View on Genius', url: artist.genius_url });
}
if (badgeSources.length > 0) {
const badgeContainer = document.createElement('div');
badgeContainer.className = 'card-badge-container';
badgeSources.forEach(source => {
const icon = document.createElement('div');
icon.className = `${source.cls} source-card-icon`;
icon.title = source.title;
if (source.logo) {
const img = document.createElement('img');
img.src = source.logo;
img.style.cssText = 'width: 16px; height: auto; display: block;';
img.onerror = () => { icon.textContent = source.fallback; };
icon.appendChild(img);
} else {
icon.textContent = source.fallback;
}
icon.onclick = (e) => {
e.stopPropagation();
window.open(source.url, '_blank');
};
badgeContainer.appendChild(icon);
});
card.appendChild(badgeContainer);
}
// Add watchlist button/indicator on the card
const hasExternalId = artist.itunes_artist_id || artist.spotify_artist_id;
@ -32277,9 +32295,13 @@ function updateArtistDetailPageHeaderWithData(artist) {
const adbSlug = artist.name ? artist.name.replace(/\s+/g, '-').replace(/[^a-zA-Z0-9-]/g, '') : '';
const sources = [
{ id: artist.spotify_artist_id, url: `https://open.spotify.com/artist/${artist.spotify_artist_id}`, logo: SPOTIFY_LOGO_URL, label: 'Spotify' },
{ id: artist.itunes_artist_id, url: `https://music.apple.com/artist/${artist.itunes_artist_id}`, logo: ITUNES_LOGO_URL, label: 'Apple Music' },
{ id: artist.musicbrainz_id, url: `https://musicbrainz.org/artist/${artist.musicbrainz_id}`, logo: MUSICBRAINZ_LOGO_URL, label: 'MusicBrainz' },
{ id: artist.deezer_id, url: `https://www.deezer.com/artist/${artist.deezer_id}`, logo: DEEZER_LOGO_URL, label: 'Deezer' },
{ id: artist.audiodb_id, url: `https://www.theaudiodb.com/artist/${artist.audiodb_id}-${adbSlug}`, logo: getAudioDBLogoURL(), label: 'TheAudioDB' },
{ id: artist.lastfm_url, url: artist.lastfm_url, logo: LASTFM_LOGO_URL, label: 'Last.fm' },
{ id: artist.genius_url, url: artist.genius_url, logo: GENIUS_LOGO_URL, label: 'Genius' },
];
sources.forEach(source => {
@ -32289,27 +32311,15 @@ function updateArtistDetailPageHeaderWithData(artist) {
link.href = source.url;
link.target = '_blank';
link.title = `View on ${source.label}`;
link.style.padding = '5px 14px';
link.style.border = '1px solid rgba(255,255,255,0.1)';
link.style.background = 'rgba(0,0,0,0.4)';
link.style.borderRadius = '20px';
link.style.marginLeft = '16px';
link.style.display = 'inline-flex';
link.style.alignItems = 'center';
link.style.textDecoration = 'none';
link.style.verticalAlign = 'middle';
const span = document.createElement('span');
span.style.cssText = `color:rgba(255,255,255,0.9); margin-left:${source.logo ? '10px' : '0'}; font-size: 13px; font-weight: 600; vertical-align: middle;`;
span.textContent = `View on ${source.label}`;
if (source.logo) {
const img = document.createElement('img');
img.src = source.logo;
img.style.cssText = 'height: 24px; width: auto; vertical-align: middle; display: inline-block;';
img.onerror = () => { img.style.display = 'none'; span.style.marginLeft = '0'; };
img.onerror = () => { img.style.display = 'none'; };
link.appendChild(img);
}
const span = document.createElement('span');
span.textContent = source.label;
link.appendChild(span);
mainTitle.appendChild(link);
});

108
webui/static/style.css
View file

@ -28250,36 +28250,24 @@ body {
vertical-align: middle;
}
.mb-card-icon {
/* Legacy .mb-card-icon override removed — now handled by .source-card-icon inside .card-badge-container */
/* Badge container for artist cards */
.card-badge-container {
position: absolute;
top: 8px;
right: 8px;
width: 28px;
height: 28px;
background: rgba(0, 0, 0, 0.75);
/* MusicBrainz Purple - Force visible color */
border: 1px solid rgba(255, 255, 255, 0.3);
border-radius: 4px;
display: flex;
align-items: center;
justify-content: center;
opacity: 0.9 !important;
/* Force visible */
transition: transform 0.2s;
cursor: pointer;
z-index: 100 !important;
/* Force on top */
color: white;
font-size: 11px;
font-weight: 800;
box-shadow: 0 2px 4px rgba(0, 0, 0, 0.5);
flex-direction: column;
gap: 4px;
z-index: 100;
max-height: calc(100% - 80px);
overflow: hidden;
}
.source-card-icon {
position: absolute;
right: 8px;
width: 28px;
height: 28px;
width: 24px;
height: 24px;
background: rgba(0, 0, 0, 0.75);
border: 1px solid rgba(255, 255, 255, 0.3);
border-radius: 4px;
@ -28289,10 +28277,10 @@ body {
opacity: 0.9 !important;
transition: transform 0.2s;
cursor: pointer;
z-index: 100 !important;
color: white;
font-size: 11px;
font-size: 9px;
font-weight: 800;
flex-shrink: 0;
box-shadow: 0 2px 4px rgba(0, 0, 0, 0.5);
}
@ -28315,6 +28303,76 @@ body {
background: #1a6bc4;
}
.spotify-card-icon:hover {
background: #1db954;
}
.itunes-card-icon:hover {
background: #fc3c44;
}
.lastfm-card-icon:hover {
background: #d51007;
}
.genius-card-icon:hover {
background: #ffff64;
}
/* Artist detail page source link buttons */
.source-link-btn {
display: inline-flex;
align-items: center;
gap: 8px;
padding: 6px 16px;
margin-left: 12px;
border-radius: 20px;
text-decoration: none;
font-size: 13px;
font-weight: 600;
color: rgba(255, 255, 255, 0.9);
background: rgba(255, 255, 255, 0.08);
border: 1px solid rgba(255, 255, 255, 0.12);
backdrop-filter: blur(8px);
transition: all 0.2s ease;
vertical-align: middle;
font-family: 'SF Pro Text', -apple-system, BlinkMacSystemFont, sans-serif;
}
.source-link-btn:hover {
background: rgba(255, 255, 255, 0.15);
border-color: rgba(255, 255, 255, 0.25);
transform: translateY(-1px);
box-shadow: 0 4px 12px rgba(0, 0, 0, 0.3);
}
.source-link-btn img {
height: 20px;
width: auto;
display: inline-block;
vertical-align: middle;
}
.source-link-btn span {
vertical-align: middle;
}
@media (max-width: 768px) {
.source-link-btn {
padding: 4px 10px;
margin-left: 6px;
font-size: 0;
}
.source-link-btn img {
height: 18px;
}
.source-link-btn span {
display: none;
}
}
.watchlist-card-icon {
position: absolute;
right: 8px;