Skip to content

[5.x] Hover images in listings - #1429

Open
Jade-GG wants to merge 2 commits into
masterfrom
feature/hover-images-5.x
Open

Jade-GG wants to merge 2 commits into
masterfrom
feature/hover-images-5.x

Conversation

@Jade-GG

@Jade-GG Jade-GG commented Oct 13, 2026

Copy link
Copy Markdown
Collaborator

ref: BORDEX-1505

#1286 but then for 5.x

@royduin

royduin commented Oct 13, 2026

Copy link
Copy Markdown
Member

I thought this was already a v5 feature, now it's again a "breaking" thing? As it's not a Magento feature, maybe we should make this configurable?

@Jade-GG

Jade-GG commented Oct 14, 2026

Copy link
Copy Markdown
Collaborator Author

I thought this was already a v5 feature, now it's again a "breaking" thing? As it's not a Magento feature, maybe we should make this configurable?

This is a weird one. I've been told it's part of FW, but there it's just a custom attribute in Magento, so while it's "implemented" in a v5 project, it's not actually part of the core.

This was supposed to be a v5 feature but then it never got given enough priority (see: RAP-1903). Now that we've gotten to updating the project this was actually for, it turned out to never be in the core and I've had to add it.

I can add a config value for this, however, to keep it fully backwards compatible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants