Skip to content

Marge main#726

Merged
dylandepass merged 15 commits into
staginguatfrom
marge-main
Jul 20, 2026
Merged

Marge main#726
dylandepass merged 15 commits into
staginguatfrom
marge-main

Conversation

@dylandepass

@dylandepass dylandepass commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

awasthiruchi and others added 15 commits July 8, 2026 15:00
Removed old Google Tag Manager configuration and updated to new configuration.
Update Google Tag Manager script configuration
fix: horizontal scrollbar caused by video carousel
Refresh the PDP alert when variants change and suppress purchase and local-dealer actions for products that are not yet available.
fix(pdp): support coming-soon variants
Clicking a product image in the PDP gallery now opens a full-size modal view. Video slides are excluded since they have their own click behavior. The modal reuses the existing createModal infrastructure for consistent close-button and backdrop-click dismissal.
Add support for message type in form handling.
…only

The image modal now sizes to its content with no overflow scrollbar. The close button sits outside the top-right corner of the image. The modal and pointer cursor are limited to desktop viewports (≥900px) since gallery images are already near full-width on mobile.
…age-should-di

feat(pdp): open image modal when clicking gallery images
@aem-code-sync

aem-code-sync Bot commented Jul 20, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run all PSI checks
  • Re-run failed PSI checks
  • Re-sync branch
Commits

@dylandepass
dylandepass merged commit 224e87e into staginguat Jul 20, 2026
2 of 3 checks passed
@dylandepass
dylandepass deleted the marge-main branch July 20, 2026 12:54
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.

3 participants