fix: checkbox state feedback

[dmaduzia]

prevent duplicated toggles on click

Summary by CodeRabbit

• Bug Fixes
  • Modified newsletter consent checkbox interaction. The checkbox now responds to keyboard input (spacebar) only; mouse clicks no longer update the checkbox state.

[Copilot]

Pull Request Overview

This PR fixes a checkbox state management issue by removing a redundant click handler that was causing duplicated toggle behavior. The checkbox value is now properly controlled through Angular's form control binding, eliminating the manual toggle logic.

Key Changes:

• Removed explicit (click) event handler that manually toggled the checkbox value

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[coderabbitai]

Walkthrough

The pull request removes the click event handler from a custom checkbox element in the newsletter component template. The spacebar keydown handler remains intact for keyboard accessibility. Clicking the checkbox no longer updates the form state, while keyboard navigation continues to function.

Changes

Cohort / File(s)	Summary
Newsletter checkbox event handler libs/blog/newsletter/feature-newsletter/src/lib/feature-newsletter/newsletter.component.html	Removed click event handler (click)="form.controls.checkbox.value = !form.controls.checkbox.value" from custom checkbox element; spacebar keydown handler retained

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• HouseOfAngular/angular-love#451: Directly reverses this change by undoing the click-to-toggle handler previously added to the same custom checkbox element.

Suggested reviewers

• DamianBrzezinskiHoA
• DDonochVA

Poem

🐰 A click was removed, oh what a sight!
No more fingers triggering checkbox delight,
But spacebar still dances, as keyboards demand—
Accessibility reigns across the land! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The pull request title "fix: checkbox state feedback" is directly related to the changeset. The title indicates a fix to checkbox state behavior, which aligns with the actual change of removing the click event handler on the checkbox UI element. While the title could be more specific by directly mentioning the prevention of duplicated toggles, it clearly references the core component and behavior being modified, making it sufficiently descriptive for understanding the primary change at a glance.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/checkbox-state-feedback

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a1ab420 and 26d2a60.

📒 Files selected for processing (1)

• libs/blog/newsletter/feature-newsletter/src/lib/feature-newsletter/newsletter.component.html (0 hunks)

💤 Files with no reviewable changes (1)

• libs/blog/newsletter/feature-newsletter/src/lib/feature-newsletter/newsletter.component.html

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[valueadd-robot]

PR is detected, will deploy to dev environment

[valueadd-robot]

Deployed to dev environment
Branch: fix/checkbox-state-feedback
BFF URL: https://45270eae-blog-bff-dev.contact-ef8.workers.dev
Deploy URL: https://96bfe3ef.angular-love-client.pages.dev
Alias URL: https://fix-checkbox-state-feedback.angular-love-client.pages.dev