diff --git a/frontend/src/components/discovery/ArtistCard.vue b/frontend/src/components/discovery/ArtistCard.vue index dc2f5ae..813aeda 100644 --- a/frontend/src/components/discovery/ArtistCard.vue +++ b/frontend/src/components/discovery/ArtistCard.vue @@ -49,9 +49,11 @@ function onCardClick() { position: relative; display: grid; grid-template-columns: repeat(3, 1fr); gap: 2px; aspect-ratio: 3 / 1; - /* Explicit floor + ceiling so tall source images can't escape the - preview slot even on browsers where aspect-ratio doesn't compute. */ - min-height: 150px; max-height: 220px; + /* Floor so tall source images can't escape the slot. NO max-height: an + aspect-ratio box capped by max-height shrinks its own WIDTH to keep the + ratio, leaving dead space beside the previews on a wide (single-column) + card. The grid below keeps columns ≲400px so the strip stays a sane height. */ + min-height: 120px; overflow: hidden; background: rgb(var(--v-theme-surface-light)); } diff --git a/frontend/src/stores/artistDirectory.js b/frontend/src/stores/artistDirectory.js index baede1d..950be2f 100644 --- a/frontend/src/stores/artistDirectory.js +++ b/frontend/src/stores/artistDirectory.js @@ -14,6 +14,10 @@ export const useArtistDirectoryStore = defineStore('artistDirectory', () => { const q = ref('') const platform = ref(null) let started = false + // Has a load ATTEMPT completed yet? Gates the "No artists match" empty state + // so it can't flash on the very first render (loading is still false before + // onMounted fires the first loadMore) or between a query change and its fetch. + const loaded = ref(false) // Typed "alice" then "alice bob" used to drop the second fetch // entirely (loading flag still true from the first), so the UI // showed alice results while the input said "alice bob". Inflight @@ -35,6 +39,7 @@ export const useArtistDirectoryStore = defineStore('artistDirectory', () => { nextCursor.value = body.next_cursor started = true }) + if (t.isCurrent()) loaded.value = true } async function reset() { @@ -42,6 +47,7 @@ export const useArtistDirectoryStore = defineStore('artistDirectory', () => { cards.value = [] nextCursor.value = null started = false + loaded.value = false await loadMore() } @@ -50,7 +56,8 @@ export const useArtistDirectoryStore = defineStore('artistDirectory', () => { const hasMore = computed(() => !started || nextCursor.value !== null) const isEmpty = computed( - () => !loading.value && cards.value.length === 0 && error.value === null + () => loaded.value && !loading.value + && cards.value.length === 0 && error.value === null ) return { diff --git a/frontend/src/views/ArtistsView.vue b/frontend/src/views/ArtistsView.vue index 3e3c613..f5582f3 100644 --- a/frontend/src/views/ArtistsView.vue +++ b/frontend/src/views/ArtistsView.vue @@ -76,9 +76,11 @@ onMounted(async () => { } .fc-artists__grid { display: grid; - /* min(440px, 100%) so a card never exceeds the viewport — on phones the - min-track collapses to 100% (single column) instead of overflowing. */ - grid-template-columns: repeat(auto-fill, minmax(min(440px, 100%), 1fr)); + /* min(360px, 100%) so a card never exceeds the viewport (phones collapse to + one column). 360 keeps a LONE column under ~732px wide, so the 3/1 preview + strip stays ≲244px tall and always fills the card width (no dead space), + while giving 2+ columns on a normal desktop. */ + grid-template-columns: repeat(auto-fill, minmax(min(360px, 100%), 1fr)); gap: 12px; } .fc-artists__sentinel {