fix: keep package name visible#1344
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
📝 WalkthroughWalkthroughThe pull request modifies layout classes in the FacetCard component. Specifically, two spans within the component have their CSS class assignments adjusted to alter flex shrinking and width constraints. The package name span transitions from a flex-shrink configuration with minimum width constraint to a no-shrink state. The value container span changes from a flex-shrink-0 setup to a minimum width constraint combined with text-end alignment. These modifications affect horizontal space distribution and text positioning without altering the component's rendering logic, data processing, or conditional behaviour. Suggested reviewers
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. Comment |
|
Thanks @43081j . |
Look, our engines in vitest is really long. For that reason the package name is not visible in mobile version due to long values. I think we can improve it.
Before:

After:
