Skip to content

IBX-10573: Shared base for form control list components#24

Merged
dew326 merged 1 commit intomainfrom
IBX-10573
Sep 2, 2025
Merged

IBX-10573: Shared base for form control list components#24
dew326 merged 1 commit intomainfrom
IBX-10573

Conversation

@GrabowskiM
Copy link
Copy Markdown
Contributor

🎫 Issue IBX-10573

Description:

For QA:

Documentation:

@GrabowskiM GrabowskiM requested review from a team and Copilot September 1, 2025 08:09
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a shared base component for form control list components. It creates a reusable BaseInputsList component that can render lists of form inputs with configurable direction (horizontal/vertical) and optional label/helper text support.

  • Creates a generic BaseInputsList component with TypeScript support for type-safe item rendering
  • Adds direction configuration (horizontal/vertical) for flexible layout options
  • Includes comprehensive Storybook stories for documentation and testing

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/components/src/internal/partials/BaseInputsList/index.ts Main export file for the BaseInputsList component
packages/components/src/internal/partials/BaseInputsList/BaseInputsList.types.ts TypeScript type definitions and DIRECTION enum
packages/components/src/internal/partials/BaseInputsList/BaseInputsList.tsx Core component implementation with generic type support
packages/components/src/internal/partials/BaseInputsList/BaseInputsList.stories.tsx Storybook stories for component documentation
packages/assets/src/scss/styles.scss Import statement for new SCSS module
packages/assets/src/scss/_inputs-list.scss Styling for the inputs list component

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ezrobot ezrobot requested review from OstafinL, RopRaptor, albozek, dew326 and tischsoic and removed request for a team September 1, 2025 08:13
@dew326 dew326 merged commit 8ed0c1e into main Sep 2, 2025
11 of 12 checks passed
@dew326 dew326 deleted the IBX-10573 branch September 2, 2025 10:39
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.

5 participants