fix: remove install size tooltip when size is not available#1721
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
📝 WalkthroughWalkthroughThe change adds conditional rendering to the install size tooltip component in the package page. The TooltipApp component is now wrapped with a v-if directive that checks for the presence of sizeTooltip text, preventing the tooltip from being rendered when it contains no content. This addresses the issue where empty tooltips were appearing on hover by ensuring the component only renders when there is actual tooltip text to display. Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
50b89f5 to
9227139
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
🔗 Linked issue
resolves #1694
🧭 Context
There's an empty tooltip when the install size isn't available for a given package.
📚 Description
Adds a
v-ifif the tooltip text is empty so we don't render it!