Skip to content

Fix HTML field ignoring layout classes#2843

Open
AbdiTolesa wants to merge 1 commit intomasterfrom
fix/html_field_layout
Open

Fix HTML field ignoring layout classes#2843
AbdiTolesa wants to merge 1 commit intomasterfrom
fix/html_field_layout

Conversation

@AbdiTolesa
Copy link
Copy Markdown
Contributor

@AbdiTolesa AbdiTolesa commented Jan 20, 2026

Related issue: https://secure.helpscout.net/conversation/3197229748/245409

Field setup:
image

Before:
image

After:
image

Summary by CodeRabbit

Release Notes

  • Style
    • Improved text wrapping behavior in form field containers to prevent text overflow and enhance readability of long text in HTML container fields.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jan 20, 2026

📝 Walkthrough

Walkthrough

A CSS styling rule is enhanced to add word-wrapping behavior to HTML form container fields. The overflow-wrap: break-word; property is appended to an existing selector in the custom theme stylesheet, affecting text layout for form descriptions when the form_desc_size parameter is configured.

Changes

Cohort / File(s) Summary
CSS Styling
css/custom_theme.css.php
Adds overflow-wrap: break-word; property to .with_frm_style .frm_form_field.frm_html_container selector to control text wrapping behavior

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

run analysis, run tests

Suggested reviewers

  • lauramekaj1
  • truongwp
  • Crabcyborg

Poem

🐰 A whisker-thin change, so small and so sweet,
CSS rules now make text wrapping complete!
One line of code keeps the form looking neat,
Breaking long words in a tidy retreat! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding CSS to fix HTML field layout issues by enabling proper text wrapping in HTML container form fields.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing touches
  • 📝 Generate docstrings

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.

@lauramekaj1
Copy link
Copy Markdown
Contributor

Hi @AbdiTolesa,
The same issue is still showing for me:
image

@AbdiTolesa
Copy link
Copy Markdown
Contributor Author

Hi @AbdiTolesa, The same issue is still showing for me: image

Hi @lauramekaj1 I forget to mention that you need to regenerate your css files by going to Formidable > Style > Update. Have you done that?

Copy link
Copy Markdown
Contributor

@lauramekaj1 lauramekaj1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AbdiTolesa I tested this and verified that the issue is fixed. Thank you!

@AbdiTolesa AbdiTolesa requested a review from truongwp January 23, 2026 18:53
Copy link
Copy Markdown
Contributor

@truongwp truongwp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. Thanks @AbdiTolesa!

@truongwp truongwp requested a review from Crabcyborg February 2, 2026 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants