fix: nullish coalescing in sidebar collapse logic#622
fix: nullish coalescing in sidebar collapse logic#622ishanloya wants to merge 1 commit intofrappe:mainfrom
Conversation
When the || operator is used, isMobile.value is always true on mobile so the Sidebar is always collapsed on mobile.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughUpdated the collapse state computation in the Sidebar component to use nullish-coalescing operator instead of logical OR. The 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
When the || operator is used, isMobile.value is always true on mobile so the Sidebar is always collapsed on mobile.
Summary by CodeRabbit