-
Notifications
You must be signed in to change notification settings - Fork 68
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: 修复effect style无__cssinjs_instance__的问题 #202
base: master
Are you sure you want to change the base?
Conversation
📝 WalkthroughWalkthrough此次更改主要集中在 Changes
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🔇 Additional comments (2)src/hooks/useStyleRegister.tsx (2)
这个改动通过将缓存实例ID赋值给
这个改动与层样式的修复保持一致,通过为 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? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #202 +/- ##
=======================================
Coverage 95.07% 95.08%
=======================================
Files 33 33
Lines 2883 2887 +4
Branches 456 456
=======================================
+ Hits 2741 2745 +4
Misses 142 142 ☔ View full report in Codecov by Sentry. |
effect style在无__cssinjs_instance__属性时,可能会被赋予新的instance id,导致被插入到head中,在qiankun微前端场景下被插入到了qiankun-head中,出现异常。
该pr是解决这个问题。