Skip to content

Conversation

@mahdiyazdani
Copy link
Member

Description

This update improves the Customizer UX for the Product Card settings on shop archive pages. When users select Layout 1 as the Card Style, the following controls become irrelevant and are now automatically hidden:

  • Card Radius
  • Card Background
  • Border Size
  • Border Color

A reusable callback function has been added, ensuring cleaner logic for style-dependent Customizer controls. This prevents confusion and avoids displaying non-functional or unused options in Layout 1.

Motivation and Context

See #426

Changelog

The changes in this PR can be summarized with this changelog record:

  • Fix: Hide card radius, background, and border controls when Card Style is set to Layout 1.

Content Review

  • The text/copy used in PR has been reviewed by the content team.
  • The URLs introduced in PR have been reviewed by the content team.

QA Review

  1. Go to Customizer → Shop → Product Card.
  2. Locate the Card Style control and select Layout 1.
  3. Confirm that the following controls are hidden:
  • Card Radius
  • Card Background
  • Border Size
  • Border Color
  1. Switch the style to Layout 2 or Layout 3.
  2. Confirm the above controls appear again and behave normally.
  3. Publish changes and verify that no JS or PHP errors occur.

Impact

This PR affects the following parts of the plugin/addons, which may require extra testing:

Self-Testing Checklist

  • I tested both Free and paid version of the plugin.
  • I tested these changes to the best of my abilities with browser console opened and debug.log monitored.
  • I checked situations when users update free/pro in a different order.
  • Backward compatibility with users updating from an older version to the one with these changes has been checked.
  • For the Block Editor related changes, I checked in both Sydney and Botiga theme.
  • For the front-end related changes, I checked in both Sydney and Botiga theme.
  • I tested in different browsers (Chrome/Firefox/Safari/Edge).
  • I tested on Multisite OR this change does not affect Multisite in any way.
  • I tested with LTR and RTL languages.

Screenshots / Screencasts

https://a.supportally.com/i/UrHgeq

@mahdiyazdani mahdiyazdani self-assigned this Nov 20, 2025
@mahdiyazdani mahdiyazdani marked this pull request as ready for review November 21, 2025 14:50
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