Reduce size of payment settings button

[Crabcyborg]

Summary by CodeRabbit

• Style
  • Adjusted Payments page tab spacing for a cleaner layout.
  • Updated the tab container and button vertical padding to improve alignment and consistency.

[deepsource-io]

DeepSource Code Review

We reviewed changes in 148700f...d54586f on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade

Security   Reliability   Complexity   Hygiene

Code Review Summary

Analyzer	Status	Updated (UTC)	Details
PHP		Jul 7, 2026 7:21p.m.	Review ↗
JavaScript		Jul 7, 2026 7:21p.m.	Review ↗

---

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

[coderabbitai]

📝 Walkthrough

Walkthrough

Modified CSS padding rules for the .frm-payments-settings-button nav tab element in _nav-tabs.scss, changing outer container padding shorthand and adding explicit padding-top/padding-bottom overrides on the nested .button element.

Changes

Payments Nav Tab Styling

Layer / File(s)	Summary
Nav tab button padding update resources/scss/admin/components/_nav-tabs.scss	Changed container padding from 8px 12px to 0 12px 8px and added padding-top/padding-bottom: var(--gap-2xs) !important on the nested .button.

Estimated code review effort: 1 (Trivial) | ~3 minutes

🚥 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 clearly matches the main change: reducing the Payments settings button's vertical size through padding adjustments.
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.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch reduce_size_of_payment_settings_button

---

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

[coderabbitai]

🧹 Nitpick comments (1)

resources/scss/admin/components/_nav-tabs.scss (1)

109-121: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Consider avoiding !important for the padding overrides.

Lines 119-120 use !important to force .button padding. If this is only to beat specificity from a base .button class, a more targeted selector (e.g. increasing specificity via nesting/parent class) would be more maintainable than !important, which can make future overrides harder to reason about.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@resources/scss/admin/components/_nav-tabs.scss` around lines 109 - 121, The
`.frm-payments-settings-button .button` padding overrides currently rely on
`!important`, which should be removed in favor of a more maintainable selector
strategy. Update the SCSS in the `.frm-payments-settings-button` block to
increase specificity through nesting or a more targeted parent-based selector so
the base `.button` styles are overridden without `!important`.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@resources/scss/admin/components/_nav-tabs.scss`:
- Around line 109-121: The `.frm-payments-settings-button .button` padding
overrides currently rely on `!important`, which should be removed in favor of a
more maintainable selector strategy. Update the SCSS in the
`.frm-payments-settings-button` block to increase specificity through nesting or
a more targeted parent-based selector so the base `.button` styles are
overridden without `!important`.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 08c6485e-fa60-457d-948f-53d6ca6f1840

📥 Commits

Reviewing files that changed from the base of the PR and between 148700f and d54586f.

📒 Files selected for processing (2)

• css/frm_admin.css
• resources/scss/admin/components/_nav-tabs.scss