Skip to content

fix: consultation-popup - Open the Consultation popup from the hero, header and phase CTA buttons (LSA-214) #161

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
brandonmarshal merged 2 commits into from
Oct 26, 2026
Merged

fix: consultation-popup - Open the Consultation popup from the hero, header and phase CTA buttons (LSA-214) #161

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b97fce0
fix: consultation-popup - Open the Consultation popup from the hero, …
brandonmarshal Oct 26, 2026
b872461
docs: changelog - Add the consultation popup buttons entry (LSA-214)
brandonmarshal Oct 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

---

## [Unreleased] — Open the Consultation popup from the hero, header and phase CTA buttons (LSA-214)

### Changed

- The consultation buttons in the hero patterns (`hero-split-glance`, `hero-split-index-card`, `hero-split-proof`, `phase-hero`, `service-hero`, `services-hero`, `solutions-hero`, `work-hero`, `work-single-hero`), `home-hero-static`, `home-hero` (the inline link), `header` and `sections/phase-cta` now open the Consultation popup instead of leaving the page, so Form 4 entries record the page the visitor was on. Each button keeps its link to `/free-consultation/` as a fallback if the popup or JavaScript fails.
- The "Request a discovery workshop" button in `sections/phase-cta` stays a plain link, because the popup form cannot carry the workshop type.

