Summary of Changes

Hello @GinWU05, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request significantly enhances the Ant Design notification component by introducing flexible and powerful customization options for its classNames and styles. Developers can now define these properties as static objects or dynamic functions, enabling fine-grained control over the styling of various semantic parts of a notification. This change improves the component's adaptability to diverse design requirements and provides a more robust styling API.

Highlights

• Enhanced Notification Customization: The notification component now supports more granular customization of its classNames and styles properties. These can be provided as either static objects or dynamic functions, allowing precise control over the appearance of individual semantic elements like the root container, title, description, actions, and icon.
• Refactored Semantic Merging Logic: The useMergeSemantic hook has been refactored. Its core merging logic is now exposed as standalone mergeSemanticClassNames and mergeSemanticStyles functions, and a non-hook mergeSemantic function. The useMergeSemantic hook now wraps this non-hook version with React.useMemo for optimized performance.
• Updated Component Implementation: The PurePanel.tsx and useNotification.tsx files have been updated to leverage the new mergeSemantic functionality, ensuring that classNames and styles are correctly applied and merged from various sources, including component props and ConfigProvider settings.
• Comprehensive Test Coverage: A new test file (semantic.test.tsx) has been added, including extensive test cases to validate the new customization features. These tests cover object-based and function-based classNames/styles, integration with ConfigProvider, merging priority, and graceful handling of empty inputs.
• Documentation and Demo Updates: The documentation for the notification component (index.en-US.md, index.zh-CN.md) has been updated to reflect the new classNames and styles props. A new demo (style-class.tsx, style-class.md) has also been added to showcase practical usage of these customization options.

Code Review

This pull request introduces a valuable feature for customizing notifications by allowing classNames and styles to be provided as functions. The refactoring of useMergeSemantic into pure functions and a corresponding hook is a solid architectural improvement. I've identified a few areas for improvement concerning performance and code redundancy within the new utility hooks. Specifically, there's some redundant logic in useMergeSemantic, and the dependency arrays in useSemanticClassNames and useSemanticStyles could be optimized to ensure memoization is effective. The new tests are comprehensive and well-written, providing good coverage for the new functionality.

Pull Request Overview

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

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