Fix mobile handling for score and document icons

- Score icon: mobile downloads PDF, desktop opens modal
- Document icon: reverts to same behavior on mobile and desktop (opens modal)
- Remove console.log from isMobile
- Remove documentLink computed property
This commit is contained in:
Michael Winter 2026-03-06 18:36:40 +01:00
parent 49f427256f
commit 1bbb7e0250
2 changed files with 21 additions and 16 deletions

View file

@ -3,24 +3,19 @@
<div v-show="visible" class="bg-black rounded-full text-xs inline-flex" > <div v-show="visible" class="bg-black rounded-full text-xs inline-flex" >
<!-- Score: Mobile = PDF (downloads), Desktop = work page with hash (opens modal) --> <!-- Score: Mobile = PDF (downloads), Desktop = work page with hash (opens modal) -->
<span v-if="type === 'score'" class="md:hidden"> <span v-if="type === 'score' && isMobile()">
<a :href="scoreLink" class="inline-flex p-1"> <a :href="scoreLink" class="inline-flex p-1">
<Icon name="ion:book-sharp" style="color: white" /> <Icon name="ion:book-sharp" style="color: white" />
</a> </a>
</span> </span>
<span v-if="type === 'score'" class="hidden md:block"> <span v-else-if="type === 'score'">
<a :href="'/works/' + workSlug + '#score'" class="inline-flex p-1"> <button @click="openWithHash('score')" class="inline-flex p-1">
<Icon name="ion:book-sharp" style="color: white" /> <Icon name="ion:book-sharp" style="color: white" />
</a> </button>
</span> </span>
<!-- Document: Mobile = PDF (downloads), Desktop = modal --> <!-- Document: Mobile = External opens new tab, PDF downloads, internal pages open modal -->
<span v-if="type === 'document'" class="md:hidden"> <span v-if="type === 'document'">
<a :href="link" class="inline-flex p-1 cursor-pointer">
<Icon name="ion:book-sharp" style="color: white" />
</a>
</span>
<span v-if="type === 'document'" class="hidden md:block">
<a :href="isExternalLink ? link : undefined" :target="isExternalLink ? '_blank' : undefined" :rel="isExternalLink ? 'noopener noreferrer' : undefined" @click="openDocument()" class="inline-flex p-1 cursor-pointer"> <a :href="isExternalLink ? link : undefined" :target="isExternalLink ? '_blank' : undefined" :rel="isExternalLink ? 'noopener noreferrer' : undefined" @click="openDocument()" class="inline-flex p-1 cursor-pointer">
<Icon name="ion:book-sharp" style="color: white" /> <Icon name="ion:book-sharp" style="color: white" />
</a> </a>
@ -87,6 +82,11 @@
return props.link && props.link.startsWith('/') && !props.link.endsWith('.pdf') return props.link && props.link.startsWith('/') && !props.link.endsWith('.pdf')
}) })
const isMobile = () => {
if (typeof window === 'undefined') return false
return window.innerWidth < 768
}
const openDocument = () => { const openDocument = () => {
if (props.link?.endsWith('.pdf')) { if (props.link?.endsWith('.pdf')) {
modalStore.setModalProps('pdf', 'aspect-[1/1.414]', true, '', '', '', props.link) modalStore.setModalProps('pdf', 'aspect-[1/1.414]', true, '', '', '', props.link)

View file

@ -8,11 +8,11 @@
<nav class="flex gap-4 mb-6"> <nav class="flex gap-4 mb-6">
<a v-if="work.vimeo_trackid" href="#video" class="hover:underline">Video</a> <a v-if="work.vimeo_trackid" href="#video" class="hover:underline">Video</a>
<a v-if="gallery && gallery.length" href="#images" class="hover:underline">Images</a> <a v-if="gallery && gallery.length" href="#images" class="hover:underline">Images</a>
<span class="md:hidden"> <span v-if="scoreUrl && isMobile()">
<a v-if="scoreUrl" :href="scoreUrl" class="hover:underline">Score</a> <a :href="scoreUrl" class="hover:underline">Score</a>
</span> </span>
<span class="hidden md:block"> <span v-else-if="scoreUrl">
<a v-if="scoreUrl" href="#score" class="hover:underline">Score</a> <a href="#score" class="hover:underline">Score</a>
</span> </span>
</nav> </nav>
</div> </div>
@ -27,7 +27,7 @@
<ImageSlider :bucket="'images'" :gallery="gallery"></ImageSlider> <ImageSlider :bucket="'images'" :gallery="gallery"></ImageSlider>
</div> </div>
<div id="score" v-if="scoreUrl" class="mb-8 scroll-mt-[280px] hidden md:block"> <div id="score" v-if="scoreUrl && !isMobile()" class="mb-8 scroll-mt-[280px]">
<iframe :src="scoreUrl + '#toolbar=1&navpanes=0&sidebar=0'" class="w-full h-[85vh] border"></iframe> <iframe :src="scoreUrl + '#toolbar=1&navpanes=0&sidebar=0'" class="w-full h-[85vh] border"></iframe>
</div> </div>
</div> </div>
@ -82,6 +82,11 @@ const itemCount = computed(() => {
return count return count
}) })
const isMobile = () => {
if (typeof window === 'undefined') return false
return window.innerWidth < 768
}
onMounted(() => { onMounted(() => {
if (work.value?.soundcloud_trackid) { if (work.value?.soundcloud_trackid) {
audioPlayerStore.setSoundCloudTrackID(work.value.soundcloud_trackid) audioPlayerStore.setSoundCloudTrackID(work.value.soundcloud_trackid)