Skip to content

updated css for registration forms to resize and reposition captcha icon#20005

Open
jkramer-debug wants to merge 1 commit into
masterfrom
jkramer/update-workshop-signup-recaptcha
Open

updated css for registration forms to resize and reposition captcha icon#20005
jkramer-debug wants to merge 1 commit into
masterfrom
jkramer/update-workshop-signup-recaptcha

Conversation

@jkramer-debug

@jkramer-debug jkramer-debug commented Jul 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Proposed changes

Updated hubspot scss to resize and relocate the reCAPTCHA icon on the registration forms for workshops. Can test going to the events page and clicking on one. http://www-testing-pulumi-docs-origin-pr-20005-e5fb79ba.s3-website.us-west-2.amazonaws.com/events/autonomous-agents-need-guardrails-openclaw/

Unreleased product version (optional)

N/A

Related issues (optional)

N/A

@pulumi-bot

pulumi-bot commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Your site preview for commit e5fb79b is ready! 🎉

http://www-testing-pulumi-docs-origin-pr-20005-e5fb79ba.s3-website.us-west-2.amazonaws.com

@pulumi-bot

pulumi-bot commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Lighthouse Performance Report

Commit: e5fb79b | Metric definitions

Page Device Score FCP LCP TBT CLS SI
Homepage Mobile 🔴 33 6.1s 7.4s 1413ms 0.001 8.2s
Homepage Desktop 🟡 82 0.8s 2.3s 188ms 0.019 1.0s
Install Pulumi Mobile 🟡 58 7.5s 15.0s 45ms 0.000 7.5s
Install Pulumi Desktop 🟡 78 1.1s 2.6s 0ms 0.005 2.7s
AWS Get Started Mobile 🟡 58 7.2s 14.5s 48ms 0.000 7.2s
AWS Get Started Desktop 🟡 85 1.1s 1.7s 0ms 0.036 2.6s

@jkramer-debug

jkramer-debug commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

@claude take a look please

@pulumi-bot

pulumi-bot commented Jul 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Claude finished @jkramer-debug's task in 1m 56s —— View job

Todo

  • Gather context (PR diff, changed file)
  • Review the SCSS change for correctness and side effects
  • Check for global class collisions (.grecaptcha-badge) elsewhere in the codebase
  • Post review feedback

Review

Looked at the diff in theme/src/scss/_hubspot.scss:180-197. The change itself is small and technically sound:

  • .event-single-registration .hs-form gets position: relative + padding-bottom: 3.5rem, and .hs-recaptcha inside it is absolutely positioned to bottom: 0; right: 0; with @apply m-0 clearing the existing mb-5 from the base .hs-recaptcha rule (line 139). That's a correct, standard "reserve space + pin the badge" pattern, and mixing @apply with plain CSS properties matches the style already used elsewhere in this file (e.g. the .newsletter block).
  • Selector nesting and naming follow the existing conventions in the file.

One thing worth double-checking with the author before merge:

  • .grecaptcha-badge (line 194) is a global, unscoped selector, so transform: scale(0.6) applies to the reCAPTCHA badge on any page that loads it — not just the workshop/event registration pages. This is likely unavoidable in practice, since Google typically injects .grecaptcha-badge as a fixed element on <body> rather than nesting it inside the form container, so it can't be scoped under .event-single-registration. But it does mean this PR will also shrink/reposition the badge on other HubSpot-form pages (e.g. contact-us, enterprise, self-hosted request-demo, gads landing pages, puluminaries) — worth a quick visual check on one of those pages to confirm the smaller badge still looks correct there, since the PR description scopes the intent to "registration forms for workshops" but the CSS effect is site-wide.

No other issues found — no lint/build was run against this change since it's a low-risk, isolated CSS addition and stylelint required interactive approval in this sandboxed environment; the author's site preview is the best way to confirm visual correctness across pages.

@jkramer-debug

jkramer-debug commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

Claude finished @jkramer-debug's task in 1m 56s —— View job

Todo

  • Gather context (PR diff, changed file)
  • Review the SCSS change for correctness and side effects
  • Check for global class collisions (.grecaptcha-badge) elsewhere in the codebase
  • Post review feedback

Review

Looked at the diff in theme/src/scss/_hubspot.scss:180-197. The change itself is small and technically sound:

  • .event-single-registration .hs-form gets position: relative + padding-bottom: 3.5rem, and .hs-recaptcha inside it is absolutely positioned to bottom: 0; right: 0; with @apply m-0 clearing the existing mb-5 from the base .hs-recaptcha rule (line 139). That's a correct, standard "reserve space + pin the badge" pattern, and mixing @apply with plain CSS properties matches the style already used elsewhere in this file (e.g. the .newsletter block).
  • Selector nesting and naming follow the existing conventions in the file.

One thing worth double-checking with the author before merge:

  • .grecaptcha-badge (line 194) is a global, unscoped selector, so transform: scale(0.6) applies to the reCAPTCHA badge on any page that loads it — not just the workshop/event registration pages. This is likely unavoidable in practice, since Google typically injects .grecaptcha-badge as a fixed element on <body> rather than nesting it inside the form container, so it can't be scoped under .event-single-registration. But it does mean this PR will also shrink/reposition the badge on other HubSpot-form pages (e.g. contact-us, enterprise, self-hosted request-demo, gads landing pages, puluminaries) — worth a quick visual check on one of those pages to confirm the smaller badge still looks correct there, since the PR description scopes the intent to "registration forms for workshops" but the CSS effect is site-wide.

No other issues found — no lint/build was run against this change since it's a low-risk, isolated CSS addition and stylelint required interactive approval in this sandboxed environment; the author's site preview is the best way to confirm visual correctness across pages.

Checked other the other forms and the workshops the only ones with the recaptcha logo. All the forms look good

@jkramer-debug jkramer-debug marked this pull request as ready for review July 2, 2026 15:52
@github-actions github-actions Bot added review:triaging Claude Triage is currently classifying the PR review:in-progress Claude review is currently running and removed review:triaging Claude Triage is currently classifying the PR labels Jul 2, 2026
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Pre-merge Review — Last updated 2026-07-02T15:54:02Z

Tip

Summary: This PR is a CSS-only change to the HubSpot form styles (theme/src/scss/_hubspot.scss), resizing and repositioning the reCAPTCHA element on the event registration form. The form-scoped rules (.event-single-registration .hs-form reserving space and absolutely positioning .hs-recaptcha in the bottom-right) are correct and well-contained. The one thing worth confirming: the .grecaptcha-badge scaling rule is written as a global selector, so it shrinks Google's floating badge on every page that renders it, not just the registration form — flagged below as low-confidence in case a site-wide change wasn't intended. No factual claims, content, or code examples are involved, so verification passes were limited to a mechanics review of the diff.

Review confidence:

Dimension Level Notes
mechanics HIGH Small, valid SCSS diff; selectors and properties are well-formed.
facts HIGH No factual claims in a CSS-only change.
Investigation log
  • Cross-sibling reads: not run (not in a templated section)
  • External claim verification: not run (no claims in this diff)
  • Cited-claim spot-checks: not run (no cited claims)
  • Frontmatter sweep: not run (no frontmatter in diff)
  • Temporal-trigger sweep: not run (no trigger words)
  • Code execution: not run (no static/programs/ change)
  • Code-examples checks: not run (no fenced code blocks in content files)
  • Editorial-balance pass: not run (not under content/blog/)
🚨 Outstanding ⚠️ Low-confidence 💡 Pre-existing ✅ Resolved
0 1 0 0

🔍 Verification trail

  • L194 in theme/src/scss/_hubspot.scss ".grecaptcha-badge scale rule is a global selector, not scoped to the event registration form" → 🤷 unverifiable (can't confirm from the diff whether reCAPTCHA renders on other pages/forms; source: PR diff)

🚨 Outstanding in this PR

No outstanding findings in this PR.

⚠️ Low-confidence

Review each and resolve as appropriate — these don't block the PR.

  • [L194] theme/src/scss/_hubspot.scss — The .grecaptcha-badge rule scales the badge to 60% and re-anchors its transform origin to bottom right using a global selector, unscoped to .event-single-registration. Google injects a single floating badge for reCAPTCHA v3, so this shrinks it on every page that shows it — not just the registration form. The rest of the change is correctly scoped under .event-single-registration .hs-form, so if the intent was to keep this fix contained to that form (other HubSpot forms in this file, e.g. .waitlist-form, may also use reCAPTCHA), consider nesting the badge rule under that ancestor too:

    .event-single-registration {
    .grecaptcha-badge {
        transform: scale(0.6);
        transform-origin: bottom right;
    }
}

    If a site-wide badge shrink is deliberate, the current global rule is fine as written — just worth confirming the scope is intended.

💡 Pre-existing issues in touched files (optional)

No pre-existing issues in touched files.

✅ Resolved since last review

No items resolved since the last review.

📜 Review history

  • 2026-07-02T15:54:02Z — CSS-only reCAPTCHA reposition/resize; form-scoped rules are clean, flagged the global .grecaptcha-badge scale for scope confirmation (e5fb79b)

Need a re-review? Want to dispute a finding? Mention @claude and include #update-review.
(For ad-hoc questions or fixes, just @claude — no hashtag.)

@github-actions github-actions Bot added review:no-blockers Claude review completed cleanly; outstanding is empty and removed review:in-progress Claude review is currently running labels Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

review:no-blockers Claude review completed cleanly; outstanding is empty

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jkramer-debug @pulumi-bot