New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: notification style width #45681
Conversation
Run & review this pull request in StackBlitz Codeflow. |
size-limit report 📦
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
size-limit 又挂了,上周刚加了1kb |
一直增加体重,是时候瘦一下身了 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #45681 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 699 700 +1
Lines 11815 11826 +11
Branches 3150 3151 +1
=========================================
+ Hits 11815 11826 +11
☔ View full report in Codecov by Sentry. |
多加个文件就大 200b |
Thanks for fixing this! |
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
close #45678
💡 Background and solution
📝 Changelog
style.width
not work.style.width
无效的问题。☑️ Self-Check before Merge
🚀 Summary
🤖 Generated by Copilot at d6914e4
This pull request adds a new feature to the notification component, allowing it to use a pure panel style. It also introduces a theme system using alias tokens and refactors the style generation functions. It updates the test and demo files accordingly.
🔍 Walkthrough
🤖 Generated by Copilot at d6914e4
components/notification/style/stack.ts
(link)components/notification/__tests__/index.test.tsx
(link)components/notification/demo/basic.tsx
to show how to customize the notification component (link)components/notification/__tests__/index.test.tsx
for readability (link)