Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Feat: Double Click on Image Assets to open the lightbox (#6955)
## Summary

Make it easier to open the lightbox preview.

## Review Focus

The first click still selects the image. I liked the suggestion to
require shift or ctrl to initiate selection and have click go back to
the previous behavior.

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-6955-Feat-Double-Click-on-Image-Assets-to-open-the-lightbox-2b76d73d365081f0bf0de847add5c820)
by [Unito](https://www.unito.io)
  • Loading branch information
DrJKL authored and github-actions[bot] committed Dec 8, 2025
commit 00d1379285988b1e61ea33b064f95f68e0767a71
65 changes: 24 additions & 41 deletions src/platform/assets/components/MediaAssetCard.vue
Original file line number Diff line number Diff line change
Expand Up @@ -59,35 +59,29 @@

<!-- Media actions - show on hover or when playing -->
<IconGroup v-else-if="showActionsOverlay">
<div v-tooltip.top="$t('mediaAsset.actions.inspect')">
<IconButton
size="sm"
@click.stop="handleZoomClick"
@mouseenter="handleOverlayMouseEnter"
@mouseleave="handleOverlayMouseLeave"
>
<i class="icon-[lucide--zoom-in] size-4" />
</IconButton>
</div>
<div v-tooltip.top="$t('mediaAsset.actions.more')">
<MoreButton
ref="moreButtonRef"
size="sm"
@menu-opened="handleMenuOpened"
@menu-closed="handleMenuClosed"
@mouseenter="handleOverlayMouseEnter"
@mouseleave="handleOverlayMouseLeave"
>
<template #default="{ close }">
<MediaAssetMoreMenu
:close="close"
:show-delete-button="showDeleteButton"
@inspect="handleZoomClick"
@asset-deleted="handleAssetDelete"
/>
</template>
</MoreButton>
</div>
<IconButton
v-tooltip.top="$t('mediaAsset.actions.inspect')"
size="sm"
@click.stop="handleZoomClick"
>
<i class="icon-[lucide--zoom-in] size-4" />
</IconButton>
<MoreButton
ref="moreButtonRef"
v-tooltip.top="$t('mediaAsset.actions.more')"
size="sm"
@menu-opened="handleMenuOpened"
@menu-closed="handleMenuClosed"
>
<template #default="{ close }">
<MediaAssetMoreMenu
:close="close"
:show-delete-button="showDeleteButton"
@inspect="handleZoomClick"
@asset-deleted="handleAssetDelete"
/>
</template>
</MoreButton>
</IconGroup>
</template>

Expand All @@ -101,8 +95,6 @@
size="sm"
:label="String(outputCount)"
@click.stop="handleOutputCountClick"
@mouseenter="handleOverlayMouseEnter"
@mouseleave="handleOverlayMouseLeave"
>
<template #icon>
<i class="icon-[lucide--layers] size-4" />
Expand Down Expand Up @@ -216,7 +208,6 @@ const moreButtonRef = ref<InstanceType<typeof MoreButton>>()
const isVideoPlaying = ref(false)
const isMenuOpen = ref(false)
const showVideoControls = ref(false)
const isOverlayHovered = ref(false)

// Store actual image dimensions
const imageDimensions = ref<{ width: number; height: number } | undefined>()
Expand Down Expand Up @@ -299,7 +290,7 @@ const durationChipClasses = computed(() => {
})

const isCardOrOverlayHovered = computed(
() => isHovered.value || isOverlayHovered.value || isMenuOpen.value
() => isHovered.value || isMenuOpen.value
)

// Show static chips when NOT hovered and NOT playing (normal state)
Expand All @@ -320,14 +311,6 @@ const showActionsOverlay = computed(
(isCardOrOverlayHovered.value || isVideoPlaying.value)
)

const handleOverlayMouseEnter = () => {
isOverlayHovered.value = true
}

const handleOverlayMouseLeave = () => {
isOverlayHovered.value = false
}

const handleZoomClick = () => {
if (asset) {
emit('zoom', asset)
Expand Down
2 changes: 2 additions & 0 deletions src/platform/assets/components/MediaImageTop.vue
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
<template>
<div
class="relative size-full overflow-hidden rounded bg-modal-card-placeholder-background"
@dblclick="emit('view')"
>
<img
v-if="!error"
Expand Down Expand Up @@ -28,6 +29,7 @@ const { asset } = defineProps<{

const emit = defineEmits<{
'image-loaded': [width: number, height: number]
view: []
}>()

const { state, error, isReady } = useImage({
Expand Down
Loading