Uh oh!
There was an error while loading. Please reload this page.
Bugfix: Use mobile search on big tablets - #743
Conversation
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughMobile search now uses a shared responsive query for toolbox and workspace layouts, including coarse-pointer devices up to 899px. CSS applies matching breakpoints and primary text colors to mobile search elements. ChangesMobile search responsive behavior
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
Summary
Large tablets (e.g. Samsung Galaxy A11) weren't getting the mobile version of workspace search, making it very difficult to move between results. Gate this at 768px OR up to 900px if it's touch.
Bonus extra: fixed wrong contrast text in dark mode 2 which meant search text was black on dark grey.
AI usage
Claude Sonnet 5 used throughout but the plan made by me.
Summary by CodeRabbit
Bug Fixes
Style