🎨 Palette: Add accessibility and tooltips to webhook header buttons

[NSEvent]

💡 What

Added .help() tooltips and .accessibilityLabel() modifiers to the icon-only plus.circle.fill and minus.circle.fill buttons in the Webhook Headers section of ActionMappingEditor.swift and ChordMappingSheet.swift.

🎯 Why

Icon-only buttons used for adding or removing items in a dictionary form were ambiguous to screen readers and lacked hover context for mouse users. This change ensures these interactive elements are clearly communicated to all users, making the dynamic form more accessible and intuitive.

♿ Accessibility

• Added Remove header label to minus buttons.
• Added Add header label to plus buttons.

Also updated .Jules/palette.md to document this learning about dynamic form lists.

---

PR created automatically by Jules for task 1136719797351930065 started by @NSEvent

Summary by CodeRabbit

• Bug Fixes
  • Added accessibility labels and tooltips to header add/remove buttons for improved screen reader support.

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: f170f25e-d2cc-43e7-afe2-4b3b41cf2cd7

📥 Commits

Reviewing files that changed from the base of the PR and between c5dbc5f and 340670d.

📒 Files selected for processing (3)

• .Jules/palette.md
• XboxControllerMapper/XboxControllerMapper/Views/MainWindow/ActionMappingEditor.swift
• XboxControllerMapper/XboxControllerMapper/Views/MainWindow/ChordMappingSheet.swift

---

📝 Walkthrough

Walkthrough

This PR adds accessibility metadata to icon-only add/remove buttons used for webhook header management across two SwiftUI view controllers. Each button now includes both a .help() hover tooltip and an .accessibilityLabel() for screen reader support.

Changes

Webhook header button accessibility

Layer / File(s)	Summary
Accessibility guidance documentation .Jules/palette.md	New palette entry documents the requirement to add .help(...) tooltips and .accessibilityLabel(...) to icon-only add/remove buttons in dynamic form sections.
ActionMappingEditor webhook header buttons XboxControllerMapper/XboxControllerMapper/Views/MainWindow/ActionMappingEditor.swift	Remove (minus icon) and add (plus icon) header buttons in the webhook headers section each receive .help() text and .accessibilityLabel() for tooltip and screen-reader support.
ChordMappingSheet webhook header buttons XboxControllerMapper/XboxControllerMapper/Views/MainWindow/ChordMappingSheet.swift	Remove and add buttons in the system command webhook header UI each receive matching .help() and .accessibilityLabel() accessibility modifiers.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• NSEvent/xbox-controller-mapper#45: Applies the same accessibility labeling pattern (.help() + .accessibilityLabel()) to icon-only action buttons in different views.
• NSEvent/xbox-controller-mapper#24: Similar accessibility improvement adding help text and accessibility labels to icon-only edit/delete/remove buttons.
• NSEvent/xbox-controller-mapper#49: Implements the same accessibility pattern by adding .help() and .accessibilityLabel() to other icon-only UI buttons in the application.

Poem

🐰 Icon buttons need a voice so clear,
Help text and labels, screen readers hear!
Across two views, the pattern flows,
Accessibility blooms wherever it goes! 🌟

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly describes the main change: adding accessibility labels and tooltips to webhook header buttons, which aligns with the actual modifications across all changed files.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch palette/add-accessibility-webhook-buttons-1136719797351930065

---

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.}