Skip to content

feat: Added support for more than 4 columns for column-layout component - #5086

Open
vikaskale2722 wants to merge 5 commits into
mainfrom
dev-v3-vgkale-column-layout-more-columns
Open

vikaskale2722 wants to merge 5 commits into
mainfrom
dev-v3-vgkale-column-layout-more-columns

Conversation

@vikaskale2722

Copy link
Copy Markdown
Member

Description

The Column layout component only supported 4 columns before this implementation. This PR enables support for more than 4 columns in the component. There was a workout to provide more than 4 columns, if we provide minColumnWidth prop. This implementation enables to allow more columns without providing minColumnWidth

Related links, issue #62223

How has this been tested?

Unit test
Manual test
Dry Run Build : #

Review checklist

The following items are to be evaluated by the author(s) and the reviewer(s).

Correctness

  • Changes include appropriate documentation updates.
  • Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
  • Changes do not include unsupported browser features, see CONTRIBUTING.md.
  • Changes were manually tested for accessibility, see accessibility guidelines.

Security

Testing

  • Changes are covered with new/existing unit tests?
  • Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov

codecov Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.74%. Comparing base (324fda3) to head (014a178).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5086   +/-   ##
=======================================
  Coverage   97.73%   97.74%           
=======================================
  Files         996      996           
  Lines       32224    32227    +3     
  Branches    11910    11911    +1     
=======================================
+ Hits        31495    31499    +4     
+ Misses        684      683    -1     
  Partials       45       45           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@vikaskale2722
vikaskale2722 added this pull request to the merge queue Oct 28, 2026
@vikaskale2722
vikaskale2722 removed this pull request from the merge queue due to a manual request Oct 28, 2026

This branch was successfully deployed

3 active deployments
dev-pages-react18 — 014a178f Deployed Oct 26, 2026 by vikaskale2722 via deploy (React 18) / deploy #2975
dev-pages-react16 — 014a178f Deployed Oct 26, 2026 by vikaskale2722 via deploy (React 16) / deploy #2975
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants