Skip to content

EPMRPP-111250 || Add box sizing property to the SegmentedControl#233

Merged
siarheirazuvalau merged 1 commit intodevelopfrom
bugfix/EPMRPP-111250-add-box-sizing-to-SegmentedControl
Feb 3, 2026
Merged

EPMRPP-111250 || Add box sizing property to the SegmentedControl#233
siarheirazuvalau merged 1 commit intodevelopfrom
bugfix/EPMRPP-111250-add-box-sizing-to-SegmentedControl

Conversation

@siarheirazuvalau
Copy link
Contributor

@siarheirazuvalau siarheirazuvalau commented Feb 3, 2026

Description

Added box-sizing css property to handle 100% width correctly. Also css was reformatted in accordance to project css formatting rules

Summary by CodeRabbit

  • Style
    • Updated internal styling organization for the segmented control component with no changes to user-visible appearance or functionality.

@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

Walkthrough

The pull request modifies the segmented control component's SCSS module file, adding box-sizing border-box property, removing inline comments from the button class, and reordering CSS properties in the icon class for consistency.

Changes

Cohort / File(s) Summary
Segmented Control Styling
src/components/segmentedControl/segmentedControl.module.scss
Added box-sizing: border-box to .segmented-control, removed inline comment headers from .button class, and reordered flexbox properties in .icon class. No functional changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested reviewers

  • AmsterGet
  • allaprischepa
  • maria-hambardzumian

Poem

🐰 A rabbit hops through styling rules so neat,
Box-sizing borders make the layout complete!
Comments fade away, properties align,
The segmented control looks fine, just fine! ✨

🚥 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 clearly and specifically identifies the main change: adding a box-sizing property to the SegmentedControl component. It accurately reflects the primary objective stated in the PR description.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bugfix/EPMRPP-111250-add-box-sizing-to-SegmentedControl

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.

@siarheirazuvalau siarheirazuvalau merged commit 6f090eb into develop Feb 3, 2026
2 checks passed
@siarheirazuvalau siarheirazuvalau deleted the bugfix/EPMRPP-111250-add-box-sizing-to-SegmentedControl branch February 3, 2026 12:29
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.

2 participants