Skip to content

add new filter in FrmStylesHelper::output_vars#2924

Open
Liviu-p wants to merge 1 commit intomasterfrom
feature/add-new-filter-to-style-helper
Open

add new filter in FrmStylesHelper::output_vars#2924
Liviu-p wants to merge 1 commit intomasterfrom
feature/add-new-filter-to-style-helper

Conversation

@Liviu-p
Copy link
Contributor

@Liviu-p Liviu-p commented Feb 3, 2026

Summary by CodeRabbit

  • Enhancements
    • Expanded plugin and addon customization capabilities for CSS style settings

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 3, 2026

📝 Walkthrough

Walkthrough

A new filter hook frm_style_settings_output_css_vars was added to the FrmStylesHelper::output_vars method, enabling external addons and plugins to modify CSS style settings before they are processed into CSS variables.

Changes

Cohort / File(s) Summary
Filter Hook Addition
classes/helpers/FrmStylesHelper.php
Introduced apply_filters call in output_vars method to pass CSS style settings through the frm_style_settings_output_css_vars filter hook, allowing external modification of settings prior to CSS variable rendering.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A filter hook hops into the code,
CSS variables find the open road,
Now addons can tweak with graceful ease,
Styles bend and flow as they please! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and clearly summarizes the main change: adding a new filter hook in the FrmStylesHelper::output_vars method, which matches the changeset perfectly.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/add-new-filter-to-style-helper

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.


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

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

@deepsource-io
Copy link

deepsource-io bot commented Feb 3, 2026

Here's the code health analysis summary for commits d66c2c8..74cd978. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource PHP LogoPHP✅ SuccessView Check ↗
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@Liviu-p Liviu-p requested a review from Crabcyborg February 9, 2026 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant