-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Fix lazy load error in Plasmo #5191
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
base: canary
Are you sure you want to change the base?
Conversation
Fix lazy load error in Plasmo
|
@Xheldon is attempting to deploy a commit to the HeroUI Inc Team on Vercel. A member of the Team first needs to authorize it. |
WalkthroughThe change in this pull request modifies how the Changes
Sequence Diagram(s)sequenceDiagram
participant TP as ToastProvider
participant LF as loadFeatures
participant DM as domMax
TP->>LF: Call loadFeatures()
LF->>DM: Return static reference
DM-->>LF: Immediately resolve with domMax
LF-->>TP: Return resolved promise with domMax
Tip ⚡💬 Agentic Chat (Pro Plan, General Availability)
📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (1)
🔇 Additional comments (2)
✨ Finishing Touches
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. 🪧 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
Documentation and Community
|
Closes #
📝 Description
When using the HeroUI Toast component in Plasmo, there's a bug that makes the page not render. After debugging, we found it's because ToastProvider lazy loads framer-motion's domMax. Check out the related error:
PlasmoHQ/plasmo#1180
PlasmoHQ/plasmo#1241
⛳️ Current behavior (updates)
You can check the image descriptions in the two issues mentioned above. This problem causes the page to not render at all, with an error saying "xxx is not a function."
Maybe it's an issue with Parcel used by Plasmo, making it unable to dynamically import some modules. But if HeroUI could provide a usable update ASAP, developers would really appreciate it.
🚀 New behavior
Toast will work as expect.
💣 Is this a breaking change (Yes/No):
No
📝 Additional Information
NO
Summary by CodeRabbit