fix(settings): securities-provider checkmarks invisible in dark mode

[vlnd0]

Problem

On the hosting settings page ("Securities Providers"), the checkboxes are invisible when checked in dark mode — a checked box shows a white check on a white fill, so there's no visual indication of which providers are enabled.

Cause

app/views/settings/hostings/_provider_selection.html.erb styled the checkbox with raw Tailwind utilities:

class="rounded border-primary text-primary focus:ring-primary"

In dark mode text-primary resolves to white. The browser's checked state fills the box with currentColor and draws the checkmark in white → white-on-white.

Every other checkbox in the app uses the theme-aware design-system component (settings/preferences, transaction filters, goals, trades, etc.):

class="checkbox checkbox--light"

.checkbox--light has a @variant theme-dark block that renders a dark check on a light fill in dark mode.

Fix

Use the design-system checkbox checkbox--light classes, consistent with the rest of the app. One-line change.

Test

• Light mode: unchanged.
• Dark mode: checked providers now show a visible dark checkmark on a light fill.

Summary by CodeRabbit

• Style
  • Updated the visual styling of checkboxes in the hosting provider selection interface for a lighter appearance.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e2facc72-b96f-4439-9fa7-87eda9a8fa4b

📥 Commits

Reviewing files that changed from the base of the PR and between fdcd0c7 and 2f70704.

📒 Files selected for processing (1)

• app/views/settings/hostings/_provider_selection.html.erb

---

📝 Walkthrough

Walkthrough

A single CSS class attribute on a checkbox input in app/views/settings/hostings/_provider_selection.html.erb is changed from inline Tailwind utility classes (rounded border-primary text-primary focus:ring-primary) to component classes (checkbox checkbox--light).

Changes

Checkbox styling in provider selection

Layer / File(s)	Summary
Checkbox class replacement app/views/settings/hostings/_provider_selection.html.erb	The securities-provider checkbox input's class attribute is updated from rounded border-primary text-primary focus:ring-primary to checkbox checkbox--light.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A checkbox once wore Tailwind's coat,
With border-primary and a ring to note.
Now dressed in checkbox--light instead,
A cleaner class upon its head. 🐇✨
One line changed — the rabbit hops ahead!

🚥 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 accurately describes the main change: fixing invisible checkmarks in dark mode for the securities-provider section.
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 unit tests (beta)

• Create PR with unit tests

---

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

[superagent-security]

Superagent didn't find any vulnerabilities or security issues in this PR.

[savionlee]

this would be great!

[vlnd0]

@jjmata if all ok, i suggest to merge