Skip to content

Conversation

@AyushSharma72
Copy link

@AyushSharma72 AyushSharma72 commented Oct 17, 2025

What does this PR do?

This PR fixes a responsive layout issue in the website where the text "security and compliance" was overflowing on smaller screens.
The problem was caused by the whitespace-nowrap utility class being applied at all screen sizes.
The fix restricts it to medium screens and above (sm:whitespace-nowrap) to ensure proper wrapping on mobile devices while maintaining layout consistency on larger screens.


Test Plan

  1. Open the website home page scroll to section containing the "security and compliance" heading.
  2. Resize the browser window or test on a mobile view (below the 370px breakpoint).
  3. Verify that:
    • The text now wraps correctly on smaller screens.
    • It remains on a single line for medium and larger screens.
  4. No visual regressions or layout shifts occur elsewhere. @eldadfux

screenshots :
before:
issue

after fix:
image

no change in the desktop view
image


Related PRs and Issues

issue -> #2495


Have you read the Contributing Guidelines on issues?

✅ Yes, I have read and followed the contributing guidelines.

Summary by CodeRabbit

  • Style
    • Text in the security and compliance section now wraps responsively based on screen size instead of remaining fixed.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 17, 2025

Walkthrough

This pull request modifies the features.svelte component to make the "security and compliance" text's wrapping behavior responsive. The change replaces the whitespace-nowrap class with sm:whitespace-nowrap, meaning the text will wrap on smaller screens and only prevent wrapping on small+ breakpoints.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

This is a single-file, CSS-only modification applying a responsive class substitution. The change is purely cosmetic styling with no logic modifications, structural changes, or complex interactions to evaluate. Review effort is minimal as only basic verification of the responsive behavior intent is needed.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The pull request title "fix: prevent text overflow by applying sm:whitespace-nowrap in the features section span tag on marketing page" is clearly and directly related to the main change in the pull request. The changeset modifies the "security and compliance" text styling to use a responsive whitespace-nowrap utility class instead of applying it at all screen sizes, which directly addresses text overflow issues on smaller screens. The title accurately describes both the problem being solved (text overflow) and the solution approach (responsive whitespace utility), and it provides specific context about which section and file is affected, allowing a developer scanning the history to quickly understand the primary purpose of this change.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

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