Skip to content

[5.x] Variant field layout tabs displayed incorrectly#4336

Merged
nfourtythree merged 2 commits into
5.xfrom
nathaniel/com-591-5x-variant-slide-out-tabs-not-switching-when-variant-has-a
Jul 16, 2026
Merged

[5.x] Variant field layout tabs displayed incorrectly#4336
nfourtythree merged 2 commits into
5.xfrom
nathaniel/com-591-5x-variant-slide-out-tabs-not-switching-when-variant-has-a

Conversation

@nfourtythree

Copy link
Copy Markdown
Contributor

Related issues

#4335

@nfourtythree
nfourtythree requested a review from a team as a code owner July 16, 2026 14:32
@linear-code

linear-code Bot commented Jul 16, 2026

Copy link
Copy Markdown

COM-591

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes a Craft Commerce control panel UI issue where variant slide-out field layout tabs don’t switch correctly when the variant has related sales, by correcting HTML structure in the purchasable price field output.

Changes:

  • Adjusted the generated markup in PurchasablePriceField to ensure the price/sales toggle content is wrapped with properly balanced tags.
  • Added a changelog entry referencing issue #4335.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/fieldlayoutelements/PurchasablePriceField.php Fixes HTML tag balancing/structure around the toggle content in the purchasable price field to prevent CP UI layout/tab issues.
CHANGELOG.md Documents the fix in the Unreleased section with a link to the reported issue.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/fieldlayoutelements/PurchasablePriceField.php
@nfourtythree
nfourtythree merged commit 16650bd into 5.x Jul 16, 2026
14 checks passed
@nfourtythree
nfourtythree deleted the nathaniel/com-591-5x-variant-slide-out-tabs-not-switching-when-variant-has-a branch July 16, 2026 15:31
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