Conversation
Co-authored-by: gantoine <[email protected]>
Co-authored-by: gantoine <[email protected]>
|
@copilot the hover icon should only show when pointerOnHover is false, ie you're on a non-clickable cover art |
…er is false) Co-authored-by: gantoine <[email protected]>
Done in |
Greptile SummaryThis PR adds a "click to zoom" feature to the common game card component ( Key changes:
Two issues were flagged:
Confidence Score: 4/5
Important Files Changed
Sequence DiagramsequenceDiagram
actor User
participant Card as Base.vue (v-img)
participant ZoomBtn as Zoom Button (v-btn)
participant Dialog as RDialog (showCoverZoom)
User->>Card: Hover over card
activate Card
Card-->>ZoomBtn: isOuterHovering && !pointerOnHover → show button
deactivate Card
User->>ZoomBtn: Click zoom button
ZoomBtn->>ZoomBtn: @click.stop (prevents card click)
ZoomBtn->>Dialog: showCoverZoom = true
activate Dialog
Dialog-->>User: Display full-size cover image
User->>Dialog: Click outside / press ESC
Dialog->>Dialog: closeDialog() → showCoverZoom = false
deactivate Dialog
Last reviewed commit: 89c482c |
Game.vue(console TV view) - click-to-zoom was added to wrong locationshowCoverZoomref toBase.vue(common game card component)pointerOnHoverisfalse(non-clickable covers)RDialogwith full-size cover image@click.stopprevents card navigation from triggering when zoom button is clickedOriginal prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.