Exclude single style custom css from css variables

[Crabcyborg]

No description provided.

[coderabbitai]

Warning

Rate limit exceeded

@Crabcyborg has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 12 minutes and 41 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 7eb9f24 and 8b1c426.

📒 Files selected for processing (1)

• classes/helpers/FrmStylesHelper.php

Walkthrough

A CSS variable key single_style_custom_css was added to the list of keys excluded from output in the FrmStylesHelper class. This is a single addition to an internal array within the get_style_keys_to_remove_from_output_vars method.

Changes

Cohort / File(s)	Summary
CSS Variable Exclusion List classes/helpers/FrmStylesHelper.php	Added single_style_custom_css to the internal array of keys removed from CSS variable output

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• Stop printing global custom css when single style custom css is not set #2703: Modifies handling of single_style_custom_css in FrmStylesController, directly related to this exclusion from CSS variables.
• Do not output invalid css vars #2296: Expands CSS variable key exclusions in FrmStylesHelper by adding use_base_font_size, following the same pattern as this change.
• Exclude bg image id from style output vars #2651: Modifies get_style_keys_to_remove_from_output_vars() to exclude bg_image_id, directly related to the same method being extended here.

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	No pull request description was provided by the author, making it impossible to evaluate whether it relates to the changeset.	Add a description explaining why this CSS variable needs to be excluded and any impact on styling or CSS variable output.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: excluding 'single_style_custom_css' from CSS variables output, which matches the actual code modification.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

---

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