([#161](https://github.com/lightspeedwp/ls-theme/pull/161))

---

## [Unreleased] — Rebuild the thank-you pattern as a reusable confirmation section (LSA-198)

### Added
Expand Down
4 changes: 2 additions & 2 deletions patterns/header.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@
<!-- /wp:group -->

<!-- wp:buttons {"style":{"spacing":{"blockGap":"0"}}} -->
<div class="wp-block-buttons"><!-- wp:button {"className":"ls-button-cta-gradient","style":{"border":{"color":"var:custom|color|border|card","radius":"var:preset|border-radius|500","style":"solid","width":"1px"},"color":{"gradient":"linear-gradient(135deg, var(--wp--custom--color--phase--create) 0%, var(--wp--custom--color--link--accent) 60%, var(--wp--custom--color--link--accent) 100%)","text":"var:custom|color|button|fill|text"},"spacing":{"padding":{"top":"var:preset|spacing|10","right":"var:preset|spacing|20","bottom":"var:preset|spacing|10","left":"var:preset|spacing|20"}},"typography":{"fontWeight":"var:custom|typography|font-weight|bold"},"dimensions":{"minHeight":"var:custom|spacing|tap-target-min"}},"fontSize":"200"} -->
<div class="wp-block-button ls-button-cta-gradient" style="min-height:var(--wp--custom--spacing--tap-target-min)"><a class="wp-block-button__link has-text-color has-background has-border-color has-200-font-size has-custom-font-size wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>" style="border-color:var(--wp--custom--color--border--card);border-style:solid;border-width:1px;border-radius:var(--wp--preset--border-radius--500);color:var(--wp--custom--color--button--fill--text);background:linear-gradient(135deg, var(--wp--custom--color--phase--create) 0%, var(--wp--custom--color--link--accent) 60%, var(--wp--custom--color--link--accent) 100%);padding-top:var(--wp--preset--spacing--10);padding-right:var(--wp--preset--spacing--20);padding-bottom:var(--wp--preset--spacing--10);padding-left:var(--wp--preset--spacing--20);font-weight:var(--wp--custom--typography--font-weight--bold)"><?php echo esc_html__( 'Start a project →', 'ls-theme' ); ?></a></div>
<div class="wp-block-buttons"><!-- wp:button {"className":"ls-button-cta-gradient pum-triggercta","style":{"border":{"color":"var:custom|color|border|card","radius":"var:preset|border-radius|500","style":"solid","width":"1px"},"color":{"gradient":"linear-gradient(135deg, var(--wp--custom--color--phase--create) 0%, var(--wp--custom--color--link--accent) 60%, var(--wp--custom--color--link--accent) 100%)","text":"var:custom|color|button|fill|text"},"spacing":{"padding":{"top":"var:preset|spacing|10","right":"var:preset|spacing|20","bottom":"var:preset|spacing|10","left":"var:preset|spacing|20"}},"typography":{"fontWeight":"var:custom|typography|font-weight|bold"},"dimensions":{"minHeight":"var:custom|spacing|tap-target-min"}},"fontSize":"200"} -->
<div class="wp-block-button ls-button-cta-gradient pum-triggercta" style="min-height:var(--wp--custom--spacing--tap-target-min)"><a class="wp-block-button__link has-text-color has-background has-border-color has-200-font-size has-custom-font-size wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>" style="border-color:var(--wp--custom--color--border--card);border-style:solid;border-width:1px;border-radius:var(--wp--preset--border-radius--500);color:var(--wp--custom--color--button--fill--text);background:linear-gradient(135deg, var(--wp--custom--color--phase--create) 0%, var(--wp--custom--color--link--accent) 60%, var(--wp--custom--color--link--accent) 100%);padding-top:var(--wp--preset--spacing--10);padding-right:var(--wp--preset--spacing--20);padding-bottom:var(--wp--preset--spacing--10);padding-left:var(--wp--preset--spacing--20);font-weight:var(--wp--custom--typography--font-weight--bold)"><?php echo esc_html__( 'Start a project →', 'ls-theme' ); ?></a></div>
<!-- /wp:button --></div>
<!-- /wp:buttons --></div>
<!-- /wp:group --></div>
Expand Down
4 changes: 2 additions & 2 deletions patterns/hero/hero-split-glance.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,8 +77,8 @@

<!-- wp:buttons -->
<div class="wp-block-buttons">
<!-- wp:button -->
<div class="wp-block-button"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a design-system audit', 'ls-theme' ); ?></a></div>
<!-- wp:button {"className":"pum-triggercta"} -->
<div class="wp-block-button pum-triggercta"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a design-system audit', 'ls-theme' ); ?></a></div>
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Resolved
<!-- /wp:button -->

<!-- wp:button {"className":"is-style-outline"} -->
Expand Down
4 changes: 2 additions & 2 deletions patterns/hero/hero-split-index-card.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -122,8 +122,8 @@

<!-- wp:buttons {"style":{"spacing":{"margin":{"top":"var:preset|spacing|20"}}}} -->
<div class="wp-block-buttons" style="margin-top:var(--wp--preset--spacing--20)">
<!-- wp:button -->
<div class="wp-block-button"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a consultation', 'ls-theme' ); ?></a></div>
<!-- wp:button {"className":"pum-triggercta"} -->
<div class="wp-block-button pum-triggercta"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a consultation', 'ls-theme' ); ?></a></div>
<!-- /wp:button -->

<!-- wp:button {"className":"is-style-outline"} -->
Expand Down
4 changes: 2 additions & 2 deletions patterns/hero/hero-split-proof.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,8 +75,8 @@

<!-- wp:buttons -->
<div class="wp-block-buttons">
<!-- wp:button -->
<div class="wp-block-button"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a consultation', 'ls-theme' ); ?></a></div>
<!-- wp:button {"className":"pum-triggercta"} -->
<div class="wp-block-button pum-triggercta"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a consultation', 'ls-theme' ); ?></a></div>
<!-- /wp:button -->

<!-- wp:button {"className":"is-style-outline"} -->
Expand Down
4 changes: 2 additions & 2 deletions patterns/hero/phase-hero.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,8 @@

<!-- wp:buttons {"layout":{"type":"flex","justifyContent":"center"},"style":{"spacing":{"margin":{"top":"var:preset|spacing|40"},"blockGap":"var:preset|spacing|20"}}} -->
<div class="wp-block-buttons" style="margin-top:var(--wp--preset--spacing--40)">
<!-- wp:button {"className":"is-style-button-phase-primary"} -->
<div class="wp-block-button is-style-button-phase-primary"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a Discovery Call', 'ls-theme' ); ?></a></div>
<!-- wp:button {"className":"is-style-button-phase-primary pum-triggercta"} -->
<div class="wp-block-button is-style-button-phase-primary pum-triggercta"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a Discovery Call', 'ls-theme' ); ?></a></div>
<!-- /wp:button -->

<!-- wp:button {"className":"is-style-button-phase-outline"} -->
Expand Down
4 changes: 2 additions & 2 deletions patterns/hero/service-hero.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,8 +84,8 @@

<!-- wp:buttons {"style":{"spacing":{"margin":{"top":"var:preset|spacing|50"},"blockGap":"var:preset|spacing|20"}},"layout":{"type":"flex","justifyContent":"left"}} -->
<div class="wp-block-buttons" style="margin-top:var(--wp--preset--spacing--50)">
<!-- wp:button {"className":"is-style-button-phase-primary"} -->
<div class="wp-block-button is-style-button-phase-primary"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a free consultation', 'ls-theme' ); ?></a></div>
<!-- wp:button {"className":"is-style-button-phase-primary pum-triggercta"} -->
<div class="wp-block-button is-style-button-phase-primary pum-triggercta"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a free consultation', 'ls-theme' ); ?></a></div>
<!-- /wp:button -->

<!-- wp:button {"className":"is-style-button-phase-outline"} -->
Expand Down
4 changes: 2 additions & 2 deletions patterns/hero/services-hero.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -164,8 +164,8 @@

<!-- wp:buttons {"className":"ls-buttons-wrap","style":{"spacing":{"margin":{"top":"var:preset|spacing|30"}}}} -->
<div class="wp-block-buttons ls-buttons-wrap" style="margin-top:var(--wp--preset--spacing--30)">
<!-- wp:button -->
<div class="wp-block-button"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Request a systems review', 'ls-theme' ); ?></a></div>
<!-- wp:button {"className":"pum-triggercta"} -->
<div class="wp-block-button pum-triggercta"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Request a systems review', 'ls-theme' ); ?></a></div>
<!-- /wp:button -->

<!-- wp:button {"className":"is-style-outline"} -->
Expand Down
4 changes: 2 additions & 2 deletions patterns/hero/solutions-hero.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,8 +55,8 @@

<!-- wp:buttons -->
<div class="wp-block-buttons">
<!-- wp:button -->
<div class="wp-block-button"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Discuss your platform', 'ls-theme' ); ?></a></div>
<!-- wp:button {"className":"pum-triggercta"} -->
<div class="wp-block-button pum-triggercta"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Discuss your platform', 'ls-theme' ); ?></a></div>
<!-- /wp:button -->

<!-- wp:button {"className":"is-style-outline"} -->
Expand Down
4 changes: 2 additions & 2 deletions patterns/hero/work-hero.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,8 @@

<!-- wp:buttons {"style":{"spacing":{"margin":{"top":"var:preset|spacing|30"}}}} -->
<div class="wp-block-buttons" style="margin-top:var(--wp--preset--spacing--30)">
<!-- wp:button -->
<div class="wp-block-button"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a consultation', 'ls-theme' ); ?></a></div>
<!-- wp:button {"className":"pum-triggercta"} -->
<div class="wp-block-button pum-triggercta"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a consultation', 'ls-theme' ); ?></a></div>
<!-- /wp:button -->

<!-- wp:button {"className":"is-style-outline"} -->
Expand Down
4 changes: 2 additions & 2 deletions patterns/hero/work-single-hero.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,8 @@

<!-- wp:buttons {"style":{"spacing":{"margin":{"top":"var:preset|spacing|20"}}}} -->
<div class="wp-block-buttons" style="margin-top:var(--wp--preset--spacing--20)">
<!-- wp:button -->
<div class="wp-block-button"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a consultation', 'ls-theme' ); ?></a></div>
<!-- wp:button {"className":"pum-triggercta"} -->
<div class="wp-block-button pum-triggercta"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a consultation', 'ls-theme' ); ?></a></div>
<!-- /wp:button -->

<!-- wp:button {"className":"is-style-outline ls-view-site-button","linkTarget":"_blank","rel":"noopener noreferrer"} -->
Expand Down
4 changes: 2 additions & 2 deletions patterns/home-hero-static.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,8 @@

<!-- wp:buttons {"className":"ls-buttons-wrap","layout":{"type":"flex","justifyContent":"center","flexWrap":"wrap"},"style":{"spacing":{"margin":{"top":"var:preset|spacing|60"}}}} -->
<div class="wp-block-buttons ls-buttons-wrap" style="margin-top:var(--wp--preset--spacing--60)">
<!-- wp:button {"className":"ls-hero-cta-fill"} -->
<div class="wp-block-button ls-hero-cta-fill"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a consultation', 'ls-theme' ); ?></a></div>
<!-- wp:button {"className":"ls-hero-cta-fill pum-triggercta"} -->
<div class="wp-block-button ls-hero-cta-fill pum-triggercta"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>"><?php echo esc_html__( 'Book a consultation', 'ls-theme' ); ?></a></div>
<!-- /wp:button -->

<!-- wp:button {"className":"is-style-outline ls-hero-cta-outline"} -->
Expand Down
2 changes: 1 addition & 1 deletion patterns/home-hero.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@
<!-- /wp:group -->

<!-- wp:paragraph {"align":"center","style":{"color":{"text":"var(--wp--custom--color--text--on-dark-muted)"},"spacing":{"margin":{"top":"var:preset|spacing|20"}}},"fontSize":"200"} -->
<p class="has-text-align-center has-text-color has-200-font-size" style="color:var(--wp--custom--color--text--on-dark-muted);margin-top:var(--wp--preset--spacing--20)"><?php echo esc_html__( 'Prefer to speak directly?', 'ls-theme' ); ?> <a href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>" style="color:var(--wp--custom--color--text--on-dark-accent)"><?php echo esc_html__( 'Book your consultation instead', 'ls-theme' ); ?></a></p>
<p class="has-text-align-center has-text-color has-200-font-size" style="color:var(--wp--custom--color--text--on-dark-muted);margin-top:var(--wp--preset--spacing--20)"><?php echo esc_html__( 'Prefer to speak directly?', 'ls-theme' ); ?> <a class="pum-triggercta" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>" style="color:var(--wp--custom--color--text--on-dark-accent)"><?php echo esc_html__( 'Book your consultation instead', 'ls-theme' ); ?></a></p>
<!-- /wp:paragraph -->
</div>
<!-- /wp:group -->
Expand Down
4 changes: 2 additions & 2 deletions patterns/sections/phase-cta.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,8 @@

<!-- wp:buttons {"className":"ls-buttons-wrap","layout":{"type":"flex","flexWrap":"wrap"},"style":{"spacing":{"margin":{"top":"var:preset|spacing|20"}}}} -->
<div class="wp-block-buttons ls-buttons-wrap" style="margin-top:var(--wp--preset--spacing--20)">
<!-- wp:button {"className":"is-style-button-phase-primary","style":{"spacing":{"padding":{"right":"var:preset|spacing|30","left":"var:preset|spacing|30"}}}} -->
<div class="wp-block-button is-style-button-phase-primary"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>" style="padding-right:var(--wp--preset--spacing--30);padding-left:var(--wp--preset--spacing--30)"><?php echo esc_html__( 'Book a consultation', 'ls-theme' ); ?></a></div>
<!-- wp:button {"className":"is-style-button-phase-primary pum-triggercta","style":{"spacing":{"padding":{"right":"var:preset|spacing|30","left":"var:preset|spacing|30"}}}} -->
<div class="wp-block-button is-style-button-phase-primary pum-triggercta"><a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( home_url( '/free-consultation/' ) ); ?>" style="padding-right:var(--wp--preset--spacing--30);padding-left:var(--wp--preset--spacing--30)"><?php echo esc_html__( 'Book a consultation', 'ls-theme' ); ?></a></div>
<!-- /wp:button -->

<!-- wp:button {"className":"is-style-button-phase-outline","style":{"spacing":{"padding":{"right":"var:preset|spacing|30","left":"var:preset|spacing|30"}}}} -->
Expand Down
Loading