Skip to content

feat(ui): Update ModalBodyHeader and Table components#363

Merged
kylengn merged 6 commits intomainfrom
feat/update-modal-body-header
Jan 30, 2026
Merged

feat(ui): Update ModalBodyHeader and Table components#363
kylengn merged 6 commits intomainfrom
feat/update-modal-body-header

Conversation

@kylengn
Copy link
Contributor

@kylengn kylengn commented Jan 29, 2026

Summary by CodeRabbit

  • New Features
    • ModalBodyHeader component now supports an optional children prop, enabling custom content to be rendered within the modal header area. This addition provides greater flexibility in header design, allowing custom elements and text alongside existing header content for more tailored modal header layouts.

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

@kylengn kylengn self-assigned this Jan 29, 2026
Copilot AI review requested due to automatic review settings January 29, 2026 04:37
@vercel
Copy link

vercel bot commented Jan 29, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
react-ui Ready Ready Preview, Comment Jan 29, 2026 8:48am

Request Review

@coderabbitai
Copy link

coderabbitai bot commented Jan 29, 2026

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

Walkthrough

This PR extends the ModalBodyHeader component by adding an optional children prop, enabling additional elements to be rendered within the header area after the description content. A corresponding changeset entry documents the minor release.

Changes

Cohort / File(s) Summary
Changeset
.changeset/wild-rabbits-kick.md
New changeset entry for @shipfox/react-ui minor release documenting ModalBodyHeader component update.
ModalBodyHeader Component
libs/react/ui/src/components/modal/modal.tsx
Added optional children?: ReactNode prop to ModalBodyHeaderProps interface; updated component function signature to destructure and render children after description content.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Suggested reviewers

  • EnzalRad
  • dvxam
  • noe-charmet

Poem

🐰 A modal header, once confined to words,
Now opens up to child components heard,
With flexibility fresh as morning clover,
The ModalBodyHeader's versatility grows over! 🌿✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ⚠️ Warning The PR title mentions updating both ModalBodyHeader and Table components, but the changeset only modifies ModalBodyHeader. The Table component is not present in the actual changes. Update the title to 'feat(ui): Update ModalBodyHeader component' to accurately reflect the actual changes in this PR.
✅ Passed checks (1 passed)
Check name Status Explanation
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.


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.

Copy link

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 adds a children prop to the ModalBodyHeader component, allowing consumers to render additional content within the component beyond the existing image, title, and description.

Changes:

  • Added optional children?: ReactNode prop to ModalBodyHeaderProps type definition
  • Added children parameter to ModalBodyHeader function signature
  • Rendered children at the end of the component's JSX

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
libs/react/ui/src/components/modal/modal.tsx Extended ModalBodyHeader component to accept and render children prop
.changeset/wild-rabbits-kick.md Added changeset documenting the minor version bump for this enhancement

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@argos-ci
Copy link

argos-ci bot commented Jan 29, 2026

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) 👍 Changes approved 56 changed Jan 29, 2026, 8:49 AM

@kylengn kylengn changed the title feat(ui): Update ModalBodyHeader component feat(ui): Update ModalBodyHeader and Table components Jan 29, 2026
@kylengn kylengn merged commit e0e55ad into main Jan 30, 2026
5 checks passed
@kylengn kylengn deleted the feat/update-modal-body-header branch January 30, 2026 03:33
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

Comments