Closed jsartisan closed 1 day ago
The changes introduce enhancements to the _Sheet
and _Sidebar
components within the design system. New props for handling transition events, such as onEnter
, onExit
, and their counterparts, have been added to improve control over animations. The sidebar's internal state management has been updated to a more flexible system, allowing for various states like "expanded" and "collapsed". Additionally, new styling and structural modifications have been made to improve the user interface and responsiveness.
File | Change Summary |
---|---|
.../Sheet/src/Sheet.tsx |
Added onEnter and onExit props to _Sheet component; updated CSSTransition to utilize these props for transition event handling; modified destructuring of props. |
.../Sheet/src/types.ts |
Updated SheetProps interface to include onEnter , onExit , and onExited callback properties. |
.../Sidebar/src/Sidebar.tsx |
Introduced new animation event props (onEnter , onEntered , onExit , onExited ); updated internal state management; encapsulated rendering logic in a new SidebarContent component. |
.../Sidebar/src/SidebarContent.tsx |
Defined SidebarContentProps interface; updated function signature to accept new props; enhanced rendering structure with Flex components. |
.../Sidebar/src/SidebarProvider.tsx |
Transitioned to a generalized state management system using SidebarState ; updated props and internal state handling; reorganized imports for clarity. |
.../Sidebar/src/styles.module.css |
Modified transition effects for smoother animations; added new styles for .sidebarHeader and adjusted existing styles based on component states. |
.../Sidebar/src/types.ts |
Updated SidebarContextType , SidebarProviderProps , and SidebarProps interfaces to reflect new state management and added optional properties for transition handling. |
.../Sidebar/stories/Sidebar.stories.tsx |
Removed SidebarContent from certain stories; added new story WithRenderProps to demonstrate render prop pattern; updated args to include default title for the Sidebar . |
_Sheet
component enhancements, similar to modifications in the _Sidebar
component for handling animation events.Sidebar
component alongside the Sheet
component enhancements.Enhancement
, Design System Product
, Widgets Product
, Task
, IDE Pod
, ADS Components
, ok-to-test
π In the land of code, where components reside,
New props were added, with transitions to guide.
The sidebar now dances, the sheet glides with grace,
Enhancements abound, in this digital space!
So letβs celebrate changes, both big and small,
For each line of code, weβre having a ball! π
[!WARNING]
Rate limit exceeded
@jsartisan has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 15 minutes and 5 seconds before requesting another review.
β How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.π¦ How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://docs.coderabbit.ai/faq) for further information.π₯ Commits
Reviewing files that changed from the base of the PR and between 0fe97bbe6d4e0f8723766e9551d0e38e233bd177 and a8170dfe131850c1a3f7a46870914d9dbc30624d.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary by CodeRabbit
Release Notes
New Features
_Sheet
and_Sidebar
components with new props for handling transition events (onEnter
,onExit
).SidebarContent
component for better organization of sidebar content.Sidebar
demonstrating render prop usage.Improvements
SidebarProvider
for more flexible sidebar behavior.Bug Fixes
Automation
/ok-to-test tags="@tag.Sanity"
:mag: Cypress test results
Communication
Should the DevRel and Marketing teams inform users about this change?
Summary by CodeRabbit